Skip to content

Conversation

@boegel
Copy link

@boegel boegel commented Nov 15, 2018

@mboisson This is a big update because i) your fix-bug-tweak branch was significantly behidnd on develop, and ii) I had to fix a couple of merge conflicts with recently merged PRs (incl. easybuilders#2660)

If you want to review this first, the key changes compared to easybuilders#2635 are in commits 4a7b9d1 and b299861 (all the rest is stuff that got merged into develop already).

boegel and others added 30 commits September 25, 2018 10:15
sync with master + bump to 3.7.1dev
…mplating

fix templating of values in list_software function
…1804

stop relying on 'easy_install' in bootstrap script, use 'python -m easy_install' instead
lower required Lmod version to 6.5.1
generate .modulerc.lua when Lua syntax and Lmod >= 7.8 is used
…abels

determine file_info for all easyconfigs before any actual copying
fix composing of lib64 fallback paths in sanity check
… module name from wrapper module + full module name of wrapper module itself
… as modules tool, enhance tests for module_wrapper_exists method
boegel and others added 25 commits November 6, 2018 09:02
use https in source_urls in gzip test easyconfigs
Thanks @boegel
fix checksum for gzip-1.4.eb test easyconfig in test_exts_list
add support for deprecating easyconfig files & toolchains
…t_dup

changed easyconfig.update to be able to specify not to allow duplicate values
…s --force is used, and to be verbose about creating backups
…eys_order

fix order of keys in 'toolchain' value for dumped easyconfig file (name, version) + run style check on dumped easyconfigs in dump tests
@mboisson mboisson merged commit 54e7c20 into ComputeCanada:fix-bug-tweak Nov 15, 2018
@boegel boegel deleted the fix-bug-tweak branch November 15, 2018 12:58
mboisson pushed a commit that referenced this pull request Nov 20, 2018
bartoldeman pushed a commit that referenced this pull request Jan 30, 2019
enhance test for check_pr_eligible_to_merge to catch fixed bug w.r.t. last test report having to be successful
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.

8 participants