Re: Time to update x11/xsnow?

[Marc Baudoin <babafou%babafou.eu.org@localhost>]

Marc Baudoin <babafou%babafou.eu.org@localhost> écrit :
> Greg Troxel <gdt%lexort.com@localhost> écrit :
> > 
> > There are two patches, and it would be nice if you could get them merged
> > upstream.  One issue is using beyond-POSIX sed
> 
> I have contacted the author and he's agreed to integrate it in
> the next release.

But he points:

   Only minor problem is, that the patched version replaces every
   occurrence of the string "-copyright-" with the copyright
   statements, while he original version  only does that for the
   first occurrence.

and suggests pkgsrc used the following patch until the next xsnow
release:

-----------------------------------------------------------------

--- addcopyright.sh.orig        2021-01-11 12:22:11.000000000
+0000
+++ addcopyright.sh
@@ -57,7 +57,7 @@ while [ "$1" ] ; do
    # Notice that sed requires a newline after the filename of the 'r' command
    sed -i "/^\s*#-#/d" "$f"
    sed -i "/$txt/{r $crfile
-   :a;n;ba}" "$f"
+:a;n;ba;}" "$f"
    n=`expr $n + 1`
 done
 echo "$n files copyrighted"

-----------------------------------------------------------------

Would that be OK?