spack/lib/spack
Todd Gamblin 0d0ff44e3e
Spec: Remove _normal attribute and unused constructor arguments (#48119)
The `_normal` attribute on specs is no longer used and has no meaning.
It's left over from part of the original concretizer.

The `concrete` constructor argument is also not used by any part of core.

- [x] remove `_normal` attribute from `Spec`
- [x] remove `concrete` argument from `Spec.__init__`
- [x] remove unused `check_diamond_normalized_dag` function in tests
- [x] simplify `Spec` constructor and docstrings

I tried to add typing to `Spec` here, but it creates a huge number of type issues
because *most* things on `Spec` are optional. We probably need separate `Spec` and
`ConcreteSpec` classes before attempting that.

Signed-off-by: Todd Gamblin <tgamblin@llnl.gov>
2024-12-14 15:24:22 +01:00
..
docs spec.parser / spec.token: improvements (#48063) 2024-12-12 17:08:20 +01:00
env cc: parse RPATHs when in ld mode 2024-11-04 19:52:08 +01:00
external archspec: update to v0.2.5 (#46958) 2024-10-14 19:09:17 +02:00
llnl ga: add a pylint check to avoid adding open calls without encoding= (#48099) 2024-12-13 21:21:26 +01:00
spack Spec: Remove _normal attribute and unused constructor arguments (#48119) 2024-12-14 15:24:22 +01:00
spack_installable Update copyright year to 2024 (#41919) 2024-01-02 09:21:30 +01:00