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
Bump nodejs-theforeman-vendor to 3.3.2 #4417
Conversation
The following PRs need to be merged "together": #4417 #4416 #4413 However, they are blocked on the discussion in theforeman/foreman#7178 |
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.
Inline
%if 0%{?scl:1} | ||
BuildRequires: %{?scl_prefix_nodejs}npm | ||
%else | ||
%if 0%{?!scl:1} |
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.
Meh, something is wrong with my nom2rpm..
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.
I kind of assumed that was intended since non-bundled packages don't really call NPM. Or is it needed for macros?
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.
Well, I don't know. It wasn't intended, so I was going with whatever was there before
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.
I still backed out that change, will revisit later.
No description provided.