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

Gerald Pfeifer gerald at pfeifer.com
Fri Jun 3 15:54:25 UTC 2011


On Fri, 3 Jun 2011, Ruslan Mahmatkhanov wrote:
>>> 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.
> Here is complete (?) list of them - 112 ports:
> http://pastebin.com/7abRca2h
> 
> I can prepare patches if wish

What's more important than that, I think, is addressing this in a
more systematic manner to prevent new occurrences from slipping in:

So I went ahead and filed PR ports/157566: portlint: Warng about use
of ${FIND}...${XARGS} {RM}.  Perhaps you can cook up a patch? :-)

In my experience, the portlint maintainer is very responsive in making
this kind of change or accepting/fixing proposed patches, and I encourage
all of us doing it

Gerald


More information about the cvs-all mailing list