-
-
Notifications
You must be signed in to change notification settings - Fork 170
Closed
Description
@gibfahn @jbergstroem @rvagg @mhdawson (or anyone else with access to ci-release): can you take a look at https://ci-release.nodejs.org/job/iojs+release/jobConfigHistory/showDiffFiles?timestamp1=2017-06-21_18-30-42×tamp2=2017-06-28_13-11-55 ? I believe the change is right, but it's better to have a review.
It became necessary after nodejs/node#13900 . I believe it is simpler to have DISTTYPEDIR set in the release script, but have no strong feelings about this.
This is currently under test with a nightly: https://ci-release.nodejs.org/job/iojs+release/1797/ (replaces the one that was automatically started and aborted by me, the job was not fixed yet). EDIT: Nightly was correctly published.
Metadata
Metadata
Assignees
Labels
No labels