On 13 Jan 2023 15:36, Karl Berry wrote: > - test -d "$(distdir)" || mkdir "$(distdir)" > + $(AM_V_at)$(MKDIR_P) "$(distdir)" > > Ok by me, but why the AM_V_at for this particular command? > I don't see it used anywhere else in distdir.am.
this is certainly true, and it's something i want to fix ... at some point. imo we overly rely on explicit @ in many places which can make debugging failures painful. i imagine the current state is due to the silent-rules work focusing on the most common paths (compile/rebuilds) and leaving the rest to "one day". so if we ever get to that day, we'd use $(AM_V_at) in this location. i figured making the change while i was here means we have a slightly higher chance of this code eventually being silent-rules compat vs finding the time to convert it all at once. -mike
signature.asc
Description: PGP signature
