diff options
| author | Todd Gamblin <tgamblin@llnl.gov> | 2021-12-23 00:29:06 -0800 |
|---|---|---|
| committer | Greg Becker <becker33@llnl.gov> | 2022-01-12 06:14:18 -0800 |
| commit | 93a6c51d8819a2f0003ed4f788f621717cbece81 (patch) | |
| tree | 739b2537fe5bb919794845f909f27ec4aee2d6cb /COPYRIGHT | |
| parent | 8880a00862026700a82de1004e24ff06e54a165a (diff) | |
| download | spack-93a6c51d8819a2f0003ed4f788f621717cbece81.tar.gz spack-93a6c51d8819a2f0003ed4f788f621717cbece81.tar.bz2 spack-93a6c51d8819a2f0003ed4f788f621717cbece81.tar.xz spack-93a6c51d8819a2f0003ed4f788f621717cbece81.zip | |
package_hash: rework `RemoveDirectives` and add a test
Previously we used `directives.__all__` to get directive names, but it wasn't
quite right -- it included `DirectiveMeta`, etc. It's not wrong, but it's also
not the clearest way to do this.
- [x] Refactor `@directive` to track names in `directive_names` global
- [x] Rename `_directive_names` to `_directive_dict_names` in `DirectiveMeta`
- [x] Add a test for `RemoveDirectives`
Co-authored-by: Danny McClanahan <1305167+cosmicexplorer@users.noreply.github.com>
Diffstat (limited to 'COPYRIGHT')
0 files changed, 0 insertions, 0 deletions