Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add dev.eessi.io support including project subdirectories #885

Open
wants to merge 71 commits into
base: 2023.06-software.eessi.io
Choose a base branch
from
Open

Add dev.eessi.io support including project subdirectories #885

wants to merge 71 commits into from

Conversation

Neves-P
Copy link
Member

@Neves-P Neves-P commented Jan 24, 2025

This PR replaces open WIP PR #804 (to merge from a more up to date and tested branch) and adds support for building into the dev.eessi.io repository.

Documentation for how to install into dev.eessi.io is available in https://www.eessi.io/docs/adding_software/adding_development_software/, with maintainer documentation to come in EESSI/dev.eessi.io-example#12

With this PR, software builds are installed into /cvmfs/dev.eessi.io/2023.06/$EESSI_DEV_PROJECT/. $EESSI_DEV_PROJECT is determined automatically during the build process and comes from the name of the GitHub repository that the bot is listening according to the format: dev.eessi.io-EESSI_DEV_PROJECT. See: https://github.com/EESSI/dev.eessi.io-example

Existing builds to dev.eessi.io were tested with the code that is about to be merged as verified in EESSI/dev.eessi.io-example#14

Neves-P and others added 30 commits November 1, 2024 11:26
@Neves-P
Add EESSI_DEV_PROJET to prefix
8a6cd13
@Neves-P
Merge branch 'EESSI:2023.06-software.eessi.io' into feature/dev.eessi.io
ee6aea9
@Neves-P
Print EESSI_CVMFS_REPO to debug
0e9402b
@Neves-P
Merge branch 'feature/dev.eessi.io' of https://github.com/Neves-P/sof…
8e628be 
…tware-layer into feature/dev.eessi.io
@Neves-P
Check if running in subshell is the cause of some variables being reset
888b874
@Neves-P
Revert to run minimal_eessi_env in subshell and add log line
1af43f1
@Neves-P
Fix condition to build with subdir in prefix in dev.eessi.io
25e8a5f
@Neves-P
Add some logs to see where EESSI_CVMFS_REPO is being ovewritten
3e5e2af
@Neves-P
Add more log output for EESSI_DEV_PROJECT variable
722ca22
@Neves-P
export EESSI_DEV_PROJECT to compat layer env
1c3f20a
@Neves-P
Expose cvmfs and version override in eessi_defaults
7c9dee8
@Neves-P
Also expose EESSI_CVMFS_REPO in defaults
d5d1522
@Neves-P
Try exporting EESSI_CVMFS_INSTALL
fb45d7e
@Neves-P
Overwrite EASYBUILD_INSTALLPATH and use EESSI_CVMFS_REPO_OVERRIDE in …
a1f11fe 
…minimal_eessi_env
@Neves-P
Load $EASYBUILD_INSTALLPATH modules in dev.eessi.io
dd9fccd
@Neves-P
Use EESSI_DEV_PROJECT for check instead of EESSI_CVMFS_REPO in EASYBU…
821455b 
…ILD_INSTALLPATH override
@Neves-P
Fix running install_scripts.sh only if not in dev.eessi.io
a19c79c
@Neves-P
Remove debug printing to log
afaa94f
@Neves-P
Improved logging when $EESSI_DEV_PROJECT is set
2c3e74e
@Neves-P
Fix tarball creations in dev.eessi.io
3d9424d
@Neves-P
Actually set install_prefix_dir
028f935
@Neves-P
Log EESSI_DEV_PROJECT and export it
4f06864
@Neves-P
Fix test checking for ${install_prefix_dir}
9eca5e4
@Neves-P
Look for installed software in the right directory
0e68f18
@Neves-P
Fix error reporting of input in create_tarball.sh
82d36c7
@Neves-P
Fix installpath to include cpu target
3248438
@Neves-P
Merge branch 'EESSI:2023.06-software.eessi.io' into feature/dev.eessi.io
d162ef3
@Neves-P
Track EASYBUILD_EXPERIMENTAL set
4db1083
@Neves-P
Fix typo in path
720d6e9
@Neves-P
Use $TOPDIR to load_eessi_extend_module.sh
e126f78
Neves-P added 30 commits February 1, 2025 16:19
@Neves-P
Shorten EESSI_PROJECT_INSTALL path
b3399f2
@Neves-P
Remove more redundant parts of path
9f0b2df
@Neves-P
Restore right part of path
3fc95e1
@Neves-P
Use EESSI_SITE_INSTALL instead
28c6b7e
@Neves-P
Try EESSI_PROJECT_INSTALL without minimal_eessi_env override
39d67eb
@Neves-P
Set EESSI_PROJECT_INSTALL in minimal_eessi_env
a37ae0b
@Neves-P
Try full path
2145b93
@Neves-P
Revert "Try full path"
145427f 
This reverts commit 2145b93.
@Neves-P
Revert "Set EESSI_PROJECT_INSTALL in minimal_eessi_env"
751aa89 
This reverts commit a37ae0b.
@Neves-P
Revert "Try EESSI_PROJECT_INSTALL without minimal_eessi_env override"
5759cd5 
This reverts commit 39d67eb.
@Neves-P
Revert "Use EESSI_SITE_INSTALL instead"
cc240d1 
This reverts commit 28c6b7e.
@Neves-P
Reapply "Use EESSI_SITE_INSTALL instead"
2769b6c 
This reverts commit cc240d1.
@Neves-P
Reapply "Try EESSI_PROJECT_INSTALL without minimal_eessi_env override"
1fead90 
This reverts commit 5759cd5.
@Neves-P
Reapply "Set EESSI_PROJECT_INSTALL in minimal_eessi_env"
13b2190 
This reverts commit 751aa89.
@Neves-P
Reapply "Try full path"
8b4b022 
This reverts commit 145427f.
@Neves-P
Revert "Try full path"
33a9d64 
This reverts commit 2145b93.
@Neves-P
Revert "Set EESSI_PROJECT_INSTALL in minimal_eessi_env"
228fe3b 
This reverts commit a37ae0b.
@Neves-P
Revert "Try EESSI_PROJECT_INSTALL without minimal_eessi_env override"
72395ea 
This reverts commit 39d67eb.
@Neves-P
Revert "Use EESSI_SITE_INSTALL instead"
b5b9916 
This reverts commit 28c6b7e.
@Neves-P
Revert "Restore right part of path"
00fc025 
This reverts commit 3fc95e1.
@Neves-P
Revert "Remove more redundant parts of path"
cb41a85 
This reverts commit 9f0b2df.
@Neves-P
Revert "Shorten EESSI_PROJECT_INSTALL path"
ab7245a 
This reverts commit b3399f2.
@Neves-P
Revert "Unset EESSI_CVMFS_INSTALL on dev.eessi.io builds"
d9bed4f 
This reverts commit 72ba311.
@Neves-P
Revert "Fix syntax error"
2dd7650 
This reverts commit 35bd8a1.
@Neves-P
Revert "Add some debugging infop"
5bb1c56 
This reverts commit a4d61d6.
@Neves-P
Revert "Set EESSI_DEV_PROJECT via EESSI-extend's EESSI_PROJECT_INSTALL"
dc95dbd 
This reverts commit 1ecd93b.
@Neves-P
Try again EESSI_PROJECT_INSTALL; temp override EESSI_CVMFS_REPO
e5517b6
@Neves-P
Also unset $EESSI_CVMFS_INSTALL
6c9e295
@Neves-P
Fix typo in unset
b71e2f9
@Neves-P
Need to export EESSI_PROJECT_INSTALL
9211451
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ocaisa ocaisa ocaisa left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
dev.eessi.io enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Neves-P @ocaisa