All of lore.kernel.org
 help / color / mirror / Atom feed
From: Harald Hoyer <harald@redhat.com>
To: linux-hotplug@vger.kernel.org
Subject: Re: [PATCH 2/2] Fixed installation path for 64-bit libvolume_id.so
Date: Tue, 08 Jul 2008 06:53:51 +0000	[thread overview]
Message-ID: <48730EFF.6040309@redhat.com> (raw)
In-Reply-To: <1215468723-18298-2-git-send-email-thomas@koeller.dyndns.org>


[-- Attachment #1.1: Type: text/plain, Size: 203 bytes --]

Thomas Koeller wrote:
> Installing 64-bit libraries in /lib and /usr/lib
> is wrong.
> 

Fedora has the attached patch and builds on 64bit systems with:
$ make libdir="/lib64" usrlibdir="/usr/lib64" ...

[-- Attachment #1.2: udev-091-lib64.patch --]
[-- Type: text/plain, Size: 2346 bytes --]

diff -up udev-120/extras/volume_id/lib/Makefile.lib64 udev-120/extras/volume_id/lib/Makefile
--- udev-120/extras/volume_id/lib/Makefile.lib64	2008-04-04 14:13:33.000000000 +0200
+++ udev-120/extras/volume_id/lib/Makefile	2008-04-04 14:15:05.000000000 +0200
@@ -4,9 +4,9 @@
 #
 # Released under the GNU General Public License, version 2.
 #
-includedir = 	${prefix}/usr/include
-libdir =	${prefix}/lib
-usrlibdir =	${prefix}/usr/lib
+includedir ?= 	${prefix}/usr/include
+libdir ?=	${prefix}/lib
+usrlibdir ?=	${prefix}/usr/lib
 
 INSTALL = install -c
 INSTALL_DATA  = ${INSTALL} -m 644
@@ -115,6 +115,7 @@ ifeq ($(libdir),$(usrlibdir))
 else
 	ln -sf $(libdir)/$(SHLIB) $(DESTDIR)$(usrlibdir)/libvolume_id.so
 endif
+	$(INSTALL_LIB) -D libvolume_id.a $(DESTDIR)$(usrlibdir)/libvolume_id.a
 	$(INSTALL) -d $(DESTDIR)$(usrlibdir)/pkgconfig
 	$(INSTALL_DATA) libvolume_id.pc $(DESTDIR)$(usrlibdir)/pkgconfig/libvolume_id.pc
 .PHONY: install
diff -up udev-120/extras/volume_id/Makefile.lib64 udev-120/extras/volume_id/Makefile
--- udev-120/extras/volume_id/Makefile.lib64	2008-04-03 21:12:53.000000000 +0200
+++ udev-120/extras/volume_id/Makefile	2008-04-04 14:13:33.000000000 +0200
@@ -17,6 +17,9 @@ usrsbindir =	${prefix}/usr/sbin
 libudevdir =	${prefix}/lib/udev
 mandir =	${prefix}/usr/share/man
 configdir =	${etcdir}/udev/
+libdir ?=       ${prefix}/lib
+usrlibdir ?=    ${prefix}/usr/lib
+
 
 INSTALL = install -c
 INSTALL_PROGRAM = ${INSTALL}
@@ -34,7 +37,11 @@ all: lib $(PROG) $(MAN_PAGES)
 	$(Q) $(CC) -c $(CFLAGS) $< -o $@
 
 lib:
-	$(Q) $(MAKE) -C $@
+	$(Q) $(MAKE) \
+		libdir=$(libdir) \
+		usrlibdir=$(usrlibdir) \
+		-C lib
+#	$(Q) $(MAKE) -C $@
 .PHONY: lib
 
 lib/libvolume_id.a: lib
diff -up udev-120/Makefile.lib64 udev-120/Makefile
--- udev-120/Makefile.lib64	2008-04-03 21:12:53.000000000 +0200
+++ udev-120/Makefile	2008-04-04 14:13:33.000000000 +0200
@@ -91,6 +91,8 @@ usrbindir =	${prefix}/usr/bin
 libudevdir =	${prefix}/lib/udev
 mandir =	${prefix}/usr/share/man
 configdir =	${etcdir}/udev
+libdir ?= 	${prefix}/lib
+usrlibdir ?= 	${prefix}/usr/lib
 udevdir =	/dev
 DESTDIR =
 
@@ -161,6 +163,8 @@ all: $(PROGRAMS) $(MAN_PAGES)
 			RANLIB="$(RANLIB)" \
 			LIB_OBJS="$(LIB_OBJS)" \
 			LIBUDEV="$(PWD)/$(LIBUDEV)" \
+			libdir=$(libdir) \
+			usrlibdir=$(usrlibdir) \
 			-C $$target $@ || exit 1; \
 	done;
 .PHONY: all

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/x-pkcs7-signature, Size: 3636 bytes --]

  reply	other threads:[~2008-07-08  6:53 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-07 22:12 [PATCH 2/2] Fixed installation path for 64-bit libvolume_id.so Thomas Koeller
2008-07-08  6:53 ` Harald Hoyer [this message]
2008-07-09 19:51 ` Thomas Koeller
2008-07-09 21:26 ` Kay Sievers
2008-07-09 21:35 ` Thomas Koeller
2008-07-09 21:44 ` Kay Sievers
2008-07-09 22:03 ` Dan Nicholson
2008-07-09 22:41 ` Thomas Koeller
2008-07-09 22:45 ` Kay Sievers
2008-07-09 23:04 ` Thomas Koeller
2008-07-22 13:59 ` Kay Sievers

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=48730EFF.6040309@redhat.com \
    --to=harald@redhat.com \
    --cc=linux-hotplug@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.