Skip to content

Commit 2d49c09

Browse files
committed
Document the ambiguity of comma separated scopes
1 parent 604dcb0 commit 2d49c09

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

text/3127-trim-paths.md

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,12 @@ the other for only debuginfo: https://reproducible-builds.org/docs/build-path/.
246246
## Per-mapping scope control
247247
If it turns out that we want to enable finer grained scoping control on each individual remapping, we could use a `scopes:from=to` syntax.
248248
E.g. `split-debuginfo,unsplit-debuginfo,diagnostics:/path/to/src=src` will remove all references to `/path/to/src` from compiler diagnostics and debug information, but
249-
they are retained in panic messages. This syntax can be used with either a brand new `--remap-path-prefix-scoped` option, or we could extend the
249+
they are retained in panic messages.
250+
251+
How exactly this new syntax will look like is, of course, up to further discussion. Using comma as a separator for scopes may look ambiguous as `macro,diagnostics:/path/from=to` could be interpreted as `macro`
252+
and `diagnostics:/path/from=to`.
253+
254+
This syntax can be used with either a brand new `--remap-path-prefix-scoped` option, or we could extend the
250255
existing `--remap-path-prefix` option to take in this new syntax.
251256

252257
If we were to extend the existing `--remap-path-prefix`, there may be an ambiguity to whether `:` means a separator between scope list and mapping,

0 commit comments

Comments
 (0)