[pkgsrc/trunk]: pkgsrc/textproc/openjade Fix issues with a nonstandard compil...

[brook <brook%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/c662170c5ad9
branches:  trunk
changeset: 436311:c662170c5ad9
user:      brook <brook%pkgsrc.org@localhost>
date:      Fri Jul 31 17:01:44 2020 +0000

description:
Fix issues with a nonstandard compiler and umask.

If the compiler is in a nonstandard location, libtool may choose the
incorrect mode.  This is described in libtool documentation:
     https://www.gnu.org/software/libtool/manual/html_node/Tags.html
The correct fix is to use --tag to explicitly set the mode.  See
https://sourceforge.net/p/openjade/patches/48/

Write permissions are removed when the package is built.  However, the
command chmod -w does not affect bits set in umask.  The correct usage
is to explicitly identify the users to which the command applies.  See
https://sourceforge.net/p/openjade/patches/49/

diffstat:

 textproc/openjade/distinfo                       |   7 +++--
 textproc/openjade/patches/patch-Makefile.comm.in |  32 ++++++++++++++++++++++++
 textproc/openjade/patches/patch-aa               |  18 +++++++++++--
 textproc/openjade/patches/patch-ab               |   9 +++++-
 4 files changed, 58 insertions(+), 8 deletions(-)

diffs (108 lines):

diff -r c417e8595b70 -r c662170c5ad9 textproc/openjade/distinfo
--- a/textproc/openjade/distinfo        Fri Jul 31 16:51:54 2020 +0000
+++ b/textproc/openjade/distinfo        Fri Jul 31 17:01:44 2020 +0000
@@ -1,11 +1,12 @@
-$NetBSD: distinfo,v 1.18 2020/04/30 23:46:13 joerg Exp $
+$NetBSD: distinfo,v 1.19 2020/07/31 17:01:44 brook Exp $
 
 SHA1 (openjade-1.3.2.tar.gz) = 54e1999f41450fbd62c5d466002d79d3efca2321
 RMD160 (openjade-1.3.2.tar.gz) = 3b12f6596a42c49a60cc07cd9b8f4c5fdf3bafd2
 SHA512 (openjade-1.3.2.tar.gz) = c39f70ec8020bd7a2b3e125cbd146b49ddb57910a042bd212da02722617ed5681f32dab60acc26ab89ac658997c6f604911d7be3be391d6278267481f3bcf2f1
 Size (openjade-1.3.2.tar.gz) = 894834 bytes
-SHA1 (patch-aa) = 4914f08b6598d25af7b7f99a58e1e6490d292609
-SHA1 (patch-ab) = 5e50562cf860c713b581247e223130ee7ef409b9
+SHA1 (patch-Makefile.comm.in) = 79ece1241a42e99eae8130be30adef42bc7bad84
+SHA1 (patch-aa) = cf55d4812e9bd5592ceb71e972c62c10de1bd02c
+SHA1 (patch-ab) = 7f54b7b2e52da456b673c9f0f539d03ef0850e81
 SHA1 (patch-ac) = 2492ccb9bd736318ecd5c37f1b2a963f3ff188ab
 SHA1 (patch-ad) = f8c950cc4a0ffb5bdba315a298c14a3e7ebcd637
 SHA1 (patch-ae) = b67fde4062691b2d953cb4cb5b4ebd2428c3c285
diff -r c417e8595b70 -r c662170c5ad9 textproc/openjade/patches/patch-Makefile.comm.in
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/textproc/openjade/patches/patch-Makefile.comm.in  Fri Jul 31 17:01:44 2020 +0000
@@ -0,0 +1,32 @@
+$NetBSD: patch-Makefile.comm.in,v 1.1 2020/07/31 17:01:44 brook Exp $
+
+Use --tag explicitly to always detect the correct compiler.  See
+https://sourceforge.net/p/openjade/patches/48/
+
+Explicitly remove all write permissions from files.  See
+https://sourceforge.net/p/openjade/patches/49/
+
+--- Makefile.comm.in.orig      2002-12-01 11:56:47.000000000 +0000
++++ Makefile.comm.in
+@@ -42,18 +42,18 @@ CXXDEPGENFLAGS=-MM
+       $(CXX) $(CXXFLAGS) -c $<
+ 
+ .cxx.lo:
+-      @$(LIBTOOL) --mode=compile $(CXX) $(CXXFLAGS) -c $<
++      @$(LIBTOOL) --mode=compile --tag=CXX $(CXX) $(CXXFLAGS) -c $<
+ 
+ .c.o:
+       $(CC) $(CFLAGS) -c $<
+ 
+ .c.lo:
+-      @$(LIBTOOL) --mode=compile $(CC) $(CFLAGS) -c $<
++      @$(LIBTOOL) --mode=compile --tag=CC $(CC) $(CFLAGS) -c $<
+ 
+ .m4.cxx:
+       rm -f $@
+       $(PERL) $(srcdir)/../instmac.pl $< >$@
+-      chmod -w $@
++      chmod a-w $@
+ 
+ %.h %.cxx %.rc: %.msg
+       $(PERL) -w $(srcdir)/../msggen.pl $(MSGGENFLAGS) $<
diff -r c417e8595b70 -r c662170c5ad9 textproc/openjade/patches/patch-aa
--- a/textproc/openjade/patches/patch-aa        Fri Jul 31 16:51:54 2020 +0000
+++ b/textproc/openjade/patches/patch-aa        Fri Jul 31 17:01:44 2020 +0000
@@ -1,7 +1,10 @@
-$NetBSD: patch-aa,v 1.3 2003/11/16 14:01:20 agc Exp $
+$NetBSD: patch-aa,v 1.4 2020/07/31 17:01:44 brook Exp $
 
---- configure.orig     Sun Nov  9 21:44:24 2003
-+++ configure  Sun Nov  9 21:45:58 2003
+Explicitly remove all write permissions from files.  See
+https://sourceforge.net/p/openjade/patches/49/
+
+--- configure.orig     2002-12-01 16:18:40.000000000 +0000
++++ configure
 @@ -4899,16 +4899,16 @@
  
  fi
@@ -68,3 +71,12 @@
    cat >>confdefs.h <<\_ACEOF
  #define SP_HAVE_WCHAR 1
  _ACEOF
+@@ -7358,7 +7358,7 @@
+ # the output directory.  Thus, making CWD read-only will cause this test
+ # to fail, enabling locking or at least warning the user not to do parallel
+ # builds.
+-chmod -w .
++chmod a-w .
+ save_CFLAGS="$CFLAGS"
+ CFLAGS="$CFLAGS -o out/conftest2.$ac_objext"
+ compiler_c_o=no
diff -r c417e8595b70 -r c662170c5ad9 textproc/openjade/patches/patch-ab
--- a/textproc/openjade/patches/patch-ab        Fri Jul 31 16:51:54 2020 +0000
+++ b/textproc/openjade/patches/patch-ab        Fri Jul 31 17:01:44 2020 +0000
@@ -1,4 +1,9 @@
-$NetBSD: patch-ab,v 1.4 2017/07/15 14:34:15 wiz Exp $
+$NetBSD: patch-ab,v 1.5 2020/07/31 17:01:44 brook Exp $
+
+Use --tag explicitly to always detect the correct compiler.  See
+https://sourceforge.net/p/openjade/patches/48/
+
+Include $(LDFLAGS) in linker arguments.
 
 --- Makefile.lib.in.orig       2002-01-22 11:57:53.000000000 +0000
 +++ Makefile.lib.in
@@ -7,7 +12,7 @@
  
  lib$(LIB).la: $(LT_OBJS)
 -      $(LIBTOOL) --mode=link $(CC) $(LINKFLAGS) -o lib$(LIB).la $(LT_OBJS) \
-+      $(LIBTOOL) --mode=link $(CXX) $(LINKFLAGS) $(LDFLAGS) -o lib$(LIB).la $(LT_OBJS) \
++      $(LIBTOOL) --mode=link --tag=CC $(CC) $(LINKFLAGS) $(LDFLAGS) -o lib$(LIB).la $(LT_OBJS) \
            -rpath $(libdir) -version-info $(LTVERSION) $(DEPLIBS)
  
  install: