FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Refactor mac installer (built on #2571) by rvagg · Pull Request #5656 · nodejs/node · GitHub

/ node Public

Refactor mac installer (built on #2571) - #5656

Closed
rvagg wants to merge 22 commits into
nodejs:masterfrom
rvagg:fhemberger-refactor-mac-installer-rebase
ConversationCommits (22)ChecksFiles changed
Closed

Refactor mac installer (built on #2571)#5656
rvagg wants to merge 22 commits into
nodejs:masterfrom
rvagg:fhemberger-refactor-mac-installer-rebase

Conversation

rvagg commented Mar 11, 2016

Copy link
Copy Markdown
Member

I've spent some more quality time with Packages today, trying to wrangle this into something closer to ready. Building on the work of @remixz work in #776 and @fhemberger's work in #2571.

  • Remove the uninstall option - while I got it to work nicely and cleaned up the script to fully remove everything, the installer that Packages makes is not really suitable for an uninstall option, it still says it's "installing", the button is "install", the summary at the end still says it's installed Node & npm, it's basically a hack to make an uninstaller possible. We should consider putting this option on the wiki or even installing it somewhere in /usr/local/ that's not in PATH but we can point users to so they can run it themselves.
  • Switched to the new logo using the commit on master already
  • Made the "Install Node.js" option mandatory, Packages has some depend-on functionality for selecting options but there is no way to make it so you have to have Node.js if you select npm, you can make the select box turn on but you can turn it off.
  • Changed install.py to add new NODE_INSTALL_NODE_ONLY, NODE_INSTALL_NPM_ONLY and NODE_INSTALL_HEADERS_ONLY (was HEADERS_ONLY) env vars so it can install npm to a different location to node so that the options work properly and you can actually opt to not install npm (yes, it works now!).
  • Made it so you have to click "Customize" to get to the checkboxes to optionally install npm.
  • Made the LICENSE an RTF, it looks so so much nicer now that it's formatted.
  • Moved the .pkg to the top-level instead of out/, which is what the current installer does and what Jenkins expects.

Commits are all over the place and need to be compacted obviously.

Will post a test build soon, got to sort out the slaves again.

mscdex added install Issues and PRs related to the installers. macos Issues and PRs related to the macOS platform / OSX. labels Mar 11, 2016
Fishrock123 reviewed Mar 11, 2016
View reviewed changes
Comment thread Makefile Outdated

Fishrock123 Mar 11, 2016

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

no ?=?

Fishrock123 commented Mar 11, 2016

Copy link
Copy Markdown
Contributor

As a note, I got a few whitespace errors when applying.

fhemberger mentioned this pull request Mar 12, 2016
Closed

rvagg commented Mar 16, 2016

Copy link
Copy Markdown
Member Author

OK, some more adjustments and I've put Packages on both OSX build machines now (was only on one before). Now I have a test package to share! Grab it at https://nodejs.org/download/test/v6.0.0-test20160316798c9dc5b3/node-v6.0.0-test20160316798c9dc5b3.pkg to experience the new goodness.

rvagg mentioned this pull request Mar 16, 2016
Closed
estliberitas force-pushed the master branch 2 times, most recently from 7da4fd4 to c7066fb Compare April 26, 2016 05:23
fhemberger mentioned this pull request May 24, 2016
Closed

eljefedelrodeodeljefe commented May 24, 2016

Copy link
Copy Markdown
Contributor

Looks reasonable to me. Build worked like a charm. thanks @fhemberger for pointing at this one (have missed it)

MylesBorins mentioned this pull request May 27, 2016
Closed
rvagg and others added 17 commits June 8, 2016 16:32
Revert "test: change duration_ms to duration"
553dec2 
This reverts commit d413378.
Refactor Mac .pkg generation
12bccda 
This refactors the Mac .pkg generation to use Packages. This moves io.js
away from using the deprecated Package Maker, and allows for easy
localization of the installer.
Add symlink step to Mac installer
0ed035d
Create empty files on pre-pkg, set up uninstall
86c5357
Wrap osx-pkg README at 80 characters
5559798
Add uninstaller
69cfb04
Remove node_modules if folder is empty
453c0ad
Replace iojs -> nodejs, remove symlink
1db12f4
build: Update OS X installer logo
8027c2e
Rename package: nodejs-* -> node-*
a828c74
Add npm preinstall script
d876918 
See nodejs#3986
Move .pkg to top-level
6c673b6
build: separate node, npm, headers install options
6f17f1e
build: fix osx uninstaller script
fe8c6c7
build: fix npm in pkg config
bee2989
Use new separate installer directories for pkg
ed5d909 
Needs splitting up, headers install bit needs to go with the install.py
changes
remove uninstaller
1ac41d9

jasnell commented Feb 28, 2017

Copy link
Copy Markdown
Member

ping @rvagg

jasnell added the stalled Issues and PRs that are stalled. label Feb 28, 2017
lance mentioned this pull request Aug 24, 2017
Closed

jasnell commented Aug 24, 2017

Copy link
Copy Markdown
Member

Closing due to lack of forward progress on this.

jasnell closed this Aug 24, 2017
jpwesselink mentioned this pull request Sep 4, 2017
Closed
4 tasks
jpwesselink added a commit to jpwesselink/node that referenced this pull request Sep 15, 2017
tools, build: macOS installer
bf9157f 
Creates macOS pkg installer by using `pkgbuild` and `productbuild`.
Removes previous npm installation before installing npm.
Packages carry correct version attributes.
Support for intl installer features, defaults to `en`.
Fancy formatted license.
Renamed `osx` references to `macOS`.
Optional installation of npm

PR-URL: nodejs#15179
Fixes: nodejs#15012
Refs: nodejs#5656
Refs: nodejs#2571
Refs: nodejs#7097
BridgeAR pushed a commit that referenced this pull request Sep 28, 2017
tools, build: refactor macOS installer
03954f7 
Creates macOS pkg installer by using `pkgbuild` and `productbuild`.
Removes previous npm installation before installing npm.
Packages carry correct version attributes.
Support for intl installer features, defaults to `en`.
Fancy formatted license.
Renamed `osx` references to `macOS`.
Optional installation of npm.

PR-URL: #15179
Fixes: #15012
Refs: #5656
Refs: #2571
Refs: #7097
Reviewed-By: Lance Ball <lball@redhat.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
MylesBorins pushed a commit that referenced this pull request Sep 29, 2017
tools, build: refactor macOS installer
803d9c1 
Creates macOS pkg installer by using `pkgbuild` and `productbuild`.
Removes previous npm installation before installing npm.
Packages carry correct version attributes.
Support for intl installer features, defaults to `en`.
Fancy formatted license.
Renamed `osx` references to `macOS`.
Optional installation of npm.

PR-URL: #15179
Fixes: #15012
Refs: #5656
Refs: #2571
Refs: #7097
Reviewed-By: Lance Ball <lball@redhat.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
addaleax pushed a commit to addaleax/ayo that referenced this pull request Sep 30, 2017
tools, build: refactor macOS installer
29a3b3a 
Creates macOS pkg installer by using `pkgbuild` and `productbuild`.
Removes previous npm installation before installing npm.
Packages carry correct version attributes.
Support for intl installer features, defaults to `en`.
Fancy formatted license.
Renamed `osx` references to `macOS`.
Optional installation of npm.

PR-URL: nodejs/node#15179
Fixes: nodejs/node#15012
Refs: nodejs/node#5656
Refs: nodejs/node#2571
Refs: nodejs/node#7097
Reviewed-By: Lance Ball <lball@redhat.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
MylesBorins pushed a commit that referenced this pull request Oct 3, 2017
tools, build: refactor macOS installer
7e8c0fd 
Creates macOS pkg installer by using `pkgbuild` and `productbuild`.
Removes previous npm installation before installing npm.
Packages carry correct version attributes.
Support for intl installer features, defaults to `en`.
Fancy formatted license.
Renamed `osx` references to `macOS`.
Optional installation of npm.

PR-URL: #15179
Fixes: #15012
Refs: #5656
Refs: #2571
Refs: #7097
Reviewed-By: Lance Ball <lball@redhat.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
MylesBorins pushed a commit that referenced this pull request Oct 3, 2017
tools, build: refactor macOS installer
eebb2d7 
Creates macOS pkg installer by using `pkgbuild` and `productbuild`.
Removes previous npm installation before installing npm.
Packages carry correct version attributes.
Support for intl installer features, defaults to `en`.
Fancy formatted license.
Renamed `osx` references to `macOS`.
Optional installation of npm.

PR-URL: #15179
Fixes: #15012
Refs: #5656
Refs: #2571
Refs: #7097
Reviewed-By: Lance Ball <lball@redhat.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jan 16, 2018
tools, build: refactor macOS installer
2400fc2 
Creates macOS pkg installer by using `pkgbuild` and `productbuild`.
Removes previous npm installation before installing npm.
Packages carry correct version attributes.
Support for intl installer features, defaults to `en`.
Fancy formatted license.
Renamed `osx` references to `macOS`.
Optional installation of npm.

PR-URL: #15179
Fixes: #15012
Refs: #5656
Refs: #2571
Refs: #7097
Reviewed-By: Lance Ball <lball@redhat.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jan 17, 2018
tools, build: refactor macOS installer
23ad5cb 
Creates macOS pkg installer by using `pkgbuild` and `productbuild`.
Removes previous npm installation before installing npm.
Packages carry correct version attributes.
Support for intl installer features, defaults to `en`.
Fancy formatted license.
Renamed `osx` references to `macOS`.
Optional installation of npm.

PR-URL: #15179
Fixes: #15012
Refs: #5656
Refs: #2571
Refs: #7097
Reviewed-By: Lance Ball <lball@redhat.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
MylesBorins pushed a commit that referenced this pull request Feb 11, 2018
tools, build: refactor macOS installer
7bab6bc 
Creates macOS pkg installer by using `pkgbuild` and `productbuild`.
Removes previous npm installation before installing npm.
Packages carry correct version attributes.
Support for intl installer features, defaults to `en`.
Fancy formatted license.
Renamed `osx` references to `macOS`.
Optional installation of npm.

PR-URL: #15179
Fixes: #15012
Refs: #5656
Refs: #2571
Refs: #7097
Reviewed-By: Lance Ball <lball@redhat.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
MylesBorins pushed a commit that referenced this pull request Feb 12, 2018
tools, build: refactor macOS installer
3a12b46 
Creates macOS pkg installer by using `pkgbuild` and `productbuild`.
Removes previous npm installation before installing npm.
Packages carry correct version attributes.
Support for intl installer features, defaults to `en`.
Fancy formatted license.
Renamed `osx` references to `macOS`.
Optional installation of npm.

PR-URL: #15179
Fixes: #15012
Refs: #5656
Refs: #2571
Refs: #7097
Reviewed-By: Lance Ball <lball@redhat.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
MylesBorins pushed a commit that referenced this pull request Feb 13, 2018
tools, build: refactor macOS installer
18803bc 
Creates macOS pkg installer by using `pkgbuild` and `productbuild`.
Removes previous npm installation before installing npm.
Packages carry correct version attributes.
Support for intl installer features, defaults to `en`.
Fancy formatted license.
Renamed `osx` references to `macOS`.
Optional installation of npm.

PR-URL: #15179
Fixes: #15012
Refs: #5656
Refs: #2571
Refs: #7097
Reviewed-By: Lance Ball <lball@redhat.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

install Issues and PRs related to the installers. macos Issues and PRs related to the macOS platform / OSX. stalled Issues and PRs that are stalled.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

Back | FazBrowse Home | New Git URL