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
3rd-party: rename libevent release to stable_ompi #12501
Conversation
I hit another issue with this libevent release though. It does not work with newer rpmbuild due to ambiguous python shebang because of this line. I had to manually change it to |
You should note this in the commit message. |
rpmbuild now does not accept '-' in release name. This patch rebuilds libevent 2.1.12-stable with: - autoconf 2.27 - automake 1.16.5 - libtool 2.4.7 Note that this libevent release contains ambiguous python shebang that will cause rpm build failures on modern systems. Read more: https://fedoraproject.org/wiki/Changes/Make_ambiguous_python_shebangs_error Signed-off-by: Wenduo Wang <wenduwan@amazon.com>
ed1a8f3
to
43291ae
Compare
Reworded commit |
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.
With your new commit wording, I'm now confused.
Your new commit says:
Note that this libevent release contains ambiguous python shebang that will
cause rpm build failures on modern systems. Read more:
https://fedoraproject.org/wiki/Changes/Make_ambiguous_python_shebangs_error
which implies that the new tarball does not fix the issue. But your previous comment says:
I had to manually change it to
#!/usr/bin/env python3
Which I interpreted to mean that you had manually changed it in the new tarball.
Which is it?
There are 2 issues exposed by newer rpmbuild in the current libevent release tarball.
Given the slack discussion I'm inclined to cancel this PR and leave it broken - it actually does what we want, i.e. fending off people trying to build rpm with the internal libevent. Personally I want to fix it and allow people to build RPMs in whichever way possible, since we don't provide any warranty on the software anyways. |
Why can't we fix the confusing shebang in the internal libevent tarball? You already modified the tarball by running new autotools on it. What's wrong with making another small, patch-level change to that tarball? |
IMO modifying the distributed source code is different - to begin with, I'm actually not sure which python version should be used. |
It's the moral equivalent of distributing a patch -- which is what just about every binary packager does. Indeed, in OMPI, we patch a bunch of stuff before using it. You should use |
rpmbuild now does not accept '-' in release name.
This patch rebuilds libevent 2.1.12-stable with: