windowmaker: Use a single reinplace instead of three

Also expand the list instead of using a foreach loop and suppress warnings when reinplace didn't change anything.
esafak · Aug 7, 2024 · 5194e08 · 5194e08
1 parent 0c36982
commit 5194e08
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/x11/windowmaker/Portfile b/x11/windowmaker/Portfile
@@ -48,11 +48,7 @@ post-patch {
     set fl [glob -directory ${worksrcpath} WindowMaker/*menu*]
     lappend fl ${worksrcpath}/WindowMaker/Defaults/WindowMaker.in \
         ${worksrcpath}/WPrefs.app/Menu.c ${worksrcpath}/WPrefs.app/Paths.c
-    foreach i $fl {
-        reinplace "s|/usr/share|${prefix}/share|g" $i
-        reinplace "s|/usr/local/share|${prefix}/share|g" $i
-        reinplace "s|/opt/kde2/share|${prefix}/share|g" $i
-    }
+    reinplace -E -q "s,(/usr|/usr/local|/opt/kde2)/share,${prefix}/share,g" {*}${fl}
 }
 
 configure.env-append \