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.

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

node 15.0.1 #63170

Closed
wants to merge 2 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
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
File renamed without changes.
4 changes: 3 additions & 1 deletion Formula/bcoin.rb
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ class Bcoin < Formula
url "https://github.com/bcoin-org/bcoin/archive/v2.1.2.tar.gz"
sha256 "b4c63598ee1efc17e4622ef88c1dff972692da1157e8daf7da5ea8abc3d234df"
license "MIT"
revision 1
revision 2
head "https://github.com/bcoin-org/bcoin.git"

bottle do
Expand All @@ -19,6 +19,8 @@ class Bcoin < Formula
depends_on "node"

def install
# These dirs must exists before npm install.
mkdir_p libexec/"lib"
system "#{Formula["node"].libexec}/bin/npm", "install", *Language::Node.std_npm_install_args(libexec)
(bin/"bcoin").write_env_script libexec/"bin/bcoin", PATH: "#{Formula["node"].opt_bin}:$PATH"
end
Expand Down
13 changes: 6 additions & 7 deletions Formula/node.rb
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
class Node < Formula
desc "Platform built on V8 to build network applications"
homepage "https://nodejs.org/"
url "https://nodejs.org/dist/v14.14.0/node-v14.14.0.tar.gz"
sha256 "afb0b401d62d9fcfc68258f50d0bf042998d349ce9c1d7a2d45dd87870b3aab7"
url "https://nodejs.org/dist/v15.0.1/node-v15.0.1.tar.gz"
sha256 "b9a00a4847863914ffe7751c2d81b67cb96a8f958cbc692f988c8c78db14ebec"
license "MIT"
head "https://github.com/nodejs/node.git"

Expand All @@ -25,8 +25,8 @@ class Node < Formula
# We track major/minor from upstream Node releases.
# We will accept *important* npm patch releases when necessary.
resource "npm" do
url "https://registry.npmjs.org/npm/-/npm-6.14.8.tgz"
sha256 "fe8e873cb606c06f67f666b4725eb9122c8927f677c8c0baf1477f0ff81f5a2c"
url "https://registry.npmjs.org/npm/-/npm-7.0.3.tgz"
sha256 "d83833ae163545abf6a54dbb019770ef9000d31f1c2dd0cd934dedf6b41382a6"
end

def install
Expand All @@ -36,9 +36,6 @@ def install
# Never install the bundled "npm", always prefer our
# installation from tarball for better packaging control.
args = %W[--prefix=#{prefix} --without-npm --with-intl=system-icu]
# Remove `--openssl-no-asm` workaround when upstream releases a fix
# See also: https://github.com/nodejs/node/issues/34043
args << "--openssl-no-asm" if Hardware::CPU.arm?
args << "--tag=head" if build.head?

system "./configure", *args
Expand All @@ -49,6 +46,8 @@ def install

bootstrap = buildpath/"npm_bootstrap"
bootstrap.install resource("npm")
# These dirs must exists before npm install.
mkdir_p libexec/"lib"
SMillerDev marked this conversation as resolved.
Show resolved Hide resolved
system "node", bootstrap/"bin/npm-cli.js", "install", "-ddd", "--global",
"--prefix=#{libexec}", resource("npm").cached_download

Expand Down