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

Add libelf + libdwarf #1283

Merged
merged 11 commits into from
Jan 16, 2015
Merged

Add libelf + libdwarf #1283

merged 11 commits into from
Jan 16, 2015

Conversation

wpoely86
Copy link
Member

@wpoely86 wpoely86 commented Jan 7, 2015

Should clean this up when I find the time...

@hpcugentbot
Copy link
Contributor

Test PASSed.

@wpoely86
Copy link
Member Author

wpoely86 commented Jan 7, 2015

Test report by @wpoely86
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
Linux debian 7.7, Intel(R) Xeon(R) CPU 5130 @ 2.00GHz, Python 2.7.3
See https://gist.github.com/ec25d6a0a4a55b49c984 for a full test report.

@hpcugentbot
Copy link
Contributor

Test PASSed.

@hpcugentbot
Copy link
Contributor

Test PASSed.

@wpoely86
Copy link
Member Author

wpoely86 commented Jan 8, 2015

Ready, I think.

It uses the dummy toolchain because I needed this for PerfExpert (#839).

@hpcugentbot
Copy link
Contributor

Test PASSed.

1 similar comment
@hpcugentbot
Copy link
Contributor

Test PASSed.

@boegel
Copy link
Member

boegel commented Jan 8, 2015

@wpoely86: not a big fan of merging in easyconfigs that use the dummy toolchain, they're too brittle... These may work for you, but may not work for me on another OS (different GCC).

That's also one of the reasons why #839 hasn't made it in yet.

The real solution here would be to i) build with a non-dummy toolchain (e.g. GCC), ii) assure the resulting build is toolchain-neutral (see easybuilders/easybuild-framework#570) and avoid using the toolchain label in the resulting module name.

So, please revamp these to use a GCC-only toolchain?

* develop:
  add easyconfigs for IMB with different toolchains
  remove faulty 'easyblock = ConfigureMake' line from HPL easyconfig
  add easyconfig file for EasyBuild v1.16.1
  fix deprecated fallback to ConfigureMake easyblock in HPL easyconfig
  add easyconfig file for EasyBuild v1.16.0
  add easyconfig for HPL 2.1 with intel/2015a
  add easyconfig file for intel/2015a
@wpoely86
Copy link
Member Author

wpoely86 commented Jan 9, 2015

Test report by @wpoely86
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
Linux debian 7.7, Intel(R) Xeon(R) CPU 5130 @ 2.00GHz, Python 2.7.3
See https://gist.github.com/fa4d354b850f13d8b836 for a full test report.

@hpcugentbot
Copy link
Contributor

Test FAILed.

@hpcugentbot
Copy link
Contributor

Test FAILed.

@hpcugentbot
Copy link
Contributor

Test PASSed.

@wpoely86
Copy link
Member Author

@boegel done

@wpoely86
Copy link
Member Author

Test report by @wpoely86
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
Linux debian 7.7, Intel(R) Xeon(R) CPU 5130 @ 2.00GHz, Python 2.7.3
See https://gist.github.com/4a523d5d60f7d952d2d1 for a full test report.

configopts = "--enable-shared "

# This is dirty but libdwarf cannot find it's own library in the build process...
prebuildopts = ' LD_LIBRARY_PATH="../libdwarf:$LD_LIBRARY_PATH" '
Copy link
Member

Choose a reason for hiding this comment

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

This is... weird.

Copy link
Member Author

Choose a reason for hiding this comment

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

The build process is also a bit non standard 😉
It's just a weird library. It correctly finds it's own library for the linking, but it also tries to run a binary to generate other stuff to compile, and there it fails.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, well, at least you're making clear why it's needed, and I've been worse. ;-)

@boegel
Copy link
Member

boegel commented Jan 16, 2015

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
Linux SL 6.6, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/162d6f3c3defe62dd768 for a full test report.

@hpcugentbot
Copy link
Contributor

Test PASSed.

@boegel
Copy link
Member

boegel commented Jan 16, 2015

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
Linux SL 6.6, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/70b20dcc0ddc2ba30c4e for a full test report.

@boegel
Copy link
Member

boegel commented Jan 16, 2015

Looking great now, thanks @wpoely86, going in.

boegel added a commit that referenced this pull request Jan 16, 2015
@boegel boegel merged commit f6d9ec5 into easybuilders:develop Jan 16, 2015
@wpoely86 wpoely86 deleted the libs branch January 16, 2015 10:29
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.

3 participants