update Doxygen easyblock for 1.10.x (CMake) (REVIEW)#734
update Doxygen easyblock for 1.10.x (CMake) (REVIEW)#734boegel merged 6 commits intoeasybuilders:developfrom
Conversation
There was a problem hiding this comment.
self is not defined here
do the version check in configure_step, always import ConfigureMake
There was a problem hiding this comment.
boegel: Ok, but how I will switch to CMakeMake?
There was a problem hiding this comment.
I mean where and how? Apparently 3rd works with the old (but then the EasyBlock is identical to the old), but not for the new.
|
Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1341/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. |
|
Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1342/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 seems to work. |
|
Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1343/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. |
There was a problem hiding this comment.
and it should be updated since we now have two cases...
what was so special about the --prefix below btw? I must be blind...
There was a problem hiding this comment.
@boegel was without =
the normal is --prefix=foo/bar and here is --prefix foo/bar
|
Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1446/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. |
|
Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1447/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. |
|
Going in, thanks @hajgato! |
update Doxygen easyblock for 1.10.x (CMake) (REVIEW)
No description provided.