Skip to content

Commit c43e5f8

Browse files
Copilotgreenc-FNAL
andcommitted
Clarify setup-env.sh paths in Spack integration section
- Change generic 'setup-env.sh' to explicit 'scripts/setup-env.sh' - Update dependency installation guidance to mention both repository and workspace-level paths - Addresses review feedback on line 165-166 Co-authored-by: greenc-FNAL <2372949+greenc-FNAL@users.noreply.github.com>
1 parent e65b475 commit c43e5f8

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/copilot-instructions.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ If the workspace root contains a `srcs/` directory, it may contain symbolic link
162162

163163
The project uses Spack for dependency management in development and CI environments:
164164

165-
- **Spack Environments**: The `setup-env.sh` script automatically activates Spack environments when available
165+
- **Spack Environments**: The `scripts/setup-env.sh` script automatically activates Spack environments when available
166166
- **Loading Additional Packages**: If you need tools or libraries not loaded by default, use `spack load <package>` to bring them into the environment
167167
- **Common Use Cases**:
168168
- `spack load cmake` - Load CMake if not in current environment
@@ -174,7 +174,7 @@ The project uses Spack for dependency management in development and CI environme
174174

175175
When suggesting installation of dependencies:
176176

177-
- Prefer `setup-env.sh` sourcing as it handles both Spack and system packages
177+
- Prefer sourcing the environment setup script (`scripts/setup-env.sh` or workspace-level `setup-env.sh`) as it handles both Spack and system packages
178178
- For manual installations, provide both Spack (`spack install/load`) and system package manager options
179179
- Consult `scripts/README.md` and `scripts/QUICK_REFERENCE.md` for common patterns
180180

0 commit comments

Comments
 (0)