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

tools, build: refactor macOS installer · nodejs/node@eebb2d7 · GitHub

/ node Public

Commit eebb2d7

Browse files
jpwesselink
authored and
MylesBorins
committed
tools, build: refactor macOS installer
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>
1 parent ca02576 commit eebb2d7

12 files changed

Lines changed: 119 additions & 122 deletions

File tree

‎.gitignore‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,11 +69,9 @@ ipch/
6969
/config_fips.gypi
7070
*-nodegyp*
7171
/gyp-mac-tool
72-
/dist-osx
7372
/npm.wxs
7473
/tools/msvs/npm.wixobj
7574
/tools/msvs/genfiles/
76-
/tools/osx-pkg.pmdoc/index.xml
7775
/test/addons/??_*/
7876
email.md
7977
deps/v8-*
@@ -101,6 +99,7 @@ deps/npm/node_modules/.bin/
10199

102100
# build/release artifacts
103101
/*.tar.*
102+
/*.pkg
104103
/SHASUMS*.txt*
105104

106105
# test artifacts

‎Makefile‎

Lines changed: 43 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -673,8 +673,7 @@ BINARYTAR=$(BINARYNAME).tar
673673
XZ=$(shell which xz > /dev/null 2>&1; echo $$?)
674674
XZ_COMPRESSION ?= 9e
675675
PKG=$(TARNAME).pkg
676-
PACKAGEMAKER ?= /Developer/Applications/Utilities/PackageMaker.app/Contents/MacOS/PackageMaker
677-
PKGDIR=out/dist-osx
676+
MACOSOUTDIR=out/macos
678677

679678
release-only:
680679
@if [ "$(DISTTYPE)" != "nightly" ] && [ "$(DISTTYPE)" != "next-nightly" ] && \
@@ -709,24 +708,54 @@ release-only:
709708
fi
710709

711710
$(PKG): release-only
712-
$(RM) -r $(PKGDIR)
713-
$(RM) -r out/deps out/Release
711+
$(RM) -r $(MACOSOUTDIR)
712+
mkdir -p $(MACOSOUTDIR)/installer/productbuild
713+
cat tools/macos-installer/productbuild/distribution.xml.tmpl \
714+
| sed -E "s/\\{nodeversion\\}/$(FULLVERSION)/g" \
715+
| sed -E "s/\\{npmversion\\}/$(NPMVERSION)/g" \
716+
>$(MACOSOUTDIR)/installer/productbuild/distribution.xml ; \
717+
718+
@for dirname in tools/macos-installer/productbuild/Resources/*/; do \
719+
lang=$$(basename $$dirname) ; \
720+
mkdir -p $(MACOSOUTDIR)/installer/productbuild/Resources/$$lang ; \
721+
printf "Found localization directory $$dirname\n" ; \
722+
cat $$dirname/welcome.html.tmpl \
723+
| sed -E "s/\\{nodeversion\\}/$(FULLVERSION)/g" \
724+
| sed -E "s/\\{npmversion\\}/$(NPMVERSION)/g" \
725+
>$(MACOSOUTDIR)/installer/productbuild/Resources/$$lang/welcome.html ; \
726+
cat $$dirname/conclusion.html.tmpl \
727+
| sed -E "s/\\{nodeversion\\}/$(FULLVERSION)/g" \
728+
| sed -E "s/\\{npmversion\\}/$(NPMVERSION)/g" \
729+
>$(MACOSOUTDIR)/installer/productbuild/Resources/$$lang/conclusion.html ; \
730+
done
714731
$(PYTHON) ./configure \
715732
--dest-cpu=x64 \
716733
--tag=$(TAG) \
717734
--release-urlbase=$(RELEASE_URLBASE) \
718735
$(CONFIG_FLAGS) $(BUILD_RELEASE_FLAGS)
719-
$(MAKE) install V=$(V) DESTDIR=$(PKGDIR)
720-
SIGN="$(CODESIGN_CERT)" PKGDIR="$(PKGDIR)/usr/local" bash \
736+
$(MAKE) install V=$(V) DESTDIR=$(MACOSOUTDIR)/dist/node
737+
SIGN="$(CODESIGN_CERT)" PKGDIR="$(MACOSOUTDIR)/dist/node/usr/local" bash \
721738
tools/osx-codesign.sh
722-
cat tools/osx-pkg.pmdoc/index.xml.tmpl \
723-
| sed -E "s/\\{nodeversion\\}/$(FULLVERSION)/g" \
724-
| sed -E "s/\\{npmversion\\}/$(NPMVERSION)/g" \
725-
> tools/osx-pkg.pmdoc/index.xml
726-
$(PACKAGEMAKER) \
727-
--id "org.nodejs.pkg" \
728-
--doc tools/osx-pkg.pmdoc \
729-
--out $(PKG)
739+
mkdir -p $(MACOSOUTDIR)/dist/npm/usr/local/lib/node_modules
740+
mkdir -p $(MACOSOUTDIR)/pkgs
741+
mv $(MACOSOUTDIR)/dist/node/usr/local/lib/node_modules/npm \
742+
$(MACOSOUTDIR)/dist/npm/usr/local/lib/node_modules
743+
unlink $(MACOSOUTDIR)/dist/node/usr/local/bin/npm
744+
unlink $(MACOSOUTDIR)/dist/node/usr/local/bin/npx
745+
$(NODE) tools/license2rtf.js < LICENSE > \
746+
$(MACOSOUTDIR)/installer/productbuild/Resources/license.rtf
747+
cp doc/osx_installer_logo.png $(MACOSOUTDIR)/installer/productbuild/Resources
748+
pkgbuild --version $(FULLVERSION) \
749+
--identifier org.nodejs.node.pkg \
750+
--root $(MACOSOUTDIR)/dist/node $(MACOSOUTDIR)/pkgs/node-$(FULLVERSION).pkg
751+
pkgbuild --version $(NPMVERSION) \
752+
--identifier org.nodejs.npm.pkg \
753+
--root $(MACOSOUTDIR)/dist/npm \
754+
--scripts ./tools/macos-installer/pkgbuild/npm/scripts \
755+
$(MACOSOUTDIR)/pkgs/npm-$(NPMVERSION).pkg
756+
productbuild --distribution $(MACOSOUTDIR)/installer/productbuild/distribution.xml \
757+
--resources $(MACOSOUTDIR)/installer/productbuild/Resources \
758+
--package-path $(MACOSOUTDIR)/pkgs ./$(PKG)
730759
SIGN="$(PRODUCTSIGN_CERT)" PKG="$(PKG)" bash tools/osx-productsign.sh
731760

732761
pkg: $(PKG)

‎tools/macos-installer/pkgbuild/npm/scripts/postinstall‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
#!/bin/sh
2+
3+
cd /usr/local/bin || exit 1
4+
ln -sf ../lib/node_modules/npm/bin/npm-cli.js npm
5+
ln -sf ../lib/node_modules/npm/bin/npx-cli.js npx

‎tools/macos-installer/pkgbuild/npm/scripts/preinstall‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
#!/bin/sh
2+
3+
[[ -d /usr/local/lib/node_modules/npm ]] \
4+
&& rm -rf /usr/local/lib/node_modules/npm
5+
exit 0

‎tools/macos-installer/productbuild/Resources/en.lproj/conclusion.html.tmpl‎

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
<html>
2+
<head>
3+
<style>
4+
body {
5+
font-family: Helvetica;
6+
font-size: 14px;
7+
}
8+
li:last-child {
9+
margin-bottom: 16px;
10+
}
11+
</style>
12+
</head>
13+
<body>
14+
<div>
15+
<p>This package has installed:</p>
16+
<ul>
17+
<li>Node.js {nodeversion} to <code>/usr/local/bin/node</code></li>
18+
<li>npm {npmversion} to <code>/usr/local/bin/npm</code></li>
19+
</ul>
20+
<p>Make sure that <code>/usr/local/bin</code> is in your <code>$PATH</code>.</p>
21+
</div>
22+
</body>
23+
</html>

‎tools/macos-installer/productbuild/Resources/en.lproj/welcome.html.tmpl‎

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
<html>
2+
<head>
3+
<style>
4+
body {
5+
font-family: Helvetica;
6+
font-size: 14px;
7+
}
8+
</style>
9+
</head>
10+
<body>
11+
<div>
12+
<p>This package will install:</p>
13+
<ul>
14+
<li>Node.js {nodeversion} to <code>/usr/local/bin/node</code></li>
15+
<li>npm {npmversion} to <code>/usr/local/bin/npm</code></li>
16+
</ul>
17+
</div>
18+
</body>
19+
</html>

‎tools/macos-installer/productbuild/distribution.xml.tmpl‎

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
<?xml version="1.0" encoding="utf-8"?>
2+
<installer-gui-script minSpecVersion="1">
3+
<title>Node.js</title>
4+
<welcome file="welcome.html"/>
5+
<conclusion file="conclusion.html"/>
6+
<background alignment="topleft" file="osx_installer_logo.png"/>
7+
<pkg-ref id="org.nodejs.node.pkg" auth="root"/>
8+
<pkg-ref id="org.nodejs.npm.pkg" auth="root"/>
9+
<options customize="allow" require-scripts="false"/>
10+
<license file="license.rtf"/>
11+
<choices-outline>
12+
<line choice="org.nodejs.node.pkg" />
13+
<line choice="org.nodejs.npm.pkg"/>
14+
</choices-outline>
15+
<choice id="org.nodejs.node.pkg" visible="true" title="Node.js {nodeversion}">
16+
<pkg-ref id="org.nodejs.node.pkg"/>
17+
</choice>
18+
<pkg-ref id="org.nodejs.node.pkg" version="{nodeversion}" onConclusion="none">node-{nodeversion}.pkg</pkg-ref>
19+
<choice id="org.nodejs.npm.pkg" visible="true" title="npm {npmversion}">
20+
<pkg-ref id="org.nodejs.npm.pkg"/>
21+
</choice>
22+
<pkg-ref id="org.nodejs.npm.pkg" version="{npmversion}" onConclusion="none">npm-{npmversion}.pkg</pkg-ref>
23+
</installer-gui-script>

‎tools/osx-pkg.pmdoc/01local-contents.xml‎

Lines changed: 0 additions & 1 deletion
This file was deleted.

‎tools/osx-pkg.pmdoc/01local.xml‎

Lines changed: 0 additions & 25 deletions
This file was deleted.

‎tools/osx-pkg.pmdoc/02npm-contents.xml‎

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)
Back | FazBrowse Home | New Git URL