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

Added GXX_ROOT to variables in module file as this impacts people who… #747

Merged
merged 4 commits into from Dec 12, 2015

Conversation

ocaisa
Copy link
Member

@ocaisa ocaisa commented Nov 24, 2015

… use locales and intel compilers

@hpcugentbot
Copy link
Contributor

Automatic reply from Jenkins: Can I test this?

# In case people are using unusual locales need to set GXXROOT, see
# https://software.intel.com/en-us/articles/
# intel-fortran-compiler-for-linux-ifort-error-could-not-find-directory-in-which-g-resides
get_gxxroot = "g++ --print-search-dirs | head -n1 | awk '{print $2}'"
Copy link
Member Author

Choose a reason for hiding this comment

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

Someone with better fu could probably improve this

@boegel
Copy link
Member

boegel commented Dec 12, 2015

Jenkins: ok to test

@hpcugentbot
Copy link
Contributor

Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1478/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@boegel
Copy link
Member

boegel commented Dec 12, 2015

cleaned up (and fixed) in ocaisa#7

parse output of 'g++ --print-search-dirs' in Python
@hpcugentbot
Copy link
Contributor

Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1480/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@boegel
Copy link
Member

boegel commented Dec 12, 2015

Going in, thanks @ocaisa!

@boegel boegel added this to the v2.5.0 milestone Dec 12, 2015
boegel added a commit that referenced this pull request Dec 12, 2015
Added GXX_ROOT to variables in module file as this impacts people who…
@boegel boegel merged commit b4a0cf2 into easybuilders:develop Dec 12, 2015
boegel added a commit to boegel/easybuild-easyblocks that referenced this pull request Dec 14, 2015
wpoely86 added a commit to wpoely86/easybuild-easyblocks that referenced this pull request Dec 14, 2015
* develop: (37 commits)
  remove unused import
  don't set $GXX_ROOT, it breaks icpc finding C++ header files (undoes easybuilders#747 and easybuilders#771)
  Sytle corrected
  remove workaround for $CPATH including icc's 'include' subdir
  update $CPATH to include path to GCC C++ headers (complex.h)
  call configure/build scripts with 'python' to dance around /usr/bin/python hardcoding
  fix typo
  fix remarks
  add support for specifying altroot/altversion in Bundle easyblock
  enable -verbose
  update MRtrix easyblock for v0.3.x
  add easyblock for MRtrix
  move defining $GXX_ROOT to icc easyblock, only needed for icc/ifort
  improved implementation of determining value for $GXX_ROOT
  parse output of 'g++ --print-search-dirs' in Python
  Fix minor style issues, add copyright
  remove unused import
  if -> elif
  Add silent_cfg_extras variable
  Add i686 platform checking for IPP
  ...
@ocaisa ocaisa deleted the intel_gxxroot branch December 15, 2015 10:15
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