Skip to content

Conversation

@JLague
Copy link
Contributor

@JLague JLague commented Jun 8, 2021

Add support for with-xaltFilePrefix=... option, which specifies where the XALT record files will be written.

@boegel boegel added this to the next release (4.4.1) milestone Jun 9, 2021
@boegel boegel changed the title Add file prefix option to xalt easyblock add file prefix option to XALT easyblock Jun 9, 2021
@smoors
Copy link
Contributor

smoors commented Jun 11, 2021

@JLague can you upload a test report with an easyconfig that uses this option?

@JLague
Copy link
Contributor Author

JLague commented Jun 11, 2021

Test report by @JLague

Overview of tested easyconfigs (in order)

  • SUCCESS XALT-2.10.15.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
build-node.computecanada.ca - Linux centos linux 7.9.2009, x86_64, Intel Xeon Processor (Skylake, IBRS), Python 3.7.7
See https://gist.github.com/bbf4b36e2111fde58d1f39ef817d2306 for a full test report.

Copy link
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

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

lgtm

@smoors
Copy link
Contributor

smoors commented Jun 11, 2021

Going in, thanks @JLague!

@smoors smoors merged commit 8d60d64 into easybuilders:develop Jun 11, 2021
@JLague JLague deleted the xalt_file_prefix branch June 11, 2021 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants