Page MenuHomeFreeBSD

Defer evaluation of modified until after it's set
ClosedPublic

Authored by imp on May 31 2019, 3:01 PM.
Tags
None
Referenced Files
F135854840: D20480.id.diff
Thu, Nov 13, 1:45 PM
F135854825: D20480.id58136.diff
Thu, Nov 13, 1:45 PM
F135854399: D20480.id58112.diff
Thu, Nov 13, 1:41 PM
F135854381: D20480.id58128.diff
Thu, Nov 13, 1:40 PM
F135854131: D20480.diff
Thu, Nov 13, 1:38 PM
F135813862: D20480.id58112.diff
Thu, Nov 13, 3:34 AM
F135792640: D20480.id58128.diff
Wed, Nov 12, 10:32 PM
Unknown Object (File)
Sun, Nov 9, 4:19 AM
Subscribers
None

Details

Summary

With the reorg r348175, we now look at modified before it is
set. Rearrange things so that we can set include_metadata to either
yes, no or some variable to be evaluated in the future (in this case
modified). Use the eval echo trick to expanded it. This should fix the
-R flag that was broken in r348175, which broke
WITH_REPRODUCIBLE_BUILD for kernels.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

No objection but I'm generally not a fan of the deferred evaluation; what about just setting include_metadata=yes/no/if-modified and making the test something like
if [${include_metadata} = yes || (${include_metadata} = if-modified && ${modified} = yes)]

Anyway if you prefer this approach that's fine with me

Fix per ed (though a little different construct)
Fix backwards test after converting from -z notation...

No objection but I'm generally not a fan of the deferred evaluation; what about just setting include_metadata=yes/no/if-modified and making the test something like
if [${include_metadata} = yes || (${include_metadata} = if-modified && ${modified} = yes)]

Anyway if you prefer this approach that's fine with me

Due to the funky way that test works, I opted for a slightly different way to say this...

This revision was not accepted when it landed; it landed in state Needs Review.May 31 2019, 10:57 PM
This revision was automatically updated to reflect the committed changes.