-
Notifications
You must be signed in to change notification settings - Fork 148
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
drop node dependencies which are unused for building #7022
drop node dependencies which are unused for building #7022
Conversation
[test packaging] |
29e7189
to
0499778
Compare
Depends on theforeman/foreman#8731 |
0499778
to
c084b60
Compare
all my changes to core have been merged, so this is now just a simple "call a script and carry on" PR 😹 |
one thing this also should help with, is plugin building. |
@@ -16,6 +16,7 @@ build: | |||
/bin/rm -f bundler.d/test.rb bundler.d/development.rb | |||
/bin/sed -i '2i gem "puma-status"' bundler.d/service.rb | |||
BUNDLE_BUILD__SASSC=--disable-march-tune-native BUNDLE_RETRY=5 BUNDLE_JOBS=4 /usr/bin/bundle package | |||
/usr/bin/python script/filter-package-json.py |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not executable with a shebang?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not (and the shebang is using py3, which we don't install by default -- the script works with py2 and 3 identically)
No description provided.