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

{bio}[GCC/10.2.0] OMA v2.5.0 #13447

Closed

Conversation

sdx23
Copy link
Contributor

@sdx23 sdx23 commented Jul 16, 2021

(created using eb --new-pr)
New version that is not broken on Debian systems. For some reason it needs Python as a build dependency. I'm not sure I've set the toolchain correctly to achieve this, could you please comment on that?

@verdurin
Copy link
Member

Test report by @verdurin
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
nuc.lan - Linux Fedora 33, x86_64, Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz, Python 3.9.5
See https://gist.github.com/d792637e9a3453bd0570a7e6fce2012a for a full test report.

@verdurin
Copy link
Member

@sdx23 welcome to EasyBuild and thanks for the contribution.

That is indeed a bit strange - a binary that nevertheless requires Python. I suppose that given it's only a build dep, there's no need to include Python in the versionsuffix. Normally a binary would use the SYSTEM toolchain. GCCcore might make more sense than GCC and make it more compatible with other toolchains.

install_cmd = 'sh ./install.sh %(installdir)s'

builddependencies = [
('Python', '3.8.6'),
Copy link
Member

Choose a reason for hiding this comment

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

@sdx23 with the toolchain change, you will need to add binutils as a dependency.

Suggested change
('Python', '3.8.6'),
('Python', '3.8.6'),
('binutils', '2.35'),

@boegel boegel added the update label Jan 14, 2024
@boegel boegel added this to the 4.x milestone Jan 14, 2024
@boegel
Copy link
Member

boegel commented Jan 14, 2024

closing this since GCC/10.2.0 is deprecated, see https://docs.easybuild.io/policies/toolchains

@boegel boegel closed this Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants