cvs commit: ports/irc/insub pkg-plist

Adam Weinberger adamw at magnesium.net
Fri Dec 17 15:02:29 PST 2004


Kris Kennaway wrote:
> On Fri, Dec 17, 2004 at 05:38:16PM -0500, Adam Weinberger wrote:
> 
>>Alfred Perlstein wrote:
>>
>>>* Adam Weinberger <adamw at magnesium.net> [041217 14:10] wrote:
>>>
>>>
>>>>Cowsay needs to be part of the RUN_DEPENDS anyway. Otherwise, you have a 
>>>>bunch of cow files around with nothing that can use them. Besides, 
>>>>cowsay isn't actually part of the build, anyway.
>>>>
>>>>Furthermore, the @unexec rm %D/share/cows 2>/dev/null|| true should be 
>>>>added to cowsay's pkg-plist, not insub's.
>>>
>>>
>>>Shall I do this?
>>>
>>
>>I think so, yes. But cowsay belongs to Kris, and you should get approval 
>>from him (*poke, poke, you're Cc:'ed*).
> 
> 
> If insub gains a dependency on cowsay (as it should), then nothing
> needs to be done to cowsay, because insub will be pkg_deleted before
> cowsay, and the share/cows directory will be empty by the time cowsay
> tries to remove it.
> 
> Kris

Right, but it means that every cowsay upgrade will barf error messages. 
Which I thought is what the 2>/dev/null||true thing was there for.

# Adam


-- 
Adam Weinberger
adamw at magnesium.net || adamw at FreeBSD.org
adamw at vectors.cx    ||   adamw at gnome.org
http://www.vectors.cx


More information about the cvs-ports mailing list