On Mon, Apr 13, 2020 at 10:29 AM Nicolas Iooss <nicolas.iooss@xxxxxxx> wrote: > > On Thu, Apr 2, 2020 at 5:45 PM Chris PeBenito > <chpebeni@xxxxxxxxxxxxxxxxxxx> wrote: > > > > The following patch set is for adding the -E option to setfiles which would > > treat conflicting specifications as errors, rather than choosing from one of the > > specifications. Choosing one of the specs introduces non-determinism which is > > problematic, such as where filesystem images are being built in Yocto. Without > > this option, the build is not repeatable, since you can have the same source > > tree and have different labeling outcomes in the output image. > > > > This results in setfiles output like the following: > > > > # ./setfiles -E file_contexts /tmp > > ./setfiles: conflicting specifications for /tmp/testfile and /tmp/hardlink, > > using system_u:object_r:no_access_t:s0. > > ./setfiles: treating conflicting specifications as an error. > > ./setfiles: filespec_add error: /tmp/testfile > > This looks good to me. If nobody raises an objection, I will merge > this tomorrow. > > Acked-by: Nicolas Iooss <nicolas.iooss@xxxxxxx> Merged. Thanks, Nicolas
