Skip to content

remove necessity for license for oneAPI versions (>= 2021.x) in itac easyblock#2492

Merged
boegel merged 10 commits intoeasybuilders:developfrom
robert-mijakovic:20210627165946_new_pr_tEcylrkYts
Dec 22, 2021
Merged

remove necessity for license for oneAPI versions (>= 2021.x) in itac easyblock#2492
boegel merged 10 commits intoeasybuilders:developfrom
robert-mijakovic:20210627165946_new_pr_tEcylrkYts

Conversation

@robert-mijakovic
Copy link
Copy Markdown

(created using eb --new-pr)

Micket
Micket previously requested changes Sep 1, 2021
Comment thread easybuild/easyblocks/i/itac.py Outdated
@boegel boegel added this to the release after 4.4.2 milestone Sep 2, 2021
@boegel boegel added the update label Sep 2, 2021
@boegel boegel modified the milestones: 4.5.1, release after 4.5.1 Dec 7, 2021
@boegel boegel changed the title itac: removes necessity for license remove necessity for license for oneAPI versions (>= 2021.x) in itac easyblock Dec 21, 2021
…of oneAPI versions of itac (and move the installation afterwards to avoid duplicate subdirectory in installation prefix
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 22, 2021

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS itac-2017.1.024.eb
  • SUCCESS itac-2018.1.017.eb
  • SUCCESS itac-2018.3.022.eb
  • SUCCESS itac-2019.2.026.eb
  • SUCCESS itac-2019.4.036.eb
  • SUCCESS itac-8.0.0.011.eb
  • SUCCESS itac-8.1.4.045.eb
  • SUCCESS itac-9.0.3.051.eb

Build succeeded for 8 out of 8 (8 easyconfigs in total)
node2613.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/fbdb51bd8401c29cde9f54281516de2a for a full test report.

@boegel boegel dismissed Micket’s stale review December 22, 2021 12:38

requested changes made

Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants