Skip to content

Update scripting tutorial section#475

Merged
alecbcs merged 2 commits intospack:mainfrom
scheibelp:tutorial-scripting-1.0
Aug 6, 2025
Merged

Update scripting tutorial section#475
alecbcs merged 2 commits intospack:mainfrom
scheibelp:tutorial-scripting-1.0

Conversation

@scheibelp
Copy link
Copy Markdown
Member

(note this isn't presented until Wednesday, so is not urgent until evening of August 5th)

This pretty-much works out of the box. I changed a couple example outputs

  • most references to Spec.concretize were already replaced with concretize.concretize_one, but one of the output files did not
  • The docstring to spack.store.STORE.db.query changed

@alecbcs
Copy link
Copy Markdown
Member

alecbcs commented Aug 5, 2025

To test this out we should rebase on main and also add scripting.sh back into outputs/Makefile.

@scheibelp
Copy link
Copy Markdown
Member Author

@alecbcs it's now updated

I see scripting.sh in outputs/Makefile currently: the two files I modified are not generated by it (though they live in outputs/scripting/)

@alecbcs
Copy link
Copy Markdown
Member

alecbcs commented Aug 6, 2025

Ahh gotcha. So I think we're good then. We'd only need to test scripting if you modified the scripting.sh file.

Copy link
Copy Markdown
Member

@alecbcs alecbcs left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks @scheibelp!

@alecbcs alecbcs merged commit b6971a0 into spack:main Aug 6, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants