Moriah Waterland wrote:
> I think that what you saw was just cr.opensolaris.org being flaky. Seems
> to work fine now.
> 
> thanks,
> -Moriah
> 
> Joseph J VLcek wrote:
>> Moriah Waterland wrote:
>>> Dave,
>>>
>>> I am moving the packaging install script i.preserve as well as
>>> i.CONFIG.prsv (which is a link to i.preserve) from the Legacy Install
>>> gate over to ON. These files were delivered from SUNWswmt (which
>>> primarily delivers the legacy patch tools) but i.preserve and
>>> i.CONF.prsv need to move to SUNWpkgcmdsu where the other install scripts
>>> are delivered from.
>>>
>>> The only changes I made to the i.preserve script was to add copyright
>>> information to the file.
>>>
>>> Webrev:
>>>  http://monaco.sfbay.sun.com/detail.jsf?cr=6873067
>>>
>>> Bug info:
>>>  http://cr.opensolaris.org/~mwaterl/bf6873067/
>>>  6873067 i.perserve and i.CONFIG.prsv need to be moved to ON
>>>
>>> thanks,
>>> Moriah
>>>
>>> _______________________________________________
>>> install-discuss mailing list
>>> install-discuss at opensolaris.org
>>> http://mail.opensolaris.org/mailman/listinfo/install-discuss
>>
>> Moriah,
>>
>> The URL you have list for the webrev is incorrect.
>>
>> Joe


usr/src/cmd/svr4pkg/pkgscripts/i.preserve.sh

- Copyright is 1998

- Instead of no and yes you can use the built in true and false

- you should use print and printf in place of echo

See: 
http://installzone-wiki.central.sun.com/wiki/index.php/Ksh93_Tips#use_true_and_false_ksh93_builtins

Hope this helps.
Joe

Reply via email to