Bug#782575: Please add Build-Depends-Package for the move to /lib

2015-04-15 Thread intrigeri
Hi Michael,

Michael Biebl wrote (14 Apr 2015 11:41:29 GMT) :
 I therefor suggest, that libapparmor1 adds
 * Build-Depends-Package: libapparmor-dev
 to its symbols file.

 This way, we can simply make the Build-Depends on libapparmor-dev
 versioned, indicating that we want a version which was moved to lib, and
 get a proper versioned library dependency.

Good catch, thanks! I didn't know about this symbols file feature.

 Please consider applying the attached patch and making a new
 experimental upload for libapparmor, so we can drop the hard-coded
 library dependency.

Sure, I'm always happy to help you folks when it comes to supporting
AppArmor in systemd. Applied locally, building. Will test and upload
to experimental shortly.

 While looking at the symbols file, I also noticed, that it contains a
 lot of versions including the debian revision 0ubuntu1.
 Is that really intentional?

I believe it may be helpful for collaborating on this package's
maintenance with the Ubuntu folks working on AppArmor, by reducing our
delta. But perhaps I'm missing something: to be honest I'm pretty much
a newbie wrt. shared libraries packaging.

Cheers,
-- 
intrigeri


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



Bug#782575: Please add Build-Depends-Package for the move to /lib

2015-04-14 Thread Michael Biebl
Package: libapparmor1
Version: 2.9.0-3+exp1
Severity: important
Tags: patch

Hi,

in version 2.9.0-3+exp1, the libraries were moved to /lib and Martin
(re)enabled apparmor support in systemd. While doing that, he also added
a hard-coded, versioned dependency on libapparmor1 (= 2.9.0-3+exp1) to
ensure that systemd depends on a libapparmor1 version which has its
libraries in /lib.

This is super-ugly, and we should not do that.

I therefor suggest, that libapparmor1 adds
* Build-Depends-Package: libapparmor-dev
to its symbols file.

This way, we can simply make the Build-Depends on libapparmor-dev
versioned, indicating that we want a version which was moved to lib, and
get a proper versioned library dependency.

Please consider applying the attached patch and making a new
experimental upload for libapparmor, so we can drop the hard-coded
library dependency.

While looking at the symbols file, I also noticed, that it contains a
lot of versions including the debian revision 0ubuntu1.
Is that really intentional?

Regards,
Michael


-- System Information:
Debian Release: 8.0
  APT prefers unstable
  APT policy: (500, 'unstable'), (200, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.16.0-4-amd64 (SMP w/4 CPU cores)
Locale: LANG=de_DE.utf8, LC_CTYPE=de_DE.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages libapparmor1 depends on:
ii  libc6  2.19-17

libapparmor1 recommends no packages.

libapparmor1 suggests no packages.

-- no debconf information
diff --git a/debian/libapparmor1.symbols b/debian/libapparmor1.symbols
index f49640d..d0185ee 100644
--- a/debian/libapparmor1.symbols
+++ b/debian/libapparmor1.symbols
@@ -1,4 +1,5 @@
 libapparmor.so.1 libapparmor1 #MINVER#
+* Build-Depends-Package: libapparmor-dev
  APPARMOR_1.0@APPARMOR_1.0 2.6~devel
  APPARMOR_1.1@APPARMOR_1.1 2.6~devel
  APPARMOR_2.9@APPARMOR_2.9 2.8.94-0ubuntu1


Bug#782575: Please add Build-Depends-Package for the move to /lib

2015-04-14 Thread Michael Biebl
Am 14.04.2015 um 13:41 schrieb Michael Biebl:
 I therefor suggest, that libapparmor1 adds
 * Build-Depends-Package: libapparmor-dev
 to its symbols file.
 
 This way, we can simply make the Build-Depends on libapparmor-dev
 versioned, indicating that we want a version which was moved to lib, and
 get a proper versioned library dependency.

See debian policy 8.6.3.2 and man 5 deb-symbols

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?



signature.asc
Description: OpenPGP digital signature