Merge pull request 'ADR 0262: a left-out module's data is backed up as this controller reads it, and an unplaceable line is listed' (#226) from fix/0262-backup-as-read into main
This commit was merged in pull request #226.
This commit is contained in:
+10
@@ -133,6 +133,16 @@ refused for it, and keeps its backup lines, so its data is still copied while it
|
||||
condition then says exactly what stopped. Until that follow-up is live, the condition and this record
|
||||
say what stops as above.
|
||||
|
||||
> **Progressive insight — 2026-10-08.** The follow-up shipped (mesh-controller pull request 162) and
|
||||
> two facts above are sharper than written. Where the paragraph says "its data is still copied", read:
|
||||
> its data is **still backed up as this controller reads it, which may not be what its newer manifest
|
||||
> asks**, because a key the controller does not know may be about that very data. And a left-out
|
||||
> module's backup lines are **best effort**: a line that cannot be placed (an access nobody placed, a
|
||||
> directory or machine fact with no value, a placement setting that does not read, in which case the
|
||||
> definition's own path is never used instead) is listed among what the plan could not place, and is
|
||||
> never an error that fails the machine's declaration. The decision stands. *Checked by*
|
||||
> `TestALeftOutModulesDataIsStillBackedUp` and `TestALeftOutModulesUnplaceableBackupLineCostsOnlyThatLine`.
|
||||
|
||||
## Consequences
|
||||
|
||||
- The notifier's font size and width become preferences with today's values as defaults, and the
|
||||
|
||||
Reference in New Issue
Block a user