On 15 Dec 2021 14:48, Karl Berry wrote: > > rm -f NOTFOUND $(VAR) > i think this is an interesting route. > > I agree. > > we could do: > rm -f $(am__rm_f_notfound) ... > and am__rm_f_notfound could be set based on the configure test. > > Sounds plausible to me. The "only" problem is that there are dozens or > hundreds of places in the code that do rm -f. They would all have to be > inspected and changed (or not) ... -k
the issue isn't `rm -f` exactly, it's `rm -f` with a variable that could be
empty. so that narrows it down at least by half.
if we focus on ones using the pattern `test -z ... || rm -f ...`, there's
barely a dozen of those. i'm not sure we need to fully audit the tree and
review every `rm`, just do a best effort and wait for feedback (i.e. bugs).
we could add a helper var to further collapse the boiler plate.
am__rm_f = rm -f $(am__rm_f_notfound)
then the rule is a bit more palatable:
$(am__rm_f) ...
or if we want to claim namespace and make it something we want to export
for users of automake akin to AC_PROG_LN_S, we could use RM_F.
-mike
signature.asc
Description: PGP signature
