Merge ~sergiodj/ubuntu/+source/openipmi:openipmi-merge-2.0.29-0.1 into ubuntu/+source/openipmi:debian/sid
- Git
- lp:~sergiodj/ubuntu/+source/openipmi
- openipmi-merge-2.0.29-0.1
- Merge into debian/sid
Status: | Merged |
---|---|
Merge reported by: | Christian Ehrhardt |
Merged at revision: | 17654a2eb8251e990b14ebefa97022d04bafe6dd |
Proposed branch: | ~sergiodj/ubuntu/+source/openipmi:openipmi-merge-2.0.29-0.1 |
Merge into: | ubuntu/+source/openipmi:debian/sid |
Diff against target: |
404 lines (+302/-1) 4 files modified
debian/changelog (+277/-0) debian/control (+2/-1) debian/patches/fix_pthread_pkg_config.patch (+22/-0) debian/patches/series (+1/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Christian Ehrhardt (community) | Approve | ||
Canonical Server Core Reviewers | Pending | ||
Canonical Server | Pending | ||
Review via email: mp+388411@code.launchpad.net |
Commit message
Description of the change
This is the merge for openipmi 2.0.29-0.1.
The Debian maintainer has been MIA for more than a year now, so I put on my DD hat and did the update from the Debian side first. I took the opportunity and incorporated the various patches that we've been carrying on Ubuntu to improve the sysv init file, and to adjust d/control et al in order to compile the newest version from upstream. That's why you will see the dropped changes there.
As for the single change I decided to keep, debian/
The package doesn't carry dep8 tests, so I installed it on a groovy container. I also wanted to test the "-pthread" thing above, so I used sources.d.o to find a package that uses openipmi's pkg-config file to determine CFLAGS/LDFLAGS, found that collectd is one of them, then I compiled and tested it inside the same VM using the new openipmi package. As I said, the compilation works even without providing "-pthread" on Libs, but I decided to keep it anyway because it's more correct this way.
Sergio Durigan Junior (sergiodj) wrote : | # |
Christian Ehrhardt (paelzer) wrote : | # |
Queued to my ever growing monday list of TODOs :-)
Christian Ehrhardt (paelzer) wrote : | # |
LGTM you were able to drop most of the things since they are upstream and/or in Debian by now which is a great result of our delta submission \o/.
The one remaining delta is from before me touching the package, I didn't look too much at it last time - but now that it is the only thing left we maybe should look more deeply.
It is reported to Debian as already as https:/
It is reported as Fixed there in version openipmi/2.0.16-1.1
So might this be able to become a sync now?
The Ubuntu fix says the fix is "Forward-ported from 2.10.16-1.1, as it seems to have been dropped in Debian".
Indeed at upstream it was fixed (late) see https:/
This upstream fix is in since 2.0.22-1 and the reason it was dropped from Debian.
And indeed this LGTM now for cflags:
root@d10-sid:~# pkg-config --libs OpenIPMIpthread
-lOpenIPMIutils -lOpenIPMIpthread -lOpenIPMIutils -lOpenIPMI
root@d10-sid:~# pkg-config --cflags OpenIPMIpthread
-pthread
The one reason against it is that you usually want it on cflags AND ldflags and not just one of them. But at least the upstream bug reporter was happy with the change as is.
So this should be:
a) not needed - good to become a sync now as it was the only delta.
OR
b) our remaining fix of adding it to "Libs:" as well is indeed important.
In that case almost +1 to this MP, but then please provide a follow on bug report
upstream and explain it there. Link tat bug report from our patch header then.
What do you think on (a) vs (b) @Sergio?
Sergio Durigan Junior (sergiodj) wrote : | # |
On Monday, August 03 2020, Christian Ehrhardt wrote:
> The one remaining delta is from before me touching the package, I didn't look too much at it last time - but now that it is the only thing left we maybe should look more deeply.
>
> It is reported to Debian as already as https:/
> It is reported as Fixed there in version openipmi/2.0.16-1.1
>
> So might this be able to become a sync now?
>
> The Ubuntu fix says the fix is "Forward-ported from 2.10.16-1.1, as it seems to have been dropped in Debian".
> Indeed at upstream it was fixed (late) see https:/
>
> This upstream fix is in since 2.0.22-1 and the reason it was dropped from Debian.
>
> And indeed this LGTM now for cflags:
> root@d10-sid:~# pkg-config --libs OpenIPMIpthread
> -lOpenIPMIutils -lOpenIPMIpthread -lOpenIPMIutils -lOpenIPMI
> root@d10-sid:~# pkg-config --cflags OpenIPMIpthread
> -pthread
>
>
> The one reason against it is that you usually want it on cflags AND ldflags and not just one of them. But at least the upstream bug reporter was happy with the change as is.
>
> So this should be:
> a) not needed - good to become a sync now as it was the only delta.
> OR
> b) our remaining fix of adding it to "Libs:" as well is indeed important.
> In that case almost +1 to this MP, but then please provide a follow on bug report
> upstream and explain it there. Link tat bug report from our patch header then.
>
> What do you think on (a) vs (b) @Sergio?
Thanks for the review, Christian!
As I wrote in the MP description, the correct thing to do here is for
"-pthread" to be passed both on CFLAGS and LDFLAGS. Therefore, the
right thing to do is to get in touch with upstream, provide a patch to
complement the existing fix there, and link it in our patch header.
I'll do that and force-push my updates later.
Cheers,
--
Sergio
GPG key ID: E92F D0B3 6B14 F1F4 D8E0 EB2F 106D A1C8 C3CB BF14
Christian Ehrhardt (paelzer) wrote : | # |
Ok - I must have read over the MP description too quickly, but I'm glad we came to the same conclusion :-)
I think these three places should be updated to properly reflect the real summary of the remaining purpose of the Delta. And if you submit something upstream as usual link that in the patches header.
Patch Header
+Description: fix pkg-config files: pthread.pc does not exist in Ubuntu
375 + Forward-ported from 2.10.16-1.1, as it seems to have been dropped in
376 + Debian.
Changelog:
Remaining changes:
9 + - debian/
10 + pthread.pc does not exist in Ubuntu. Thanks to Simon McVittie
11 + <email address hidden>. Closes #474087, LP #1311888.
Commit message:
- debian/
pthread.pc does not exist in Ubuntu. Thanks to Simon McVittie
<email address hidden>. Closes #474087, LP #1311888.
+1 under the condition that this is done
- e7fb7e6... by Sergio Durigan Junior
-
merge-changelogs
- 3d04011... by Sergio Durigan Junior
-
reconstruct-
changelog - 17654a2... by Sergio Durigan Junior
-
update-maintainer
Sergio Durigan Junior (sergiodj) wrote : | # |
On Wednesday, August 05 2020, Christian Ehrhardt wrote:
> Ok - I must have read over the MP description too quickly, but I'm glad we came to the same conclusion :-)
:-)
> I think these three places should be updated to properly reflect the real summary of the remaining purpose of the Delta. And if you submit something upstream as usual link that in the patches header.
>
> Patch Header
> +Description: fix pkg-config files: pthread.pc does not exist in Ubuntu
> 375 + Forward-ported from 2.10.16-1.1, as it seems to have been dropped in
> 376 + Debian.
>
> Changelog:
> Remaining changes:
> 9 + - debian/
> 10 + pthread.pc does not exist in Ubuntu. Thanks to Simon McVittie
> 11 + <email address hidden>. Closes #474087, LP #1311888.
>
> Commit message:
> - debian/
> pthread.pc does not exist in Ubuntu. Thanks to Simon McVittie
> <email address hidden>. Closes #474087, LP #1311888.
I adjusted the descriptions on these 3 places as requested.
> +1 under the condition that this is done
Thanks! If you can take a quick look to make sure you like the new
description, and then sponsor the package for me, I'd appreciate.
Cheers,
--
Sergio
GPG key ID: E92F D0B3 6B14 F1F4 D8E0 EB2F 106D A1C8 C3CB BF14
Christian Ehrhardt (paelzer) wrote : | # |
new head is 17654a2eb8251e9
LGTM now - thanks!
Christian Ehrhardt (paelzer) wrote : | # |
Total 24 (delta 13), reused 1 (delta 1), pack-reused 0
To ssh://git.
* [new tag] upload/
Uploading to ubuntu (via ftp to upload.ubuntu.com):
Uploading openipmi_
Uploading openipmi_
Uploading openipmi_
Uploading openipmi_
Uploading openipmi_
Successfully uploaded packages.
Preview Diff
1 | diff --git a/debian/changelog b/debian/changelog |
2 | index a46b8db..7ef0dc7 100644 |
3 | --- a/debian/changelog |
4 | +++ b/debian/changelog |
5 | @@ -1,3 +1,21 @@ |
6 | +openipmi (2.0.29-0.1ubuntu1) groovy; urgency=medium |
7 | + |
8 | + * Merge with Debian unstable. Remaining changes: |
9 | + - debian/patches/fix_pthread_pkg_config.patch: fix pkg-config file: |
10 | + Upstream's OpenIPMIpthread.pc.in file passes "-pthread" on |
11 | + CFLAGS, but not on LDFLAGS. Even though this might be OK, GCC |
12 | + recommends that, when "-pthread" is used, it should be passed on |
13 | + CFLAGS and LDFLAGS. This patch does that. |
14 | + * Dropped changes, accepted by Debian. |
15 | + - d/openipmi.init: fix various issues identified upstream (LP #1844448) |
16 | + - New upstream release 2.0.27 |
17 | + - d/rules: disable the now default-on python support |
18 | + - d/control: now needs libreadline-dev as build-depends |
19 | + - d/p/sample-fix-readline-include-in-ipmi_serial_bmc_emu.c.patch: |
20 | + fix readline include in ipmi_serial_bmc_emu.c |
21 | + |
22 | + -- Sergio Durigan Junior <sergio.durigan@canonical.com> Wed, 05 Aug 2020 13:07:18 -0400 |
23 | + |
24 | openipmi (2.0.29-0.1) unstable; urgency=medium |
25 | |
26 | [ Sergio Durigan Junior ] |
27 | @@ -33,6 +51,32 @@ openipmi (2.0.29-0.1) unstable; urgency=medium |
28 | |
29 | -- Sergio Durigan Junior <sergiodj@debian.org> Tue, 07 Jul 2020 22:28:53 -0400 |
30 | |
31 | +openipmi (2.0.27-0ubuntu2) focal; urgency=medium |
32 | + |
33 | + * Rebuild for libsnmp35. |
34 | + |
35 | + -- Rafael David Tinoco <rafaeldtinoco@ubuntu.com> Tue, 14 Jan 2020 18:04:48 +0000 |
36 | + |
37 | +openipmi (2.0.27-0ubuntu1) focal; urgency=medium |
38 | + |
39 | + * New upstream release 2.0.27 |
40 | + - d/rules: disable the now default-on python support |
41 | + - d/control: now needs libreadline-dev as build-depends |
42 | + - d/p/sample-fix-readline-include-in-ipmi_serial_bmc_emu.c.patch: fix |
43 | + FTBFS due to readline include in ipmi_serial_bmc_emu.c |
44 | + * d/openipmi.init: fix various issues identified upstream (LP: #1844448) |
45 | + |
46 | + -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Thu, 28 Nov 2019 14:12:35 +0100 |
47 | + |
48 | +openipmi (2.0.25-2.1ubuntu1) disco; urgency=low |
49 | + |
50 | + * Merge from Debian unstable. Remaining changes: |
51 | + - debian/patches/fix_pthread_pkg_config.patch: fix pkg-config files: |
52 | + pthread.pc does not exist in Ubuntu. Thanks to Simon McVittie |
53 | + <smcv@debian.org>. Closes #474087, LP #1311888. |
54 | + |
55 | + -- Gianfranco Costamagna <locutusofborg@debian.org> Sun, 07 Apr 2019 21:24:31 +0200 |
56 | + |
57 | openipmi (2.0.25-2.1) unstable; urgency=medium |
58 | |
59 | * Non-maintainer upload, with pre-approval from current maintainer. |
60 | @@ -42,6 +86,15 @@ openipmi (2.0.25-2.1) unstable; urgency=medium |
61 | |
62 | -- Thomas Goirand <zigo@debian.org> Wed, 27 Mar 2019 22:57:19 +0100 |
63 | |
64 | +openipmi (2.0.25-2ubuntu1) cosmic; urgency=medium |
65 | + |
66 | + * Merge with Debian; remaining changes: |
67 | + - debian/patches/fix_pthread_pkg_config.patch: fix pkg-config files: |
68 | + pthread.pc does not exist in Ubuntu. Thanks to Simon McVittie |
69 | + <smcv@debian.org>. Closes #474087, LP #1311888. |
70 | + |
71 | + -- Matthias Klose <doko@ubuntu.com> Sun, 17 Jun 2018 14:02:26 +0200 |
72 | + |
73 | openipmi (2.0.25-2) unstable; urgency=medium |
74 | |
75 | * Fixed openipmi init file for other archs. Thanks Majon! |
76 | @@ -66,6 +119,43 @@ openipmi (2.0.25-1) unstable; urgency=medium |
77 | |
78 | -- Noël Köthe <noel@debian.org> Fri, 18 May 2018 13:44:36 +0200 |
79 | |
80 | +openipmi (2.0.22-1.1ubuntu4) cosmic; urgency=medium |
81 | + |
82 | + * d/openipmi.init: Fixed openipmi init file for other archs. (LP: #1716517) |
83 | + |
84 | + -- Manoj Iyer <manoj.iyer@canonical.com> Mon, 21 May 2018 15:50:05 -0500 |
85 | + |
86 | +openipmi (2.0.22-1.1ubuntu3) cosmic; urgency=medium |
87 | + |
88 | + * No-change rebuild for ncurses soname changes. |
89 | + |
90 | + -- Matthias Klose <doko@ubuntu.com> Thu, 03 May 2018 14:17:44 +0000 |
91 | + |
92 | +openipmi (2.0.22-1.1ubuntu2) bionic; urgency=high |
93 | + |
94 | + * No change rebuild against openssl1.1. |
95 | + |
96 | + -- Dimitri John Ledkov <xnox@ubuntu.com> Tue, 06 Feb 2018 17:54:46 +0000 |
97 | + |
98 | +openipmi (2.0.22-1.1ubuntu1) artful; urgency=medium |
99 | + |
100 | + * Merge with Debian unstable. Remaining changes: |
101 | + - debian/openipmi.init: Change lockfile path. '/var/log/subsys' does |
102 | + not exists in Debian. (LP #722770) |
103 | + - Fix up packaging to install and use /etc/default/openipmi. |
104 | + (LP #473332) Thanks to Serge Hallyn. |
105 | + - Detect that a module is built into the kernel. |
106 | + + LP #1309860 |
107 | + - debian/openipmi.init: Do not unload all modules if some configured |
108 | + HW modules are not successfully loaded (LP #1318317). |
109 | + - debian/openipmi.init: add Default-Start and Default-Stop values |
110 | + (LP #1596474). |
111 | + - debian/patches/fix_pthread_pkg_config.patch: fix pkg-config files: |
112 | + pthread.pc does not exist in Ubuntu. Thanks to Simon McVittie |
113 | + <smcv@debian.org>. Closes #474087, LP #1311888. |
114 | + |
115 | + -- Nishanth Aravamudan <nish.aravamudan@canonical.com> Thu, 04 May 2017 15:21:46 -0700 |
116 | + |
117 | openipmi (2.0.22-1.1) unstable; urgency=medium |
118 | |
119 | * Non-maintainer upload. |
120 | @@ -73,6 +163,53 @@ openipmi (2.0.22-1.1) unstable; urgency=medium |
121 | |
122 | -- Sebastian Andrzej Siewior <sebastian@breakpoint.cc> Wed, 14 Dec 2016 22:19:50 +0100 |
123 | |
124 | +openipmi (2.0.22-1ubuntu1) zesty; urgency=medium |
125 | + |
126 | + * Merge with Debian unstable (LP: #1593461). Remaining changes: |
127 | + - debian/openipmi.init: Change lockfile path. '/var/log/subsys' does |
128 | + not exists in Debian. (LP #722770) |
129 | + - Fix up packaging to install and use /etc/default/openipmi. |
130 | + (LP #473332) Thanks to Serge Hallyn. |
131 | + - Detect that a module is built into the kernel. |
132 | + + LP #1309860 |
133 | + - debian/openipmi.init: Do not unload all modules if some configured |
134 | + HW modules are not successfully loaded (LP #1318317). |
135 | + - debian/openipmi.init: add Default-Start and Default-Stop values |
136 | + (LP #1596474). |
137 | + - debian/patches/fix_pthread_pkg_config.patch: fix pkg-config files: |
138 | + pthread.pc does not exist in Ubuntu. Thanks to Simon McVittie |
139 | + <smcv@debian.org>. Closes #474087, LP #1311888. |
140 | + * Drop: |
141 | + - debian/rules, debian/openipmi.init: |
142 | + + added init script |
143 | + [ Fixed in Debian ] |
144 | + - Do not explicitly depend on pthread. |
145 | + [ previously documented ] |
146 | + - Only use SIGPWR if defined. |
147 | + [ Previously undocumented ] |
148 | + - Delete copyright. |
149 | + [ previously undocumented ] |
150 | + - Add usr/sbin to debian/dirs |
151 | + [ previously undocumented ] |
152 | + - d/watch: update for new location |
153 | + [ previously undocumented ] |
154 | + - swig/pythong/openipmigui/Makefile: new file |
155 | + [ previously undocumented ] |
156 | + - New upstream version. |
157 | + [ Debian has moved past ] |
158 | + - Revert local changes to config.guess/config.sub; instead, use |
159 | + autotools-dev's debhelper integration to update these files for |
160 | + each build. |
161 | + [ Fixed in Debian ] |
162 | + - Enabling openssl support as it is causing |
163 | + problems running third party applications. (closes #775847) |
164 | + (LP #1546735) |
165 | + [ Fixed in 2.0.21-1 ] |
166 | + - Patch libtool macro support in for ppc64el shared libaries. |
167 | + [ Fixed in 2.0.16-1.4 by moving to autotools-dev ] |
168 | + |
169 | + -- Nishanth Aravamudan <nish.aravamudan@canonical.com> Tue, 22 Nov 2016 16:33:42 -0800 |
170 | + |
171 | openipmi (2.0.22-1) unstable; urgency=medium |
172 | |
173 | * new upstream release from 2016-06-01 |
174 | @@ -112,6 +249,80 @@ openipmi (2.0.21-1) unstable; urgency=medium |
175 | |
176 | -- Noël Köthe <noel@debian.org> Thu, 25 Feb 2016 12:06:21 +0100 |
177 | |
178 | +openipmi (2.0.18-0ubuntu11) xenial; urgency=medium |
179 | + |
180 | + * debian/openipmi.init: Do not unload all modules if some configured |
181 | + HW modules are not successfully loaded (LP: #1318317). |
182 | + |
183 | + -- Nishanth Aravamudan <nish.aravamudan@canonical.com> Thu, 03 Mar 2016 16:55:22 -0800 |
184 | + |
185 | +openipmi (2.0.18-0ubuntu10) xenial; urgency=medium |
186 | + |
187 | + * Enabling openssl support as it is causing |
188 | + problems running third party applications. (closes: #775847) |
189 | + (LP: #1546735) |
190 | + |
191 | + -- Eric Desrochers <slashd@ubuntu.com> Fri, 04 Mar 2016 17:28:50 +0000 |
192 | + |
193 | +openipmi (2.0.18-0ubuntu9) xenial; urgency=medium |
194 | + |
195 | + * No-change rebuild for ncurses6 transition. |
196 | + |
197 | + -- Matthias Klose <doko@ubuntu.com> Sun, 07 Feb 2016 09:44:35 +0000 |
198 | + |
199 | +openipmi (2.0.18-0ubuntu8) utopic; urgency=medium |
200 | + |
201 | + * Detect that a module is built into the kernel. |
202 | + -LP: #1309860 |
203 | + |
204 | + -- Tim Gardner <tim.gardner@canonical.com> Tue, 20 May 2014 11:15:44 -0700 |
205 | + |
206 | +openipmi (2.0.18-0ubuntu7) trusty; urgency=medium |
207 | + |
208 | + * Patch libtool macro support in for ppc64el shared libraries. |
209 | + |
210 | + -- Adam Conrad <adconrad@ubuntu.com> Tue, 18 Mar 2014 22:40:54 -0600 |
211 | + |
212 | +openipmi (2.0.18-0ubuntu6) saucy; urgency=low |
213 | + |
214 | + * Revert local changes to config.guess/config.sub; instead, use |
215 | + autotools-dev's debhelper integration to update these files for each |
216 | + build. |
217 | + |
218 | + -- Colin Watson <cjwatson@ubuntu.com> Tue, 08 Oct 2013 23:06:03 +0100 |
219 | + |
220 | +openipmi (2.0.18-0ubuntu5) saucy; urgency=low |
221 | + |
222 | + * Rebuild for libsnmp30. |
223 | + |
224 | + -- Colin Watson <cjwatson@ubuntu.com> Mon, 20 May 2013 11:05:34 +0100 |
225 | + |
226 | +openipmi (2.0.18-0ubuntu4) quantal; urgency=low |
227 | + |
228 | + * Rebuild for new armel compiler default of ARMv5t. |
229 | + |
230 | + -- Colin Watson <cjwatson@ubuntu.com> Mon, 08 Oct 2012 08:36:00 +0100 |
231 | + |
232 | +openipmi (2.0.18-0ubuntu3) natty; urgency=low |
233 | + |
234 | + * Fix up packaging to install and use /etc/default/openipmi. |
235 | + (LP: #473332) Thanks to Serge Hallyn. |
236 | + |
237 | + -- Chuck Short <zulcss@ubuntu.com> Fri, 04 Mar 2011 10:38:42 -0500 |
238 | + |
239 | +openipmi (2.0.18-0ubuntu2) natty; urgency=low |
240 | + |
241 | + * debian/openipmi.init: Change lockfile path. '/var/log/subsys' does not |
242 | + exists in Debian. (LP: #722770) |
243 | + |
244 | + -- Andres Rodriguez <andreserl@ubuntu.com> Thu, 24 Feb 2011 11:34:04 -0500 |
245 | + |
246 | +openipmi (2.0.18-0ubuntu1) maverick; urgency=low |
247 | + |
248 | + * New upstream version. |
249 | + |
250 | + -- Chuck Short <zulcss@ubuntu.com> Wed, 26 May 2010 14:58:52 -0400 |
251 | + |
252 | openipmi (2.0.16-1.4) unstable; urgency=medium |
253 | |
254 | * Non-maintainer upload. |
255 | @@ -153,6 +364,16 @@ openipmi (2.0.16-1.1) unstable; urgency=low |
256 | |
257 | -- Micah Anderson <micah@debian.org> Sat, 30 Jan 2010 19:17:13 -0500 |
258 | |
259 | +openipmi (2.0.16-1ubuntu1) lucid; urgency=low |
260 | + |
261 | + * Sync with debian. |
262 | + * debian/control: |
263 | + - changed maintainer to match DebianMaintainerField spec. |
264 | + * debian/rules, debian/openipmi.init: |
265 | + - added init script |
266 | + |
267 | + -- Chuck Short <zulcss@ubuntu.com> Thu, 05 Nov 2009 12:52:52 -0500 |
268 | + |
269 | openipmi (2.0.16-1) unstable; urgency=low |
270 | |
271 | * new upstream release from 2009-03-16 |
272 | @@ -164,6 +385,20 @@ openipmi (2.0.16-1) unstable; urgency=low |
273 | |
274 | -- Noèl Köthe <noel@debian.org> Tue, 28 Jul 2009 10:17:23 +0200 |
275 | |
276 | +openipmi (2.0.14-2ubuntu2) karmic; urgency=low |
277 | + |
278 | + * No-change upload to (hopefully) fix FTBFS. |
279 | + |
280 | + -- Martin Pitt <martin.pitt@ubuntu.com> Mon, 19 Oct 2009 10:11:04 +0200 |
281 | + |
282 | +openipmi (2.0.14-2ubuntu1) jaunty; urgency=low |
283 | + |
284 | + * Merge from debian unstable, remaining changes: |
285 | + - debian/control: changed maintainer to match DebianMaintainerField spec. |
286 | + - debian/rules: debian/openipmi.init: added init-script. |
287 | + |
288 | + -- Chuck Short <zulcss@ubuntu.com> Wed, 05 Nov 2008 03:00:14 +0000 |
289 | + |
290 | openipmi (2.0.14-2) unstable; urgency=low |
291 | |
292 | * added GNU/kFreeBSD patch from Petr. Thanks |
293 | @@ -171,6 +406,14 @@ openipmi (2.0.14-2) unstable; urgency=low |
294 | |
295 | -- Noèl Köthe <noel@debian.org> Sun, 19 Oct 2008 15:58:15 +0200 |
296 | |
297 | +openipmi (2.0.14-1ubuntu1) intrepid; urgency=low |
298 | + |
299 | + * Merge from debian unstable, remaining changes: |
300 | + - debian/control: changed maintainer to match DebianMaintainerField spec. |
301 | + - debian/rules: debian/openipmi.init: added init-script. |
302 | + |
303 | + -- Chuck Short <zulcss@ubuntu.com> Sun, 20 Jul 2008 13:51:04 +0100 |
304 | + |
305 | openipmi (2.0.14-1) unstable; urgency=medium |
306 | |
307 | * new upstream release |
308 | @@ -179,6 +422,15 @@ openipmi (2.0.14-1) unstable; urgency=medium |
309 | |
310 | -- Noèl Köthe <noel@debian.org> Sat, 19 Jul 2008 20:30:39 +0200 |
311 | |
312 | +openipmi (2.0.13-1ubuntu1) intrepid; urgency=low |
313 | + |
314 | + * Merge from Debian unstable (LP: #242632), remaining changes: |
315 | + - debian/control: remove unecessary cdbs build-dep |
316 | + - debian/rules, debian/openipmi.init: added init script |
317 | + - debian/control: changed Maintainer to match DebianMaintinaerField spec. |
318 | + |
319 | + -- Thierry Carrez <thierry.carrez@ubuntu.com> Tue, 24 Jun 2008 14:24:49 +0200 |
320 | + |
321 | openipmi (2.0.13-1) unstable; urgency=low |
322 | |
323 | * new upstream release |
324 | @@ -189,6 +441,31 @@ openipmi (2.0.13-1) unstable; urgency=low |
325 | |
326 | -- Noèl Köthe <noel@debian.org> Mon, 25 Feb 2008 20:07:00 +0100 |
327 | |
328 | +openipmi (2.0.13-0ubuntu3) hardy; urgency=low |
329 | + |
330 | + * Make the init script more lsb specific. |
331 | + |
332 | + -- Chuck Short <zulcss@ubuntu.com> Wed, 13 Feb 2008 14:03:38 -0500 |
333 | + |
334 | +openipmi (2.0.13-0ubuntu2) hardy; urgency=low |
335 | + |
336 | + * Added init script. |
337 | + |
338 | + -- Chuck Short <zulcss@ubuntu.com> Tue, 12 Feb 2008 19:24:00 -0500 |
339 | + |
340 | +openipmi (2.0.13-0ubuntu1) hardy; urgency=low |
341 | + |
342 | + * New upstream release. |
343 | + * Cleaned up debian/control. |
344 | + |
345 | + -- Chuck Short <zulcss@ubuntu.com> Tue, 12 Feb 2008 12:49:05 -0500 |
346 | + |
347 | +openipmi (2.0.11-2build1) hardy; urgency=low |
348 | + |
349 | + * Rebuild for libsnmp10 -> libsnmp15 transition. |
350 | + |
351 | + -- Steve Kowalik <stevenk@ubuntu.com> Mon, 10 Dec 2007 20:32:47 +1100 |
352 | + |
353 | openipmi (2.0.11-2) unstable; urgency=low |
354 | |
355 | * correcting build dependency libsnmp10-dev -> libsnmp-dev |
356 | diff --git a/debian/control b/debian/control |
357 | index 20bfe69..a20136c 100644 |
358 | --- a/debian/control |
359 | +++ b/debian/control |
360 | @@ -1,7 +1,8 @@ |
361 | Source: openipmi |
362 | Section: admin |
363 | Priority: optional |
364 | -Maintainer: Noël Köthe <noel@debian.org> |
365 | +Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com> |
366 | +XSBC-Original-Maintainer: Noël Köthe <noel@debian.org> |
367 | Build-Depends: debhelper-compat (= 13), |
368 | libsnmp-dev, |
369 | libpopt-dev, |
370 | diff --git a/debian/patches/fix_pthread_pkg_config.patch b/debian/patches/fix_pthread_pkg_config.patch |
371 | new file mode 100644 |
372 | index 0000000..928e5c9 |
373 | --- /dev/null |
374 | +++ b/debian/patches/fix_pthread_pkg_config.patch |
375 | @@ -0,0 +1,22 @@ |
376 | +Description: fix pkg-config files: |
377 | + Upstream's OpenIPMIpthread.pc.in file passes "-pthread" on |
378 | + CFLAGS, but not on LDFLAGS. Even though this might be OK, GCC |
379 | + recommends that, when "-pthread" is used, it should be passed on |
380 | + CFLAGS and LDFLAGS. This patch does that. |
381 | +Author: Simon McVittie <smcv@debian.org> |
382 | +Reviewed-By: Sergio Durigan Junior <sergio.durigan@canonical.com> |
383 | +Bug-Debian: https://bugs.debian.org/474087 |
384 | +Bug-Ubuntu: https://bugs.launchpad.net/bugs/1311888 |
385 | +Forwarded: https://sourceforge.net/p/openipmi/code/merge-requests/6/ |
386 | + |
387 | +Index: b/OpenIPMIpthread.pc.in |
388 | +=================================================================== |
389 | +--- a/OpenIPMIpthread.pc.in |
390 | ++++ b/OpenIPMIpthread.pc.in |
391 | +@@ -7,5 +7,5 @@ Name: OpenIPMIpthread |
392 | + Description: Pthread OS handler for OpenIPMI |
393 | + Version: @VERSION@ |
394 | + Requires: OpenIPMI |
395 | +-Libs: -L${libdir} -lOpenIPMIutils -lOpenIPMIpthread |
396 | ++Libs: -L${libdir} -lOpenIPMIutils -lOpenIPMIpthread -pthread |
397 | + Cflags: -I${includedir} -pthread |
398 | diff --git a/debian/patches/series b/debian/patches/series |
399 | index 732215f..7112540 100644 |
400 | --- a/debian/patches/series |
401 | +++ b/debian/patches/series |
402 | @@ -1 +1,2 @@ |
403 | increase-MAX_CONFIG_LINE.patch |
404 | +fix_pthread_pkg_config.patch |
Christian, I'm requesting a review from you because you're the person who last touched the package, but feel free to decline if you have too much to do already :-). TIA!