[Groff-commit] groff/contrib/groffer ChangeLog Makefile.sub

[Werner LEMBERG]

CVSROOT:        /cvsroot/groff
Module name:    groff
Changes by:     Werner LEMBERG <wl>     11/10/17 06:01:59

Modified files:
        contrib/groffer: ChangeLog Makefile.sub 

Log message:
        Do not use INSTALL_DATA and INSTALL_SCRIPT during build phase.
        
        * Makefile.sub (groffer): Replace INSTALL_DATA and
        INSTALL_SCRIPT with `cp'.  Otherwise the build phase might
        fail with `permission denied' if you use the `-o root' flag
        of the `install' command.  The use of INSTALL_* is an abuse
        anyway because it's not installing anything at that point;
        it's just copying files around locally, within in the build
        directories.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/groff/contrib/groffer/ChangeLog?cvsroot=groff&r1=1.59&r2=1.60
http://cvs.savannah.gnu.org/viewcvs/groff/contrib/groffer/Makefile.sub?cvsroot=groff&r1=1.31&r2=1.32

Patches:
Index: ChangeLog
===================================================================
RCS file: /cvsroot/groff/groff/contrib/groffer/ChangeLog,v
retrieving revision 1.59
retrieving revision 1.60
diff -u -b -r1.59 -r1.60
--- ChangeLog   21 Mar 2011 21:37:54 -0000      1.59
+++ ChangeLog   17 Oct 2011 06:01:57 -0000      1.60
@@ -1,3 +1,15 @@
+2011-10-17  Ingo Schwarze  <address@hidden>
+
+       Do not use INSTALL_DATA and INSTALL_SCRIPT during build phase.
+
+       * Makefile.sub (groffer): Replace INSTALL_DATA and
+       INSTALL_SCRIPT with `cp'.  Otherwise the build phase might
+       fail with `permission denied' if you use the `-o root' flag
+       of the `install' command.  The use of INSTALL_* is an abuse
+       anyway because it's not installing anything at that point;
+       it's just copying files around locally, within in the build
+       directories.
+
 2011-03-21  Jan Vcelak <address@hidden>
            Bernd Warken
 

Index: Makefile.sub
===================================================================
RCS file: /cvsroot/groff/groff/contrib/groffer/Makefile.sub,v
retrieving revision 1.31
retrieving revision 1.32
diff -u -b -r1.31 -r1.32
--- Makefile.sub        22 Mar 2011 03:01:46 -0000      1.31
+++ Makefile.sub        17 Oct 2011 06:01:58 -0000      1.32
@@ -6,7 +6,7 @@
 #   Free Software Foundation, Inc.
 # Written by Werner Lemberg <address@hidden> and Bernd Warken.
 
-# Last update: 22 Mar 2011
+# Last update: 17 Oct 2011
 
 # This file is part of `groffer' which is part of `groff'.
 
@@ -77,9 +77,9 @@
            -e "s|@VERSION@|$(version)$(revision)|g" \
            -e "$(SH_SCRIPT_SED_CMD)" \
            $(srcdir)/perl/groffer.pl >$@; \
-         $(INSTALL_DATA) $(srcdir)/perl/groffer.man groffer.man; \
+         cp $(srcdir)/perl/groffer.man groffer.man; \
          for f in $(ROFF2PROGS); do \
-           $(INSTALL_SCRIPT) $(srcdir)/perl/roff2.pl $$f; \
+           cp $(srcdir)/perl/roff2.pl $$f; \
          done; \
        else \
          : >HAVE_SHELL; \
@@ -92,9 +92,9 @@
            -e "s|@VERSION@|$(version)$(revision)|g" \
            -e "$(SH_SCRIPT_SED_CMD)" \
             $(srcdir)/shell/groffer.sh >$@; \
-         $(INSTALL_DATA) $(srcdir)/shell/groffer.man groffer.man; \
+         cp $(srcdir)/shell/groffer.man groffer.man; \
          for f in $(ROFF2PROGS); do \
-           $(INSTALL_SCRIPT) $(srcdir)/shell/roff2.sh $$f; \
+           cp $(srcdir)/shell/roff2.sh $$f; \
          done; \
        fi
        chmod +x $@