Skip to content
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

{devel,vis}[GCCcore/5.4.0] GLib v2.49.5, PCRE v8.39 (WIP) #4695

Closed

Conversation

vanzod
Copy link
Member

@vanzod vanzod commented Jun 8, 2017

(created using eb --new-pr)

@vanzod
Copy link
Member Author

vanzod commented Jun 8, 2017

Test report by @vanzod
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
cermis - Linux debian 8.8, Intel(R) Xeon(R) CPU E5620 @ 2.40GHz, Python 2.7.9
See https://gist.github.com/a95cad3d1a800857f30dc96f6795c675 for a full test report.

]

#GLib compiles correctly even without this dependency
#builddependencies = [('Python', '2.7.12')]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vanzod It does, because EasyBuild requires a Python 2.x to be available as well, but we shouldn't rely on the system Python here?

In the hopefully not so distant future, you'll be able to run EasyBuild on top of a system Python 3, which would break this build?

Copy link
Member Author

@vanzod vanzod Jun 9, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point. However that would require Python at the GCCcore level. I would rather build it against a bare python at the GCC or iccifort level.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, yes, that compilates things a bit...

Although, a bare Python at GCCcore should be fine, no?

Copy link
Member Author

@vanzod vanzod Jun 9, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess so but on our cluster at the moment we provide it at the GCC and iccifort level.
I can easily add Python-2.7.12-GCCcore-5.4.0 to this PR if you wish to merge it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, yeah, that will work too, I'd rather do that than commenting out the build dep, that'll backfire sometime in the future imho...

@boegel boegel added this to the 3.3.0 milestone Jun 9, 2017
@vanzod vanzod changed the title {devel,vis}[GCCcore/5.4.0] GLib v2.49.5, PCRE v8.39 {devel,vis}[GCCcore/5.4.0] GLib v2.49.5, PCRE v8.39 (WIP) Jun 20, 2017
@vanzod
Copy link
Member Author

vanzod commented Jun 20, 2017

Before pushing a Python-2.7.12-GCCcore easyconfig to solve the missing dependency if not using the system Python, a SciPy with a dedicated interpreter needs to be available in order to overcome the performance issue arising from the incorrect use of math functions.

@boegel boegel modified the milestone: 3.3.0 Jun 25, 2017
@migueldiascosta
Copy link
Member

@vanzod, this PR is being closed for the following reason(s): no activity for > 1 year.
Please don't hesitate to reopen this PR or add a comment if you feel this contribution is still relevant.
For more information on our policy w.r.t. closing PRs, see https://easybuild.readthedocs.io/en/latest/Contributing.html#why-a-pull-request-may-be-closed-by-a-maintainer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants