webrev updated:

http://cr.opensolaris.org/~gvasick/6838600/

See comments inline below.


Pradhap Devarajan wrote:
> Hi George,
>    Small change use we d'nt have to explicitly untar the sources. When 
> you define TARBALL in Makefile, it automatically takes care of untar'ing 
> sources. You can just use
> 
> $(VER)/configure:
>     $(TOUCH) $(VER)/configure

The extraction did not occur automatically with the explicit unzip/untar 
removed.  They have been retained unless somebody can provide details on 
the automatic invocation.

> 
> Use $(TOUCH) and $(CFGPREFIX) for "/usr" instead of hardcoding the path.

Done and done plus some additional cleanup in the install rule to be 
consistent.


Thanks,
George

> 
> ~ Pradhap.D
> 
> Paul Cunningham wrote:
>> George,
>>
>> This looks okay to me
>>
>> Paul
>>
>> George Vasick wrote:
>>> Updated webrev posted at the same location:
>>>
>>> http://cr.opensolaris.org/~gvasick/6838600/
>>
>>>> George Vasick wrote:
>>>>>
>>>>> Please review my changes for the upgrade of autoconf from 2.61 to 
>>>>> 2.63:
>>>>>
>>>>> http://cr.opensolaris.org/~gvasick/6838600/
>>
>> _______________________________________________
>> sfwnv-discuss mailing list
>> sfwnv-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/sfwnv-discuss

Reply via email to