Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update links for AIX and Linux on Z downloads #940

Merged
merged 1 commit into from
Oct 27, 2016

Conversation

mhdawson
Copy link
Member

@mhdawson mhdawson commented Oct 6, 2016

NOTE: This should not land until 6.X becomes LTS. This is because binaries exist only for 6.X and later and I've not special cases LTS or Current. Once 6.X becomes LTS then binaries will be available for both LTS and Current.

I'd like to get this reviewed in advance so that ideally it can land on the same day that 6.X becomes LTS.

Now that binaries are being built as part of release:

  • Update the links for AIX and Linux on Z on the download
    page so that they point to the binaries available on
    the community download site instead of the
    external IBM download page.
  • Add AIX, Linux s390x and Linux PPC BE to the helper
    used to generate release announcements.

Now that binaries are being built as part of release:

- Update the links for AIX and Linux on Z on the download
  page so that they point to the binaries available on
  the community download site instead of the
  external IBM download page.
- Add AIX, Linux s390x and Linux PPC BE to the helper
  used to generate release announcements.
@mhdawson
Copy link
Member Author

mhdawson commented Oct 7, 2016

@rvagg can you review as you seem to have done the last updates to: scripts/helpers/downloads.js

Copy link
Member

@jasnell jasnell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rvagg
Copy link
Member

rvagg commented Oct 12, 2016

OK, yeah, lgtm I guess this should work when we have LTS=6 and Current=7, I don't think there's another way of getting that page for older builds. I'd like someone from @nodejs/website to sign-off though. Perhaps @phillipj?

@thefourtheye
Copy link
Contributor

Changes look okay. Was there any official communication stating that Node.js officially supports these builds?

@mhdawson
Copy link
Member Author

@thefourtheye we were just formalizing under nodejs/node#8922 so no formal announce yet. I'm hoping it will be part of the initial 6.X release/announcement as we have achieved consensus and its just down to getting the doc PR ready to land.

@fhemberger I know you commented before care to review as website rep?

Copy link
Contributor

@zlatanvasovic zlatanvasovic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :shipit:

@fhemberger
Copy link
Contributor

@mhdawson LGTM as well.

@mhdawson
Copy link
Member Author

Ok, seems we are now back to having LTS and Current and we should be good to land this. Last chance to object, otherwise I'll land this later today.

@MylesBorins
Copy link
Contributor

LGTM, sorry we missed this on Tuesday

@fhemberger fhemberger merged commit 005c719 into nodejs:master Oct 27, 2016
@mhdawson
Copy link
Member Author

@fhemberger thanks.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants