re: CVS commit: src/share/mk

[matthew green <mrg%eterna.com.au@localhost>]

   In article <26371.1260567226%splode.eterna.com.au@localhost>,
   matthew green  <mrg%eterna.com.au@localhost> wrote:
   >
   >   Module Name:     src
   >   Committed By:    uebayasi
   >   Date:            Fri Dec 11 08:37:34 UTC 2009
   >   
   >   Modified Files:
   >    src/share/mk: bsd.own.mk
   >   
   >   Log Message:
   >   ${GENCMD} is used to generate only one output from file generation 
commands
   >   which potentially have multiple output files, which confuses make(1).
   >   
   >   This wrapper copies all involved files into /tmp, execute the command, 
and
   >   copy the wanted file back to ${.OBJDIR}.  If this is expensive, consider 
to
   >   rewrite the generation command to output files individually.
   >
   >
   >this seems pretty gross.  i'm not sure it's a good way to
   >solve the problem at hand.  it hides everything away with
   >@.  it also doesn't appear to avoid updating a file if it
   >hasn't actually changed.  it also hard codes /tmp, which
   >it shouldn't need to use at all.  it also uses "cp"
   >instead of "mv".
   >
   >did anyone review this?
   
   I think this should be backed out; there must be better ways of dealing
   with the problem.


i agree.  the problem at hand is not new and there are much better
workarounds than this..


.mrg.