2015-10-24 14:54 GMT+02:00 Donovan Watteau <tso...@gmail.com>:
> ping
>
> On Sun, 11 Oct 2015, Donovan Watteau wrote:
>> Hi,
>>
>> The following diff updates archivers/innoextract to 1.5.
>>
>> Here are the changes, among other things:
>>
>>  - Added support for Inno Setup 5.5.6 installers
>>  - Added support for a modified Inno Setup 5.5.0 variant
>>  - Added a simple --include (-I) option to filter files
>>  - Added a --exclude-temp (-m) option to not extract temporary files
>>  - Added a --collisions option to abort or rename files on collision
>>  - Added a --gog (-g) option to extract additional .bin files using
>>    unrar or unar
>>  - Colors are now only enabled automatically if $TERM is set to
>>    something other than "dumb"
>>
>> Tested on loongson and macppc.
>>
>> Index: Makefile
>> ===================================================================
>> RCS file: /cvs/ports/archivers/innoextract/Makefile,v
>> retrieving revision 1.1.1.1
>> diff -u -p -r1.1.1.1 Makefile
>> --- Makefile  21 Nov 2014 19:04:14 -0000      1.1.1.1
>> +++ Makefile  11 Oct 2015 13:16:23 -0000
>> @@ -2,12 +2,12 @@
>>
>>  COMMENT=     unpack Inno Setup installers
>>
>> -DISTNAME=    innoextract-1.4
>> +DISTNAME=    innoextract-1.5
>>
>>  CATEGORIES=  archivers
>> -MASTER_SITES=        ${MASTER_SITE_SOURCEFORGE:=innoextract/}
>> +HOMEPAGE=    https://constexpr.org/innoextract/
>>
>> -HOMEPAGE=    http://constexpr.org/innoextract/
>> +MASTER_SITES=        ${HOMEPAGE}files/
>>
>>  MAINTAINER=  Donovan Watteau <tso...@gmail.com>
>>
>> @@ -19,6 +19,8 @@ WANTLIB+=   boost_system-mt c lzma m pthre
>>
>>  LIB_DEPENDS= archivers/xz \
>>               devel/boost>=1.53.0p6
>> +
>> +RUN_DEPENDS= archivers/unrar
>>
>>  MODULES=     converters/libiconv devel/cmake
>>
>> Index: distinfo
>> ===================================================================
>> RCS file: /cvs/ports/archivers/innoextract/distinfo,v
>> retrieving revision 1.1.1.1
>> diff -u -p -r1.1.1.1 distinfo
>> --- distinfo  21 Nov 2014 19:04:14 -0000      1.1.1.1
>> +++ distinfo  11 Oct 2015 13:16:23 -0000
>> @@ -1,2 +1,2 @@
>> -SHA256 (innoextract-1.4.tar.gz) = 
>> WklLWuHkEENXQJZ2HTFpXtix053+ZIlTBtDZJiOQHMk=
>> -SIZE (innoextract-1.4.tar.gz) = 151230
>> +SHA256 (innoextract-1.5.tar.gz) = 
>> 8llOmSzPKFlFVHV5SAOymmc5P622nU3x7sNMRR/6SM8=
>> +SIZE (innoextract-1.5.tar.gz) = 179582

The update itself looks like okay (I'm not an innoextract user,
though). I'm a bit worried about the implicit Python dependency,
though: the build process uses Python directly via StyleCheck.cmake,
and the Python is picked "by accident" via boost dependency. IMHO,
it'll be better to add MODULES+=lang/python and MODPY_RUNDEP=No.

I'll commit the patch as it is for now.

--
  WBR,
  Vadim Zhukov

Reply via email to