[rb-general] BUILD_PATH_PREFIX_MAP code examples and test cases

[Ximin Luo]

Ian Jackson:
> [..]
> 
> I am surprised at your choice of ordering of the key/value pairs.  I
> would expect that most producers will have in their head a list like
>    FOO_BUILDDIR=/path/to/foo
>    BAR_BUILDDIR=/path/to/bar
>    TMP_AREA=/path/to/our/tmp
> 
> Your ordering means that such a producer must make a map specifying
>    /path/to/foo=FOO_BUILDDIR
>    /path/to/bar=BAR_BUILDDIR
>    /path/to/our/tmp=TMP_AREA
> 
> This seems unnaturla.
> 
> Or to put it another way: normally, the actual paths will vary, and
> the replacement tokens will be fixed.  = normally has the more-fixed
> thing on the left.
> 
> So I think you should swap the order of the pairs.
> 

Hi, sorry I missed this the first time around.

This is a pretty good idea, which is slightly annoying for me because it's going to take a fair bit of effort to change all the writing and the code and the test cases. But I think I'll do it.

It does make reproducible builds "look" more obviously like the primary purpose of this, which is of course a good thing. And perhaps it helps shift people's perspective away from "we are fiddling with debug paths" to "we are describing how to achieve reproducible builds".

(The ordering is reversed currently because GCC has the debug-prefix-map option like that, and joining them up was "the obvious way" of doing it.)

X

-- 
GPG: ed25519/56034877E1F87C35
GPG: rsa4096/1318EFAC5FBBDBCE
https://github.com/infinity0/pubkeys.git