-
Notifications
You must be signed in to change notification settings - Fork 720
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
{vis}[foss/2018a] gnuplot v5.2.2 #5955
{vis}[foss/2018a] gnuplot v5.2.2 #5955
Conversation
('libpng', '1.6.32'), | ||
('fontconfig', '2.12.6'), | ||
('libjpeg-turbo', '1.5.3'), | ||
('libpng', '1.6.34'), |
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.
@reedts Please don't bump dependency versions in existing easyconfig, you'll introduce conflicts in other places...
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'll take it out
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.
@reedts You want to add a new easyconfig for libgd
2.2.5, not change the existing one for
2.2.4` in place...
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.
Ah I found an error in my libgd-2.2.5 easyconfig. The version there was still 2.2.4. Seems like it messed things up...
@reedts This PR is getting quite big, to the point where it's going to be difficult to get it merged. A rule of thumb is to not include/tweak more than 10 easyconfigs per PR. Can you flesh out some of this in a separate PR? |
@boegel I agree, I'll restructure this and split it up into several PRs. |
@boegel Removed the Mesa and cairo packages. The other easyconfigs are already outsourced in other PRs, see updated PR description. Read this too late, sorry.. |
easyblock = 'ConfigureMake' | ||
|
||
name = 'libgd' | ||
version = '2.2.4' |
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.
@reedts Please don't remove existing easyconfigs...
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.
@boegel My fault. Looks like I have to re-add it to remove it?
easyblock = 'ConfigureMake' | ||
|
||
name = 'libdrm' | ||
version = '2.4.90' |
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.
@reedts We already have an existing easyconfig for libdrm
2.4.91
, which is more recent than this...
Also, this is not related to gnuplot
at all? So can just be removed imho...
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.
@boegel Right, it must have got in here somehow. I'll remove it.
@reedts All PRs this PR depends on are now merged, I've retriggered the tests in Travis to get a better view on current status. |
('cairo', '1.14.12'), | ||
('libjpeg-turbo', '1.5.3'), | ||
('libpng', '1.6.34'), | ||
('libgd', '2.2.5'), |
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.
@reedts Easyconfig file for this libgd
dep is still missing?
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.
@boegel Should be included now.
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.
looks good now indeed 👍
('libpng', '1.6.32'), | ||
('fontconfig', '2.12.6'), | ||
('libjpeg-turbo', '1.5.3'), | ||
('libpng', '1.6.34'), |
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.
@reedts You want to add a new easyconfig for libgd
2.2.5, not change the existing one for
2.2.4` in place...
@@ -1,7 +1,7 @@ | |||
easyblock = 'ConfigureMake' | |||
|
|||
name = 'libgd' | |||
version = '2.2.4' | |||
version = '2.2.5' |
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.
@reedts You're now renaming the libgd
2.2.4 easyconfig to 2.2.5
, which means removing the 2.2.4
easyconfig...
Also, the checksum for libgd
2.2.5 needs to be fixed.
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.
@boegel I fixed the checksum but still struggling to get that version 2.2.4 out. Will removing it and readding it work (before I try and it still won't work)?
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.
@reedts Just re-adding it should do it (you can't remove it anymore, it's already gone since it was renamed to 2.2.5
).
Test report by @boegel |
Test report by @boegel |
@reedts Tests complain that both I'm fixing that in #5999, I will re-trigger the tests here once that PR is merged. |
@boegel All right! Good that check exists now in the first place! |
@reedts Seems like you'll need to sync your branch with current |
sync with develop
@boegel Merged. |
Going in, thanks @reedts! |
(created using
eb --new-pr
)Edit: depends on
#5988,#5987,#5979,#5980