Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Refactor mac installer (built on #2571) #5656

Closed
wants to merge 22 commits into from
Closed

Refactor mac installer (built on #2571) #5656

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
553dec2
Revert "test: change duration_ms to duration"
rvagg Jun 8, 2016
12bccda
Refactor Mac .pkg generation
remixz Feb 10, 2015
0ed035d
Add symlink step to Mac installer
remixz Feb 12, 2015
86c5357
Create empty files on pre-pkg, set up uninstall
remixz Feb 12, 2015
5559798
Wrap osx-pkg README at 80 characters
remixz Feb 13, 2015
69cfb04
Add uninstaller
remixz Feb 14, 2015
453c0ad
Remove node_modules if folder is empty
remixz Mar 5, 2015
1db12f4
Replace iojs -> nodejs, remove symlink
fhemberger Aug 27, 2015
8027c2e
build: Update OS X installer logo
fhemberger Sep 9, 2015
a828c74
Rename package: nodejs-* -> node-*
fhemberger Oct 23, 2015
d876918
Add npm preinstall script
fhemberger Dec 26, 2015
6c673b6
Move .pkg to top-level
rvagg Mar 11, 2016
6f17f1e
build: separate node, npm, headers install options
rvagg Mar 11, 2016
fe8c6c7
build: fix osx uninstaller script
rvagg Mar 11, 2016
bee2989
build: fix npm in pkg config
rvagg Mar 11, 2016
ed5d909
Use new separate installer directories for pkg
rvagg Mar 11, 2016
1ac41d9
remove uninstaller
rvagg Mar 11, 2016
f06d845
Fix installer options, tweak intro and summary
rvagg Mar 11, 2016
00baf29
build: update Node.js logo on OSX installer
rvagg Feb 24, 2016
5671878
change background
rvagg Mar 11, 2016
e943c43
undo temporary debug make stuff
rvagg Mar 11, 2016
5313db4
PACKAGESBUILD?=, don't run pre-pkg before building
rvagg Mar 16, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use new separate installer directories for pkg 
Needs splitting up, headers install bit needs to go with the install.py
changes
  • Loading branch information
@rvagg
rvagg committed Jun 8, 2016
commit ed5d9093c325228910aac170488936af0e701ed9
9 changes: 6 additions & 3 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -487,8 +487,11 @@ $(PKG): release-only pre-pkg
--tag=$(TAG) \
--release-urlbase=$(RELEASE_URLBASE) \
$(CONFIG_FLAGS) $(BUILD_RELEASE_FLAGS)
$(MAKE) install V=$(V) DESTDIR=$(PKGDIR)
SIGN="$(CODESIGN_CERT)" PKGDIR="$(PKGDIR)" bash tools/osx-codesign.sh
$(MAKE) all V=$(V)
NODE_INSTALL_NODE_ONLY=1 $(PYTHON) tools/install.py install '$(PKGDIR)/node' '$(PREFIX)'
NODE_INSTALL_HEADERS_ONLY=1 $(PYTHON) tools/install.py install '$(PKGDIR)/node' '$(PREFIX)'
NODE_INSTALL_NPM_ONLY=1 $(PYTHON) tools/install.py install '$(PKGDIR)/npm' '$(PREFIX)'
SIGN="$(CODESIGN_CERT)" PKGDIR="$(PKGDIR)/node" bash tools/osx-codesign.sh
$(PACKAGESBUILD) tools/osx-pkg/osx-pkg-out.pkgproj
SIGN="$(PRODUCTSIGN_CERT)" PKG="$(PKG)" bash tools/osx-productsign.sh

Expand Down Expand Up @@ -550,7 +553,7 @@ $(TARBALL)-headers: release-only
--tag=$(TAG) \
--release-urlbase=$(RELEASE_URLBASE) \
$(CONFIG_FLAGS) $(BUILD_RELEASE_FLAGS)
HEADERS_ONLY=1 $(PYTHON) tools/install.py install '$(TARNAME)' '/'
NODE_INSTALL_HEADERS_ONLY=1 $(PYTHON) tools/install.py install '$(TARNAME)' '/'
find $(TARNAME)/ -type l | xargs rm -f
tar -cf $(TARNAME)-headers.tar $(TARNAME)
rm -rf $(TARNAME)
Expand Down