doc issue about develop support
albestro opened this issue · comments
stackinator/docs/configuring.md
Lines 45 to 51 in 5f16110
In addition to the repeated "the following", IMHO the bullet points list is not straightforward to understand. Some doubts I have:
- it probably refers to
spack env activate --with-view <name>
and notspack env --with-view
- first point: I would specify that the second part of the sentence refers to the environment configuration
- second point: are you referring to
<name>
with "positional argument"? - third point sounds like a bit unlinked. I think doc meant that
--with-view <name>
applies starting from 0.21 (and so alsodevelop
included). right? - fourth point: are you referring to
--develop
? in case, I might argue that it would be beneficial to keep it there for a while. Two arguments in support of this argument:- what if you remove it and then you have to re-add a
develop
option because another similar workaround is needed? develop
sounds general enough, so it applies to all possible workarounds needed. otherwise we might end up having different flags for different workarounds of this kind and drop just the ones not used anymore
- what if you remove it and then you have to re-add a
I really hope that spack will allows us to completely forget about this option sooner rather than later.
I agree that this could be better explained.
second point: are you referring to with "positional argument"?
This is how I interpret it.
third point sounds like a bit unlinked. I think doc meant that
--with-view <name>
applies starting from 0.21 (and so also develop included). right?
I think the doc is describing #164. The spack_version
is defined here
stackinator/stackinator/recipe.py
Lines 206 to 222 in 5f16110
So if you are not using 0.20
or 0.21
, develop
or unknown
is returned.
The string comparison in #164 looks dangerous (it evaluates to True
with both develop
and unknown
) and should be fixed, but what the doc is likely trying to convey is that if you are not using a specific version it's up to the user to trigger the condition with --develop
.
fourth point: are you referring to
--develop
?
It should only refer to the specific spack env activate --with-view <name>
workaround (see #152 for the removal of an old workaround, when it became unnecessary). --develop
will need to stay around until 1.x
, I think.
I don't recall if it's clearly stated in the documentation, but the idea is to support the last two minor versions of Spack, plus develop. So right now we should be able to work with v0.20
, v0.21
and develop
.