Skip to content

fix: improve platform-dependent command execution in grunt tasks#2019

Open
bartvanandel wants to merge 3 commits intogchq:masterfrom
bartvanandel:fix/2010_npm_install_fails_on_macos
Open

fix: improve platform-dependent command execution in grunt tasks#2019
bartvanandel wants to merge 3 commits intogchq:masterfrom
bartvanandel:fix/2010_npm_install_fails_on_macos

Conversation

@bartvanandel
Copy link
Contributor

Fixes #2010

@bartvanandel bartvanandel force-pushed the fix/2010_npm_install_fails_on_macos branch from 7e564a9 to d1a2b8a Compare April 8, 2025 15:26
@bartvanandel bartvanandel marked this pull request as draft April 8, 2025 16:12
@bartvanandel bartvanandel force-pushed the fix/2010_npm_install_fails_on_macos branch 2 times, most recently from b7e7f33 to eef1d5d Compare April 8, 2025 16:34
@bartvanandel bartvanandel marked this pull request as ready for review April 8, 2025 16:35
@bartvanandel
Copy link
Contributor Author

Note to maintainers: I noticed that glob isn't installed as a direct dev dependency, but is being imported in Gruntfile.js. You may want to address that.

Use lazy, memoized getter to prevent unnecessary execution of `sed`
every time the Gruntfile.js file is loaded.
@bartvanandel bartvanandel force-pushed the fix/2010_npm_install_fails_on_macos branch from eef1d5d to 5a9ede0 Compare February 5, 2026 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug report: npm install fails at exec:fixCryptoApiImports on MacOS when sed is GNU sed

2 participants