ports/102439: port upgrade: devel/monotone 0.29

Lapo Luchini lapo at lapo.it
Tue Aug 29 19:45:54 UTC 2006


Pav Lucistnik wrote:
>> It was meant to get rid of $PREFIX/share/zsh/site-functions directory
>> for people that don't use shells/zsh, while people that do use zsh
>> should have that directory (empty or not).
>>     
> I'm not sure it would work. Probably it wouldn't.
>   
...I wasn't "sure" neither, that's why I spent time to actually test it
both with and without zsh, before sending the PR...
> Right now, monotone port will kill the site-functions directory if user
> got zsh installed.
OK, the "totally correct" solution would be to add a dependence on zsh,
but seems quite overkill to me to add a dependence just for a completion
script. (and Musha Akinori seems to agree, as he committed the similar
line in sysutils/portupgrade/pkg-plist#rev1.25)

On the other hand I see no reason at all to "just remove it" when there
is a fairly simple solution to remove it only when necessary.

Please, do test it as long as you need to be sure about it, but consider
applying the rest of the patch:

diff -ruN monotone/Makefile monotone-0.29/Makefile
--- monotone/Makefile   Mon Aug 28 15:46:03 2006
+++ monotone-0.29/Makefile      Wed Aug 23 12:34:51 2006
@@ -26,7 +26,7 @@
                MAKEINFO="makeinfo --no-split"
 
 PLIST_FILES=   bin/mtn share/zsh/site-functions/_mtn \
-               "@dirrm share/zsh/site-functions" \
+               "@unexec [ -f %D/bin/zsh ] || rmdir
%D/share/zsh/site-functions 2>/dev/null || true" \
                "@dirrmtry share/zsh"
 INFO=          monotone
 MAN1=          mtn.1

PS: thanks for having added the missing ${MKDIR}.



More information about the freebsd-ports-bugs mailing list