Skip to content

Make repos clone from JCSDA-internal#109

Merged
rtodling merged 2 commits into
developfrom
bugfix/check_branch
Feb 17, 2026
Merged

Make repos clone from JCSDA-internal#109
rtodling merged 2 commits into
developfrom
bugfix/check_branch

Conversation

@mranst
Copy link
Copy Markdown
Collaborator

@mranst mranst commented Feb 17, 2026

Closes #108. Fixes bug related to checking for existence of commit. Changes build order so that commits will clone from JCSDA-internal first

@mranst mranst requested review from Dooruk and rtodling February 17, 2026 16:35
@mranst mranst added the bug Something isn't working label Feb 17, 2026
@rtodling
Copy link
Copy Markdown
Contributor

rtodling commented Feb 17, 2026

@Dooruk @mranst @mathomp4 I will say there now even more problems ... no w/ the changes above (I can test them yet).

I am following the bundle instructions to build on discover ... the first item is:

module purge
/home/rtodling% source /discover/nobackup/projects/gmao/advda/swell/jedi_modules/spackstack_1.9_intel_bundle

and this the error I get:

Bad : modifier in $ '-'.

What's going on?!

@mranst
Copy link
Copy Markdown
Collaborator Author

mranst commented Feb 17, 2026

I am following the bundle instructions to build on discover ... the first item is:

module purge /home/rtodling% source /discover/nobackup/projects/gmao/advda/swell/jedi_modules/spackstack_1.9_intel_bundle

Try this:

module purge
source /discover/nobackup/projects/gmao/advda/swell/jedi_modules/spackstack_1.9_intel

(No "bundle" at the end)

That's the one we use for swell, I've never used the other one before. I can fix that in the docs here

@rtodling
Copy link
Copy Markdown
Contributor

I am following the bundle instructions to build on discover ... the first item is:
module purge /home/rtodling% source /discover/nobackup/projects/gmao/advda/swell/jedi_modules/spackstack_1.9_intel_bundle

Try this:

module purge
source /discover/nobackup/projects/gmao/advda/swell/jedi_modules/spackstack_1.9_intel

(No "bundle" at the end)

That's the one we use for swell, I've never used the other one before. I can fix that in the docs here

Ok @mranst thanks ... but if the one in the documentation doesn't work perhaps we need to update the doc.

Copy link
Copy Markdown
Contributor

@rtodling rtodling left a comment

Choose a reason for hiding this comment

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

I hope this works ... but it's really not simple to test jedi_bundle ...

@rtodling rtodling merged commit 51e246c into develop Feb 17, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Check for existence of commit is causing repos to not be cloned from JCSDA-internal

2 participants