Skip to content

Sarus: Use commits instead of sha 256 for correct version information#28705

Merged
haampie merged 4 commits intospack:developfrom
teonnik:sarus_use_commits
Feb 1, 2022
Merged

Sarus: Use commits instead of sha 256 for correct version information#28705
haampie merged 4 commits intospack:developfrom
teonnik:sarus_use_commits

Conversation

@teonnik
Copy link
Copy Markdown
Contributor

@teonnik teonnik commented Feb 1, 2022

  • To make sure Sarus' version information is available, the package needs to be installed from a git repository instead of an archive.
  • Add a new version of spack
  • Add another package maintainer.
  • Add a note that sudo/root installation is strongly recommended for production deployment.
  • Add an option to enable unit tests only when requested with --test in the newest version

- To retrieve the correct spack version we need to get it from the git
  repo.

- Recommend installing the package with root for production

- Add Tomas as maintainer to sarus' spack package
@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Feb 1, 2022

@Madeeks @taliaga can you review this PR?

This PR modifies the following package(s), for which you are listed as a maintainer:

  • sarus

haampie
haampie previously approved these changes Feb 1, 2022
@haampie haampie enabled auto-merge (squash) February 1, 2022 11:09
auto-merge was automatically disabled February 1, 2022 14:17

Head branch was pushed to by a user without write access

- Improve post-install message
haampie
haampie previously approved these changes Feb 1, 2022
@haampie haampie enabled auto-merge (squash) February 1, 2022 14:31
auto-merge was automatically disabled February 1, 2022 14:43

Head branch was pushed to by a user without write access

@haampie haampie enabled auto-merge (squash) February 1, 2022 15:48
@haampie haampie merged commit 9cf20b9 into spack:develop Feb 1, 2022
RikkiButler20 pushed a commit to RikkiButler20/spack that referenced this pull request Feb 1, 2022
- To retrieve the correct spack version we need to get it from the git
  repo.

- Recommend installing the package with root for production

- Add Tomas as maintainer to sarus' spack package

- Add the option to disable unit tests in latest versions
RikkiButler20 pushed a commit to RikkiButler20/spack that referenced this pull request Feb 8, 2022
- To retrieve the correct spack version we need to get it from the git
  repo.

- Recommend installing the package with root for production

- Add Tomas as maintainer to sarus' spack package

- Add the option to disable unit tests in latest versions
EthanS94 pushed a commit to EthanS94/spack that referenced this pull request Feb 17, 2022
- To retrieve the correct spack version we need to get it from the git
  repo.

- Recommend installing the package with root for production

- Add Tomas as maintainer to sarus' spack package

- Add the option to disable unit tests in latest versions
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.

2 participants