Right -  I have fixed this too and updated the webrev.

Steve C.



Paul Cunningham wrote:
> 
> 
> Steve Christensen wrote:
>> You are right - I have fixed the pkginfo.tmpl file.  The webrev has 
>> been updated.
> 
> I don't think that there is supposed to be alpha chars in the 
> pkginfo.tmpl version lines if I remember correctly ...
> 
>    +VERSION="2.5.2p1,REV=0.0.0"
>    +SUNW_PRODVERS="RELEASE/2.5.2p1"
> 
> I think the get-around for that was to replace it with a dot
> 
> paul
> 
>>
>> Steve C.
>>
>> Paul Cunningham wrote:
>>
>>> The stuff in the webrev looks fine to me, but shouldn't the package's 
>>> SFWamnda/pkginfo.tmpl file have changed also?
>>>
>>> Paul
>>>
>>> Steve Christensen wrote:
>>>
>>>> This is a code review request for the update of the SFWamnda 
>>>> (amanda) package to version 2.5.2p1.
>>>>
>>>> The webrev is at
>>>> http://companion.sunfreeware.com/downloads/amandawebrev/
>>>>
>>>> The changes were:
>>>>
>>>> 1. Update of source version numbers and source file.
>>>> 2. Update of copyrights and removal of sccs ids.
>>>> 3. Addition of METADATA file.
>>>> 4. Changes of library version numbers in install-sfw and 
>>>> prototype_com files.
>>>> 5. Addition and removal of files in install-sfw and prototype_com 
>>>> files.
>>>>
>>>> This build cleanly on Nevada 62 and the package installs with no 
>>>> problems
>>>> and produces working executables.
>>>>
>>>> Steve Christensen
>>>> -- 
>>>> This messages posted from opensolaris.org
>>>> _______________________________________________
>>>> companion-discuss mailing list
>>>> companion-discuss at opensolaris.org
>>>> http://opensolaris.org/mailman/listinfo/companion-discuss
>>>>
>>>
>>
> 

Reply via email to