apple / cups

Apple CUPS Sources
https://www.cups.org
Apache License 2.0
1.91k stars 464 forks source link

libcups.a installed with 555 permissions, making ranlib fail #3209

Closed michaelrsweet closed 15 years ago

michaelrsweet commented 15 years ago

Version: 1.4-current CUPS.org User: fhimpe.telenet

When building cups svn, it fails because libcups.a is installed with 555 permissions (I guess because of INSTALL_LIB = $(LIBTOOL) $(INSTALL) -c -m 555 @INSTALL_STRIP@ in Makedefs.in). After libcups.a is installed, ranlib is run on the installed file, but then it fails because the file is unwritable.

make[1]: Entering directory `/home/fhimpe/cups/BUILD/cups-1.4svn-r8684/cups' /home/fhimpe/cups/BUILD/cups-1.4svn-r8684/install-sh -d -m 755 /home/fhimpe/cups/BUILDROOT/cups-1.4.0-0.8684.1mdv2010.0.i386/usr/lib /home/fhimpe/cups/BUILD/cups-1.4svn-r8684/install-sh -c -m 555 -s libcups.a /home/fhimpe/cups/BUILDROOT/cups-1.4.0-0.8684.1mdv2010.0.i386/usr/lib ranlib /home/fhimpe/cups/BUILDROOT/cups-1.4.0-0.8684.1mdv2010.0.i386/usr/lib/libcups.a ranlib: unable to copy file '/home/fhimpe/cups/BUILDROOT/cups-1.4.0-0.8684.1mdv2010.0.i386/usr/lib/libcups.a'; reason: Permission denied make[1]: *\ [installstatic] Error 1

michaelrsweet commented 15 years ago

CUPS.org User: mike

P2, as this only affects non-root installs on old systems that still ship and require a ranlib command to be run...

Will look into some possible solutions (we still want mode 555 for the final permissions...)

michaelrsweet commented 15 years ago

CUPS.org User: mike

Also, static libraries are no longer installed by default...

michaelrsweet commented 15 years ago

CUPS.org User: mike

Fixed in Subversion repository.

michaelrsweet commented 15 years ago

"str3209.patch":

Index: Makedefs.in

--- Makedefs.in (revision 8685) +++ Makedefs.in (working copy) @@ -20,6 +20,7 @@ AR = @AR@ AWK = @AWK@ CC = @LIBTOOL@ @CC@ +CHMOD = @CHMOD@ CXX = @LIBTOOL@ @CXX@ DSO = @DSO@ DSOXX = @DSOXX@

Index: config-scripts/cups-common.m4

--- config-scripts/cups-common.m4 (revision 8685) +++ config-scripts/cups-common.m4 (working copy) @@ -49,6 +49,7 @@ AC_PROG_CXX AC_PROG_RANLIB AC_PATH_PROG(AR,ar) +AC_PATH_PROG(CHMOD,chmod) AC_PATH_PROG(HTMLDOC,htmldoc) AC_PATH_PROG(LD,ld) AC_PATH_PROG(LN,ln)

Index: driver/Makefile

--- driver/Makefile (revision 8685) +++ driver/Makefile (working copy) @@ -164,8 +164,9 @@

installstatic: $(INSTALL_DIR) -m 755 $(LIBDIR)