Bug#474087: libopenipmi-dev: pkg-config --libs OpenIPMIpthread fails

2009-11-29 Thread Sebastian Harl
Hi,

On Tue, Nov 17, 2009 at 02:51:41AM +, Simon McVittie wrote:
 I've prepared a slightly more complete patch, fixing some other issues
 including a lintian error.

Thanks for the patch! It looks fine to me (even though, I'd have used
dpatch or quilt to manage the patches against the upstream sources).

 I don't plan to NMU this since I can't meaningfully test it (I don't
 have the relevant hardware!)

… however, the same applies to me. I do not use any IPMI stuff myself
but use libopenipmi as a build-dependency only.

Since the Debian maintainer did not react at all within more than 1.5
years, an NMU would be very appreciated. If you're using IPMI and are
able to test and upload the package, please go ahead.

Cheers,
Sebastian

-- 
Sebastian tokkee Harl +++ GnuPG-ID: 0x8501C7FC +++ http://tokkee.org/

Those who would give up Essential Liberty to purchase a little Temporary
Safety, deserve neither Liberty nor Safety. -- Benjamin Franklin



signature.asc
Description: Digital signature


Bug#474087: libopenipmi-dev: pkg-config --libs OpenIPMIpthread fails

2009-11-29 Thread Simon McVittie
On Sun, 29 Nov 2009 at 21:53:30 +0100, Sebastian Harl wrote:
 On Tue, Nov 17, 2009 at 02:51:41AM +, Simon McVittie wrote:
  I've prepared a slightly more complete patch, fixing some other issues
  including a lintian error.
 
 Thanks for the patch! It looks fine to me (even though, I'd have used
 dpatch or quilt to manage the patches against the upstream sources).

I considered this to be a NMU waiting for happen, and I don't think it's
appropriate to change the patch system (away from none, in this case) in an
NMU. Opinions may vary, obviously.

I've started to provide a git repository on git.debian.org for all my NMUs,
which I think is an even better way to get nicely-separated patches to the
maintainer.

  I don't plan to NMU this since I can't meaningfully test it (I don't
  have the relevant hardware!)
 
 … however, the same applies to me. I do not use any IPMI stuff myself
 but use libopenipmi as a build-dependency only.
 
 Since the Debian maintainer did not react at all within more than 1.5
 years, an NMU would be very appreciated. If you're using IPMI and are
 able to test and upload the package, please go ahead.

Since the changes we made are build-system-only, and I verified that they let
collectd compile against openipmi even after collectd's workaround has been
removed, I'm getting quite tempted to just upload it anyway...

S


signature.asc
Description: Digital signature


Bug#474087: libopenipmi-dev: pkg-config --libs OpenIPMIpthread fails

2009-11-16 Thread Simon McVittie
tags 474087 + patch
thanks

I've prepared a slightly more complete patch, fixing some other issues
including a lintian error. I don't plan to NMU this since I can't meaningfully
test it (I don't have the relevant hardware!) but I can confirm that collectd
compiles against this version, even when the hack it currently uses to avoid
this bug has been removed.

Temporary git repository:
http://git.debian.org/?p=users/smcv/nmu/openipmi.git

Regards,
Simon
diffstat for openipmi_2.0.16-1 openipmi_2.0.16-1.1

 OpenIPMIpthread.pc.in|6 +++---
 OpenIPMIui.pc.in |2 +-
 openipmi-2.0.16/debian/changelog |   13 +
 openipmi-2.0.16/debian/control   |3 ++-
 openipmi-2.0.16/debian/rules |   10 +-
 5 files changed, 28 insertions(+), 6 deletions(-)

diff -u openipmi-2.0.16/debian/changelog openipmi-2.0.16/debian/changelog
--- openipmi-2.0.16/debian/changelog
+++ openipmi-2.0.16/debian/changelog
@@ -1,3 +1,16 @@
+openipmi (2.0.16-1.1) UNRELEASED; urgency=low
+
+  * Non-maintainer upload.
+  * Fix pkg-config files: neither pthread.pc nor ncurses.pc exists in Debian
+(Closes: #474087)
+  * Use chrpath to remove unnecessary rpath from libOpenIPMIposix.so.0.0.1
+  * Remove a generated Makefile during clean to avoid it getting into the
+Debian diff after repeated builds (as seen in the Ubuntu diff)
+  * Actually compile the package in build-arch rather than waiting until
+install-arch
+
+ -- Simon McVittie s...@debian.org  Tue, 17 Nov 2009 01:31:30 +
+
 openipmi (2.0.16-1) unstable; urgency=low
 
   * new upstream release from 2009-03-16
diff -u openipmi-2.0.16/debian/rules openipmi-2.0.16/debian/rules
--- openipmi-2.0.16/debian/rules
+++ openipmi-2.0.16/debian/rules
@@ -39,7 +39,7 @@
 build-arch-stamp:  config.status
 
 	# Add here commands to compile the arch part of the package.
-	#$(MAKE) 
+	$(MAKE) 
 	touch build-arch-stamp
 
 clean:
@@ -49,6 +49,9 @@
 
 	# Add here commands to clean up after the build process.
 	[ ! -f Makefile ] || $(MAKE) distclean
+	# generated file not removed by upstream Makefile
+	rm swig/python/openipmigui/Makefile
+
 ifneq $(wildcard /usr/share/misc/config.sub) 
 	cp -f /usr/share/misc/config.sub config.sub
 endif
@@ -71,6 +74,11 @@
 	# debian/tmp.
 	$(MAKE) install DESTDIR=$(CURDIR)/debian/tmp
 
+	# remove useless rpath from libOpenIPMIposix (which is built with
+	# -R/usr/lib, whereas only -rpath=/usr/lib is disabled by
+	# --disable-rpath)
+	chrpath -d debian/tmp/usr/lib/libOpenIPMIposix.so.*.*
+
 	dh_movefiles
 
 	dh_install -s
diff -u openipmi-2.0.16/debian/control openipmi-2.0.16/debian/control
--- openipmi-2.0.16/debian/control
+++ openipmi-2.0.16/debian/control
@@ -2,7 +2,8 @@
 Section: admin
 Priority: optional
 Maintainer: Noèl Köthe n...@debian.org
-Build-Depends: debhelper ( 5.0.0), libsnmp-dev, libpopt-dev, libncurses5-dev
+Build-Depends: debhelper ( 5.0.0), libsnmp-dev, libpopt-dev, libncurses5-dev,
+   chrpath
 Standards-Version: 3.8.2
 Homepage: http://openipmi.sourceforge.net/
 
only in patch2:
unchanged:
--- openipmi-2.0.16.orig/OpenIPMIui.pc.in
+++ openipmi-2.0.16/OpenIPMIui.pc.in
@@ -6,6 +6,6 @@
 Name: OpenIPMIui
 Description: UI front-end for OpenIPMI
 Version: @VERSION@
-Requires: OpenIPMI ncurses
+Requires: OpenIPMI
 Libs: -L${libdir} -lOpenIPMIutils -lOpenIPMIui -lOpenIPMI -lncurses
 Cflags: -I${includedir}
only in patch2:
unchanged:
--- openipmi-2.0.16.orig/OpenIPMIpthread.pc.in
+++ openipmi-2.0.16/OpenIPMIpthread.pc.in
@@ -6,6 +6,6 @@
 Name: OpenIPMIpthread
 Description: Pthread OS handler for OpenIPMI
 Version: @VERSION@
-Requires: OpenIPMI pthread
-Libs: -L${libdir} -lOpenIPMIutils -lOpenIPMIpthread
-Cflags: -I${includedir}
+Requires: OpenIPMI
+Libs: -L${libdir} -lOpenIPMIutils -lOpenIPMIpthread -pthread
+Cflags: -I${includedir} -pthread


signature.asc
Description: Digital signature


Bug#474087: libopenipmi-dev: pkg-config --libs OpenIPMIpthread fails.

2009-10-03 Thread Luk Claes
Hi Noèl

Any progress in uploading a fixed package?

If you're not interested in this package anymore, please let someone
adopt it, TIA.

Cheers

Luk



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#474087: libopenipmi-dev: pkg-config --libs OpenIPMIpthread fails.

2009-08-29 Thread Sebastian Harl
severity 474087 grave
thanks

Hi,

On Mon, Aug 11, 2008 at 04:06:01PM +0200, Sebastian Harl wrote:
 On Thu, Apr 03, 2008 at 10:09:08AM +0200, Florian Forster wrote:
  in order to determine if libopenipmi is installed, my software calls
   $ pkg-config --libs OpenIPMIpthread.
  This fails with the following error:
  -- 8 --
   Package pthread was not found in the pkg-config search path.
   Perhaps you should add the directory containing `pthread.pc'
   to the PKG_CONFIG_PATH environment variable
   Package 'pthread', required by 'OpenIPMIpthread', not found
  -- 8 --
  
  The problem isn't new: Both, version 2.0.7-1 (Etch) and version 2.0.13-1
  (Lenny, Sid), have this problem.
 
 This is fixed in version 2.0.14-1 even though neither the upstream nor
 the Debian changelog mentions an appropriate fix at all.

In the same, undocumented way it re-appeared in version 2.0.16-1.

OpenIPMIpthread.pc in version 2.0.14-1 includes the following:

  Requires: OpenIPMI
  […]
  Cflags: -I${includedir} -pthread

That's the way it's supposed to be (on Debian).

In version 2.0.16-1 this is (again):

  Requires: OpenIPMI pthread
  […]
  Cflags: -I${includedir}

Which does not work since there is no pthread.pc available in Debian.

 However, I'm not closing this bug as the same problem exists for
 ncurses.pc which is required by OpenIPMIui.pc but does not exist in
 Debian (according to apt-file).

This is still valid as well.

I've raised the severity since this issue makes the .pc files unusable
and breaks unrelated software (e.g. collectd FTBFS because of that).

This is trivial to fix. I'd really appreciate if you could take care of
it or tell us in case we're missing something.

Cheers,
Sebastian

-- 
Sebastian tokkee Harl +++ GnuPG-ID: 0x8501C7FC +++ http://tokkee.org/

Those who would give up Essential Liberty to purchase a little Temporary
Safety, deserve neither Liberty nor Safety. -- Benjamin Franklin



signature.asc
Description: Digital signature


Bug#474087: libopenipmi-dev: pkg-config --libs OpenIPMIpthread fails.

2008-08-11 Thread Sebastian Harl
Hi,

On Thu, Apr 03, 2008 at 10:09:08AM +0200, Florian Forster wrote:
 in order to determine if libopenipmi is installed, my software calls
  $ pkg-config --libs OpenIPMIpthread.
 This fails with the following error:
 -- 8 --
  Package pthread was not found in the pkg-config search path.
  Perhaps you should add the directory containing `pthread.pc'
  to the PKG_CONFIG_PATH environment variable
  Package 'pthread', required by 'OpenIPMIpthread', not found
 -- 8 --
 
 The problem isn't new: Both, version 2.0.7-1 (Etch) and version 2.0.13-1
 (Lenny, Sid), have this problem.

This is fixed in version 2.0.14-1 even though neither the upstream nor
the Debian changelog mentions an appropriate fix at all. Anyway, the
Requires field of OpenIPMIpthread.pc now lists OpenIPMI only while
-pthread has been added to the Cflags field.

However, I'm not closing this bug as the same problem exists for
ncurses.pc which is required by OpenIPMIui.pc but does not exist in
Debian (according to apt-file).

Cheers,
Sebastian

-- 
Sebastian tokkee Harl +++ GnuPG-ID: 0x8501C7FC +++ http://tokkee.org/

Those who would give up Essential Liberty to purchase a little Temporary
Safety, deserve neither Liberty nor Safety. -- Benjamin Franklin



signature.asc
Description: Digital signature


Bug#474087: libopenipmi-dev: pkg-config --libs OpenIPMIpthread fails.

2008-04-03 Thread Florian Forster
Package: libopenipmi-dev
Version: 2.0.7-1
Severity: important

Hi,

in order to determine if libopenipmi is installed, my software calls
 $ pkg-config --libs OpenIPMIpthread.
This fails with the following error:
-- 8 --
 Package pthread was not found in the pkg-config search path.
 Perhaps you should add the directory containing `pthread.pc'
 to the PKG_CONFIG_PATH environment variable
 Package 'pthread', required by 'OpenIPMIpthread', not found
-- 8 --

The problem isn't new: Both, version 2.0.7-1 (Etch) and version 2.0.13-1
(Lenny, Sid), have this problem.

According to `apt-file' `/usr/lib/pkgconfig/pthread.pc' is not provided
by any package.

I propose to remove `pthread' from the `Requires:' line of
`/usr/lib/pkgconfig/OpenIPMIpthread.pc' and add `-pthread' to the
`Cflags:' line instead.

I've tagged this bug `important' because
 $ pkg-config --exists OpenIPMIpthread
exits with a non-zero exit status, telling the calling process that the
library isn't installed. This renders the pthread OS handler unusable by
software that uses pkg-config to check if the library is installed. An
alternative *-config script doesn't exist.

Regards,
-octo

-- System Information:
Debian Release: lenny/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'unstable')
Architecture: i386 (i686)

Kernel: Linux 2.6.22-3-k7 (SMP w/1 CPU core)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8) (ignored: LC_ALL 
set to en_US.UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages libopenipmi-dev depends on:
ii  libgdbm-dev   1.8.3-3GNU dbm database routines (develop
ii  libopenipmi0  2.0.13-1   Intelligent Platform Management In

libopenipmi-dev recommends no packages.

-- no debconf information



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]