Skip to content

1.0/last updates tuesday sections#478

Merged
becker33 merged 5 commits into
mainfrom
1.0/last-updates-tuesday-sections
Aug 5, 2025
Merged

1.0/last updates tuesday sections#478
becker33 merged 5 commits into
mainfrom
1.0/last-updates-tuesday-sections

Conversation

@becker33

@becker33 becker33 commented Aug 5, 2025

Copy link
Copy Markdown
Member

Final minor fixes from testing on both potential AMIs

becker33 and others added 3 commits August 5, 2025 04:38
Signed-off-by: Gregory Becker <becker33@llnl.gov>
Signed-off-by: Gregory Becker <becker33@llnl.gov>
@alalazo

alalazo commented Aug 5, 2025

Copy link
Copy Markdown
Member

Instead of:

Screenshot from 2025-08-05 14-01-55

we can show the available scopes using:

$ spack config scopes
command_line             
env:/home/spack/stacks   
user                     
site                     
system                   
defaults                 
defaults:linux           
defaults:base            
_builtin                 

and maybe refer to include.yaml docs for further customization.

@alalazo alalazo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I quickly checked the configuration tutorial, and it LGTM. If there's time, I wonder if we should introduce prefer: and require: and skip the soft preferences which are the lowest priority

@alalazo alalazo Aug 5, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

(nitpick, feel free to close without modifications if it seems pedantic) Changes to this file looks good. A minor point is in this wording:

Note the identifier clang_gfortran is not itself a spec (you don’t version it). You reference it in other specs. 

I'd rather say something like:

Toolchains are not specs, so they don't have versions or variants, but are referenced only by their names, in this case by `clang_gfortran`. 

@becker33 becker33 merged commit 93b1fd8 into main Aug 5, 2025
1 check passed
@becker33 becker33 deleted the 1.0/last-updates-tuesday-sections branch August 5, 2025 13:03
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