cvs commit: ports/net/erlyvideo Makefile pkg-plist

[Ruslan Mahmatkhanov]

02.06.2011 23:52, Wesley Shields пишет:
> On Thu, Jun 02, 2011 at 10:21:40PM +0400, Ruslan Mahmatkhanov wrote:
>> 02.06.2011 20:57, Wesley Shields ??????????:
>>
>>>> | @@ -39,6 +40,7 @@ post-patch:
>>>> |  		${WRKSRC}/priv/${PORTNAME}.conf.sample
>>>> |  	@${FIND} ${WRKSRC} -type f -name .gitignore | ${XARGS} ${RM}
>>>> |  	@${FIND} ${WRKSRC} -type f -name "*.bak" | ${XARGS} ${RM}
>>>> | +	@${FIND} ${WRKSRC} -type f -name "*.orig" | ${XARGS} ${RM}
>>>> |  	@${RMDIR} ${WRKSRC}/src/plugins
>>>
>>> I understand Ruslan was just following the convention used in this port
>>> but I don't like this construct. We should at least be doing -print0 |
>>> xargs -0 if we are going to be using xargs like this. I, personally,
>>> would prefer us to use the -delete option to find.
>>
>> I'm fully agree with that findelete stuff, so here is the patch,
>> thanks for pointing out. Surely i picked this construct in some
>> another port, but can't recall which one.
>
> Thanks for taking care of this quickly. I will try and scour the tree in
> the upcoming days to find more cases of this.


Here is complete (?) list of them - 112 ports:
http://pastebin.com/7abRca2h

I can prepare patches if wish

>
>>>
>>> I wonder if there are other places in the ports tree where a similar
>>> construct can be found that we should also cleanup.
>>>
>>> -- WXS
>>
>> There is plenty of ports that use `find | xargs rm/rmdir` construct.
>> ros-*/rox-* ports for example.
>
> Thanks for pointing these out!
>
> -- WXS
>
>

-- 
Regards,
Ruslan