Merge lp:~3v1n0/bamf/soname-up into lp:bamf/0.4

Proposed by Marco Trevisan (Treviño) on 2012-12-19
Status: Merged
Approved by: Michael Terry on 2012-12-19
Approved revision: 517
Merged at revision: 513
Proposed branch: lp:~3v1n0/bamf/soname-up
Merge into: lp:bamf/0.4
Diff against target: 200 lines (+19/-41)
9 files modified
.bzrignore (+6/-13)
configure.ac (+1/-2)
debian/changelog (+3/-0)
debian/control (+5/-7)
debian/libbamf3-1.symbols (+1/-1)
lib/libbamf/Makefile.am (+1/-0)
lib/libbamf/libbamf.pc.in (+0/-13)
lib/libbamf/libbamf3.pc.in (+2/-2)
tests/libbamf/Makefile.am (+0/-3)
To merge this branch: bzr merge lp:~3v1n0/bamf/soname-up
Reviewer Review Type Date Requested Status
Michael Terry 2012-12-19 Approve on 2012-12-19
PS Jenkins bot continuous-integration Approve on 2012-12-19
Review via email: mp+140695@code.launchpad.net

Commit Message

libbamf: bump the SONAME value and remove the unneeded libwnck dependencies

Description of the Change

BUMP the SONAME version and remove some unneeded libwnck dependencies of the library.

To post a comment you must log in.
Michael Terry (mterry) wrote :

Are you bumping from 0 to 4? The SONAME is currently 0 I thought. Also, you'll need to update the packaging a bit. Change the package name from libbamf3-0 to libbamf3-1 etc.

lp:~3v1n0/bamf/soname-up updated on 2012-12-19
513. By Marco Trevisan (Treviño) on 2012-12-19

libbamf: use 1.0.0 as SONAME

514. By Marco Trevisan (Treviño) on 2012-12-19

debian: remove the unneded dependencies, rename libbamf3-0.* files

515. By Marco Trevisan (Treviño) on 2012-12-19

debian/control: put back the trailing comma

Michael Terry (mterry) wrote :

One last thing, sorry!

I see that you edited lib/libbamf/libbamf.pc.in, but that can actually be removed entirely. Instead edit libbamf3.pc.in.

Looks good after that.

lp:~3v1n0/bamf/soname-up updated on 2012-12-19
516. By Marco Trevisan (Treviño) on 2012-12-19

libbamf: remove the libbamf.pc.in file

517. By Marco Trevisan (Treviño) on 2012-12-19

Updating .bzrignore

Marco Trevisan (Treviño) (3v1n0) wrote :

> One last thing, sorry!
>
> I see that you edited lib/libbamf/libbamf.pc.in, but that can actually be
> removed entirely. Instead edit libbamf3.pc.in.
>
> Looks good after that.

Right. Done.

Michael Terry (mterry) wrote :

Awesome, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file '.bzrignore'
2--- .bzrignore 2012-12-18 01:53:22 +0000
3+++ .bzrignore 2012-12-19 18:29:24 +0000
4@@ -7,6 +7,7 @@
5 compile
6 config.guess
7 config.h
8+config.h.in
9 config.log
10 config.status
11 config.sub
12@@ -24,11 +25,6 @@
13 examples/Makefile.in
14 examples/alt-tabber
15 examples/desktop-file-launcher
16-lib/libwncksync/Makefile
17-lib/libwncksync/Makefile.in
18-lib/libwncksync/libwncksync.la
19-lib/libwncksync/libwncksync.pc
20-lib/libwncksync/libwncksync_la-libwncksync.lo
21 module/Makefile
22 module/Makefile.in
23 module/libgiodbus.la
24@@ -36,14 +32,8 @@
25 module/libgiodbus_la-gapplaunchhandlerdbus.lo
26 src/Makefile
27 src/Makefile.in
28-src/wncksyncdaemon
29-lib/libbamf/libbamf.la
30-lib/libbamf/libbamf.pc
31-lib/libbamf/libbamf_la-bamf-application.lo
32-lib/libbamf/libbamf_la-bamf-matcher.lo
33-lib/libbamf/libbamf_la-bamf-proxy.lo
34-lib/libbamf/libbamf_la-bamf-view.lo
35-src/bamfdbus-glue.h
36+src/*-generated.c
37+src/*-generated.h
38 *.lo
39 gtk-doc.make
40 bamf-0.2.34.tar.gz
41@@ -77,6 +67,9 @@
42 lib/libbamf/bamf-marshal.h
43 lib/libbamf/bamf-tab-source-glue.h
44 lib/libbamf/stamp-marshal
45+lib/libbamf/libbamf3.la
46+lib/libbamf/libbamf3.pc
47+lib/libbamf/libbamf3.vapi
48 m4/gtk-doc.m4
49 module/libgiobamf.la
50 src/bamf-marshal.h
51
52=== modified file 'configure.ac'
53--- configure.ac 2012-12-13 21:54:59 +0000
54+++ configure.ac 2012-12-19 18:29:24 +0000
55@@ -151,14 +151,13 @@
56 AC_SUBST(BAMFDAEMON_LIBS)
57
58 origgiomodulesdir=`pkg-config --variable=giomoduledir gio-2.0`
59-# Make giomodulesdir honour 'prefix', so that distcheck works.
60+# Make giomodulesdir honour 'prefix', so that distcheck works.
61 giomodulesdir=`echo "$origgiomodulesdir" | sed 's|/usr|${prefix}|'`
62 AC_SUBST(giomodulesdir)
63
64 AC_OUTPUT([
65 Makefile
66 lib/libbamf/Makefile
67-lib/libbamf/libbamf.pc
68 lib/libbamf/libbamf3.pc
69 src/Makefile
70 tests/functional/Makefile
71
72=== modified file 'debian/changelog'
73--- debian/changelog 2012-12-18 01:53:22 +0000
74+++ debian/changelog 2012-12-19 18:29:24 +0000
75@@ -6,6 +6,9 @@
76 [ Marco Trevisan (Treviño) ]
77 * Remove the unneeded libbamf0.symbols and libbamf0.install
78 * Update libbamf3.symbols removing the indicator functions
79+ * Remove the unneeded library dependencies on libwnck
80+ * Bump the SONAME
81+ * Update the package version to libbamf3-1
82
83 -- Didier Roche <didrocks@ubuntu.com> Wed, 05 Dec 2012 17:34:41 +0100
84
85
86=== modified file 'debian/control'
87--- debian/control 2012-12-05 16:35:24 +0000
88+++ debian/control 2012-12-19 18:29:24 +0000
89@@ -39,7 +39,7 @@
90 module that facilitates the matching of applications started
91 through GDesktopAppInfo
92
93-Package: libbamf3-0
94+Package: libbamf3-1
95 Architecture: any
96 Depends: ${shlibs:Depends},
97 ${misc:Depends},
98@@ -55,11 +55,9 @@
99 Package: libbamf3-dev
100 Section: libdevel
101 Architecture: any
102-Depends: ${shlibs:Depends},
103- ${misc:Depends},
104- libbamf3-0 (= ${binary:Version}),
105- libwnck-3-dev,
106- libglib2.0-dev (>= 2.23.0-1ubuntu3~),
107+Depends: ${misc:Depends},
108+ libbamf3-1 (= ${binary:Version}),
109+ libglib2.0-dev (>= 2.30.0),
110 Suggests: libbamf-doc
111 Description: Window matching library - development files
112 bamf matches application windows to desktop files
113@@ -86,7 +84,7 @@
114 Priority: extra
115 Depends: ${shlibs:Depends},
116 ${misc:Depends},
117- libbamf3-0 (= ${binary:Version}),
118+ libbamf3-1 (= ${binary:Version}),
119 Description: Window matching library - debugging symbols
120 bamf matches application windows to desktop files
121 .
122
123=== renamed file 'debian/libbamf3-0.install' => 'debian/libbamf3-1.install'
124=== renamed file 'debian/libbamf3-0.symbols' => 'debian/libbamf3-1.symbols'
125--- debian/libbamf3-0.symbols 2012-12-18 01:53:22 +0000
126+++ debian/libbamf3-1.symbols 2012-12-19 18:29:24 +0000
127@@ -1,4 +1,4 @@
128-libbamf3.so.0 libbamf3-0 #MINVER#
129+libbamf3.so.1 libbamf3-1 #MINVER#
130 bamf_application_get_application_menu@Base 0.3.0
131 bamf_application_get_application_type@Base 0.2.20
132 bamf_application_get_desktop_file@Base 0.2.20
133
134=== modified file 'lib/libbamf/Makefile.am'
135--- lib/libbamf/Makefile.am 2012-12-18 01:53:22 +0000
136+++ lib/libbamf/Makefile.am 2012-12-19 18:29:24 +0000
137@@ -71,6 +71,7 @@
138 libbamf3_la_LDFLAGS = \
139 -shared \
140 -export-symbols-regex "^bamf_" \
141+ -version-info 1:0:0 \
142 $(COVERAGE_LDFLAGS) \
143 $(NULL)
144
145
146=== removed file 'lib/libbamf/libbamf.pc.in'
147--- lib/libbamf/libbamf.pc.in 2010-09-14 13:07:10 +0000
148+++ lib/libbamf/libbamf.pc.in 1970-01-01 00:00:00 +0000
149@@ -1,13 +0,0 @@
150-prefix=@prefix@
151-exec_prefix=@exec_prefix@
152-libdir=@libdir@
153-includedir=@includedir@
154-
155-Name: libbamf
156-Description: C Convenience library for bamf d-bus server.
157-Version: @VERSION@
158-
159-Requires: glib-2.0 libwnck-1.0
160-Libs: -L${libdir} -lbamf
161-Cflags: -I${includedir}/libbamf -I${libdir}/libbamf/include -DWNCK_I_KNOW_THIS_IS_UNSTABLE
162-
163
164=== modified file 'lib/libbamf/libbamf3.pc.in'
165--- lib/libbamf/libbamf3.pc.in 2011-05-31 14:28:05 +0000
166+++ lib/libbamf/libbamf3.pc.in 2012-12-19 18:29:24 +0000
167@@ -7,7 +7,7 @@
168 Description: C Convenience library for bamf d-bus server.
169 Version: @VERSION@
170
171-Requires: glib-2.0 libwnck-3.0
172+Requires: glib-2.0
173 Libs: -L${libdir} -lbamf3
174-Cflags: -I${includedir}/libbamf3 -I${libdir}/libbamf3/include -DWNCK_I_KNOW_THIS_IS_UNSTABLE
175+Cflags: -I${includedir}/libbamf3 -I${libdir}/libbamf3/include
176
177
178=== modified file 'tests/libbamf/Makefile.am'
179--- tests/libbamf/Makefile.am 2012-11-19 18:43:50 +0000
180+++ tests/libbamf/Makefile.am 2012-12-19 18:29:24 +0000
181@@ -11,11 +11,9 @@
182 -I$(top_srcdir)/src \
183 -I$(top_srcdir)/lib/libbamf \
184 -I$(top_builddir)/lib/libbamf \
185- -DWNCK_I_KNOW_THIS_IS_UNSTABLE \
186 -DTESTDIR=\""$(abs_top_srcdir)/tests/libbamf"\" \
187 $(GCC_FLAGS) \
188 $(GTK_CFLAGS) \
189- $(WNCK_CFLAGS) \
190 $(LIBWSBIND_CFLAGS) \
191 $(DBUS_CFLAGS) \
192 $(GLIB_CFLAGS)
193@@ -23,7 +21,6 @@
194 test_libbamf_LDADD = \
195 $(top_builddir)/lib/libbamf/libbamf$(LIBBAMF_VER).la \
196 $(LIBWSBIND_LIBS) \
197- $(WNCK_LIBS) \
198 $(GLIB_LIBS) \
199 $(GTK_LIBS) \
200 $(DBUS_LIBS)

Subscribers

People subscribed via source and target branches