Merge lp:~dpshorten/ubuntu/oneiric/libpam-encfs/ftbfs-jam into lp:ubuntu/oneiric/libpam-encfs

Proposed by David Shorten
Status: Merged
Merged at revision: 9
Proposed branch: lp:~dpshorten/ubuntu/oneiric/libpam-encfs/ftbfs-jam
Merge into: lp:ubuntu/oneiric/libpam-encfs
Diff against target: 43 lines (+11/-3)
2 files modified
debian/changelog (+7/-0)
debian/rules (+4/-3)
To merge this branch: bzr merge lp:~dpshorten/ubuntu/oneiric/libpam-encfs/ftbfs-jam
Reviewer Review Type Date Requested Status
Stefano Rivera Pending
Review via email: mp+67448@code.launchpad.net
To post a comment you must log in.
10. By David Shorten

removed -lc from DEBIAN_LD_FLAGS

11. By David Shorten

added in a line of commenting to debian/rules

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2011-04-12 13:06:10 +0000
+++ debian/changelog 2011-07-10 11:06:35 +0000
@@ -1,3 +1,10 @@
1libpam-encfs (0.1.4.4-3ubuntu1) oneiric; urgency=low
2
3 * Changed debian/rules so that the package built with the
4 ld option --as-needed enabled. (LP: #803183)
5
6 -- David Shorten <dpshorten@gmail.com> Sun, 10 Jul 2011 12:16:37 +0200
7
1libpam-encfs (0.1.4.4-3) unstable; urgency=low8libpam-encfs (0.1.4.4-3) unstable; urgency=low
29
3 * debian/NEWS,README.Debian: Document possible behavior10 * debian/NEWS,README.Debian: Document possible behavior
411
=== modified file 'debian/rules'
--- debian/rules 2011-02-28 17:34:22 +0000
+++ debian/rules 2011-07-10 11:06:35 +0000
@@ -14,12 +14,13 @@
14PAM_CONFIG_DIR = $(CURDIR)/debian/libpam-encfs/usr/share/pam-configs14PAM_CONFIG_DIR = $(CURDIR)/debian/libpam-encfs/usr/share/pam-configs
15PAM_SEC_DIR = $(CURDIR)/debian/libpam-encfs/etc/security15PAM_SEC_DIR = $(CURDIR)/debian/libpam-encfs/etc/security
1616
17# Explicitly link to C library (we use ld)
18DEBIAN_PAMLIB = -lpam -lc
19
17# Original upstream value (Check!)20# Original upstream value (Check!)
18DEBIAN_LD_FLAGS = --shared21DEBIAN_LD_FLAGS = --shared
19# Delete all local symbols.22# Delete all local symbols.
20DEBIAN_LD_FLAGS += -x23DEBIAN_LD_FLAGS += -x
21# Explicitly link to C library (we use ld)
22DEBIAN_LD_FLAGS += -lc
23# Check binutils-gold like behavior24# Check binutils-gold like behavior
24# DEBIAN_LD_FLAGS += --no-as-needed25# DEBIAN_LD_FLAGS += --no-as-needed
2526
@@ -47,7 +48,7 @@
47 dh_testdir48 dh_testdir
4849
49 # Add here commands to compile the package.50 # Add here commands to compile the package.
50 $(MAKE) LDFLAGS="$(DEBIAN_LD_FLAGS)"51 $(MAKE) LDFLAGS="$(DEBIAN_LD_FLAGS)" PAMLIB="$(DEBIAN_PAMLIB)"
51 #docbook-to-man debian/pam-encfs.sgml > pam-encfs.152 #docbook-to-man debian/pam-encfs.sgml > pam-encfs.1
5253
53 cp -i debian/Upstream.Changelog changelog54 cp -i debian/Upstream.Changelog changelog

Subscribers

People subscribed via source and target branches

to all changes: