Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 01 Nov 1999 18:02:51 -0500 (EST)
From:      John Baldwin <jobaldwi@vt.edu>
To:        "Rodney W. Grimes" <freebsd@gndrsh.dnsmgr.net>
Cc:        cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org, (Chris Costello) <chris@FreeBSD.org>
Subject:   Re: cvs commit: src/usr.sbin/mergemaster Makefile mergemaster
Message-ID:  <199911012302.SAA95938@server.baldwin.cx>
In-Reply-To: <199911011727.JAA40199@gndrsh.dnsmgr.net>

next in thread | previous in thread | raw e-mail | index | archive | help

On 01-Nov-99 Rodney W. Grimes wrote:
>> chris       1999/10/31 20:29:06 PST
>> 
>>   Modified files:
>>     usr.sbin/mergemaster Makefile 
>>   Removed files:
>>     usr.sbin/mergemaster mergemaster 
>>   Log:
>>   ``mergemaster'' was repo-copied to ``mergemaster.sh'' as it's a
>>   shell script.
>>   
>>   Delete the original mergemaster and modify Makefile to install
>>   mergemaster.sh.
> 
> Does it install it as ``mergmaster.sh''?  If so then this is not a
> good idea,
> there have always been things that are shell scipts installed in *bin
> that are
> not called *.sh:
> 
> gndrsh:rgrimes {1015}% file /usr/bin/* | grep -i shell
> /usr/bin/apropos:      Bourne shell script text
./gnu/usr.bin/man/apropos/apropos.sh

> /usr/bin/c89:          Bourne shell script text
src/usr.bin/c89/c89.sh

> /usr/bin/clear:        Bourne shell script text
./usr.bin/tput/clear.sh

etc..

Using .sh in src/ and then installing without the '.sh' is very
prominent throughout teh source tree.  Shoot, even sys.mk includes
rules for installing foo.sh as foo and making it executable.  I
completely understood the commit message, and the silent majority seems
to have as well.

<rant>
I like green bike sheds myself.
</rant>
 
> -- 
> Rod Grimes - KD7CAX @ CN85sl - (RWG25)              
> rgrimes@gndrsh.dnsmgr.net

---

John Baldwin <jobaldwi@vt.edu> -- http://www.cslab.vt.edu/~jobaldwi/
PGP Key: http://www.cslab.vt.edu/~jobaldwi/pgpkey.asc
"Power Users Use the Power to Serve!"  -  http://www.FreeBSD.org/


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199911012302.SAA95938>