[@ ports committers] ports/173531

[Marco Bröder]

On 16.12.2012 12:25, Ruslan Makhmatkhanov wrote:
> Jason E. Hale wrote on 16.12.2012 15:13:
>> On Sunday, December 16, 2012 15:07:48 Ruslan Makhmatkhanov wrote:
>>> Ruslan Makhmatkhanov wrote on 16.12.2012 14:59:
>>>> Hi Marco,
>>>>
>>>> Marco Bröder wrote on 16.12.2012 13:52:
>>>>> Hi,
>>>>>
>>>>> could a ports committer take ports/173531 pr, please? It is unassigned
>>>>> for
>>>>> over a month, now.
>>>>>
>>>>> Thank you very much!
>>>>
>>>> Please review patch attached. I changed some things in your patch:
>>>> - using OPTIONSFILE instead of UNIQUENAME (it's common practice to fix
>>>>
>>>>     things like that)
>>>>
>>>> - limit to python 2.x (dulwich is not python 3.x-aware yet)
>>>> - tab -> space change in pkg-descr:WWW (style bug)
>>>> - remove DOCS_DESC (we have generic description in bsd.options.desc.mk)
>>>> - use PYTHON_CMD instead of ``/usr/bin/env python'' (to make sure it
>>>>
>>>>     will be runned with the same python branch, that it was built with)
>>>>
>>>> Please let me know if you agree with the changes, and I commit the
>>>> update shortly.
>>>>
>>>> PS. Surely, it builds fine with this patch.
>>>
>>> Oops, I see that this pr was taken by jhale already :)
>>
>> Ruslan,
>>
>> If you want to take this that is fine with me...I haven't started working on
>> it yet and it seems you have looked into it already.
>>
>>> One more thing - PORTDOCS should be defined uncoditionally, but
>>> installed conditionally:
>>>
>>> +.if ${PORT_OPTIONS:MDOCS}
>>> +PORTDOCS=	AUTHORS COPYING HACKING NEWS README introduction.txt \
>>> +		object-store.txt protocol.txt remote.txt repo.txt
>>>    .endif
>>>
>> COPYING should be removed here too.  LICENSE is already defined.
>>
>>> This condition is not necessary.
> 
> Ok, both done (take-pr and COPYING). Thanks. Awaiting of maintainer's 
> response.
> 

I reviewed and tested your changes. I am fine with all of it. Please commit.
Many thanks for your help and comments!

-- 
Kind regards

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freebsd.org/pipermail/freebsd-ports/attachments/20121216/a5590c45/attachment.sig>