New program: cygtweak

[Christopher Faylor]

On Tue, Sep 30, 2003 at 11:29:29AM -0400, Igor Pechtchanski wrote:
>On Tue, 30 Sep 2003, Christopher Faylor wrote:
>
>> On Tue, Sep 30, 2003 at 11:02:39AM -0400, Igor Pechtchanski wrote:
>> >On Tue, 30 Sep 2003, Corinna Vinschen wrote:
>> >> Wouldn't it be sufficient to add $(srcdir)/cygprogctl to PROGS and to
>> >> drop the copy rule?  The script only needs installing and that should
>> >> work then.
>> >
>> >I just remembered why I did it this way: aren't all of the $(PROGS) are
>> >deleted on "make clean"?  If I added $(srcdir)/cygprogctl to PROGS, I'd
>> >have to change the "clean" rule.  I thought a copy would be easier and
>> >less intrusive.
>>
>> Good point, but I think I'd prefer something like:
>>
>>   install: all cygprogctl
>> 	  $(SHELL) $(updir1)/mkinstalldirs $(bindir) $(etcdir)
>> 	  for i in $(PROGS) ${word 2,$^} ; do \
>> 	    n=`echo $$i | sed '$(program_transform_name)'`; \
>> 	    $(INSTALL_PROGRAM) $$i $(bindir)/$$n; \
>> 	  done
>>
>> That would just let the standard install deal with installation.
>> cgf
>
>Hmm, then it'll have to be
>
>install: all $(srcdir)/cygprogctl
>	...

No, it wouldn't.  Try it.

>How is this cleaner than adding it to PROGS?

Weren't you saying that adding it to PROGS causes it to be removed with
a 'make clean'.

>dependence in the "all" rule, as a value for CLEAN_PROGS, and in the for
>loop in the "install" rule.

Except there is no reason for it to be in the all rule since it isn't
being built.

>Wait, doesn't ":=" expand the asigned value immediately?  Then Corinna's
>right, and adding to PROGS after the assignment to CLEAN_PROGS will not
>change CLEAN_PROGS (so the copy is redundant).  It would simply become
>
>Index: winsup/utils/Makefile.in
>===================================================================
>RCS file: /cvs/src/src/winsup/utils/Makefile.in,v
>retrieving revision 1.53
>diff -u -p -r1.53 Makefile.in
>--- winsup/utils/Makefile.in    12 Sep 2003 01:51:21 -0000      1.53
>+++ winsup/utils/Makefile.in    27 Sep 2003 16:24:54 -0000
>@@ -84,6 +84,8 @@ PROGS:=warn_dumper $(PROGS)
> CLEAN_PROGS+=dumper.exe
> endif
>
>+PROGS+=$(srcdir)/cygprogctl
>+
> .SUFFIXES:
> .NOEXPORT:

My proposed change eliminates the need for $(srcdir)/ but I guess the
above is fine.

cgf