archivers/zip in 9.x:

[Ruslan Mahmatkhanov]

Ruslan Mahmatkhanov wrote on 10.01.2012 01:06:
> Michael Scheidell wrote on 10.01.2012 00:30:
>> I think I saw something in the list that FreeBSD 9.x has zip already
>> built in? so, ports that need 'zip' won't need zip?
>>
>> for maintainers, they should use something like:
>>
>> USE_ZIP= yes
>>
>> to:
>>
>> if ${OSVERSION} <= 900000
>> USE_ZIP= yes
>> .endif
>>
>> and/or BUILD_DEPENDS+= zip:${PORTSDIR}/archivers/zip
>>
>> (at what OSVERSION was zip included in the base?)
>>
>>
>
> unzip, not zip. I believe it should be handled in bsd.port.mk like it
> done for USE_XZ. Something like:
>
> .if defined(USE_ZIP) && ${OSVERSION} < 900000
> EXTRACT_DEPENDS+= ${LOCALBASE}/bin/unzip:${PORTSDIR}/archivers/unzip
> .endif
>
> Correct OSVERSION should be used instead of 900000. There is also should
> be some logic for UNZIP_CMD.
>

.if exists(/usr/bin/unzip)
UNZIP_CMD?=        /usr/bin/unzip
.else
UNZIP_CMD?=        ${LOCALBASE}/bin/unzip
.endif

in bsd.commands.mk

-- 
Regards,
Ruslan

Tinderboxing kills... the drives.