Skip to content

Commit 585102c

Browse files
committed
Correct the scope note on CmdStan identity
Purpose and scope still said the installation path is not compared. That was the version-only rule, and it survived the at-a-glance rewrite because it sits in a different section — the same reviewer finding, in a place the restructure did not reach.
1 parent 0df0437 commit 585102c

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

dev-notes/compilation-state.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,9 @@ means when it reaches `make`; when the record is validated; what can and cannot
3333
known about an executable cmdstanr did not build.
3434

3535
**Out of scope:** the toolchain itself — compiler version, system libraries — which
36-
is deliberately untracked (§6). The *CmdStan version* is in scope as `builder`
37-
identity, as is whether the recorded installation still exists; the installation's
38-
path is not compared (§6).
36+
is deliberately untracked (§6). The CmdStan installation *is* in scope: its
37+
normalised path and version together are `builder` identity, and whether that
38+
installation still exists is checked as well (§6).
3939

4040
**Relation to the 1.0 milestone.** Every issue in this area is milestoned
4141
`v1.0.0 - release`, so the milestone is mostly this work — but not only: it also

0 commit comments

Comments
 (0)