-
Notifications
You must be signed in to change notification settings - Fork 706
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
switch back to Bundle easyblock for PyQt5 5.15.1 + add back source URLs for components #13371
switch back to Bundle easyblock for PyQt5 5.15.1 + add back source URLs for components #13371
Conversation
Test report by @Micket Disk space issues, retrying |
Test report by @Flamefire |
Test report by @Micket |
Any idea on your build failure @Flamefire ? |
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.
lgtm
@boegelbot please test @ generoso |
@Micket: Request for testing this PR well received on generoso PR test command '
Test results coming soon (I hope)... - notification for comment with ID 876013782 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@Micket Not really, but doesn't look good. Retrying interactively... |
Test report by @Flamefire |
Ok it works. Might have been interference from my other tests |
Test report by @Flamefire Edit: Ignore this. Missing patch for Qt5: https://www.mail-archive.com/easybuild@lists.ugent.be/msg06069.html |
Test report by @Flamefire |
Going in, thanks @Flamefire! |
(created using
eb --new-pr
)Fix for incomplete revert of #13253 done in #13336 which didn't revert the Bundle->PythonBundle change and missed the need to specify source_urls (as it is a bundle with components not pythonbundle with extensions)