Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1171)

Issue 338790044: [plaso] Fix windows release script

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 week, 5 days ago by romaing
Modified:
1 week, 2 days ago
Reviewers:
Joachim Metz, onager
CC:
jberggren, kiddi, log2timeline-dev_googlegroups.com, aaronp
Visibility:
Public.

Description

[plaso] Fix windows release script

Patch Set 1 #

Total comments: 3

Patch Set 2 : better errors #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -6 lines) Patch
M config/windows/make_release.ps1 View 1 3 chunks +26 lines, -6 lines 0 comments Download

Messages

Total messages: 6
romaing
1 week, 5 days ago (2017-11-09 14:46:27 UTC) #1
Joachim Metz
https://codereview.appspot.com/338790044/diff/1/config/windows/make_release.ps1 File config/windows/make_release.ps1 (right): https://codereview.appspot.com/338790044/diff/1/config/windows/make_release.ps1#newcode32 config/windows/make_release.ps1:32: If ( $LastExitCode -ne 0 ) { maybe add ...
1 week, 5 days ago (2017-11-09 18:33:31 UTC) #2
romaing
https://codereview.appspot.com/338790044/diff/1/config/windows/make_release.ps1 File config/windows/make_release.ps1 (right): https://codereview.appspot.com/338790044/diff/1/config/windows/make_release.ps1#newcode32 config/windows/make_release.ps1:32: If ( $LastExitCode -ne 0 ) { On 2017/11/09 ...
1 week, 5 days ago (2017-11-10 10:39:36 UTC) #3
romaing
Code updated.
1 week, 5 days ago (2017-11-10 11:11:41 UTC) #4
romaing
https://codereview.appspot.com/338790044/diff/1/config/windows/make_release.ps1 File config/windows/make_release.ps1 (right): https://codereview.appspot.com/338790044/diff/1/config/windows/make_release.ps1#newcode32 config/windows/make_release.ps1:32: If ( $LastExitCode -ne 0 ) { On 2017/11/10 ...
1 week, 5 days ago (2017-11-10 11:12:03 UTC) #5
Joachim Metz
1 week, 2 days ago (2017-11-13 06:07:15 UTC) #6
LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld c59f3c4