Bug#856665: jessie-pu: package commons-daemon/1.0.15-6

[Adam D. Barratt]

Control: tags -1 + pending

On Thu, 2017-03-09 at 19:58 +, Adam D. Barratt wrote:
> Control: tags -1 + confirmed
> 
> On Fri, 2017-03-03 at 17:07 -0300, Breno Leitao wrote:
> > Hi Adam,
> > 
> > On Fri, Mar 03, 2017 at 05:41:51PM +, Adam D. Barratt wrote:
> > > On 2017-03-03 17:29, Breno Leitao wrote:
> > > >Package jsvc (commons-daemon) contains a patch enabling ppc64el on
> > > >version 1.0.15-6 (currently on Jessie), but it does not work. A new
> > > >patch required to add functional support for commons-daemon on ppc64el.
> > > >
> > > >This patch is already commited on upstream as in Stretch.
> > > >
> > > >You can find more information about this bug at #856560.
> > > >
> > > >Let me know if I can update a fixed package in stable.
> > > 
> > > We'd need to see a source debdiff for the proposed (and tested) package
> > > first, please.
> > 
> > Sure. This is the debdiff I have. I tested it and it solves the problem
> > on ppc64el, and it does not seem cause any regression on amd64.
> 
> +  * This package is broken on Jessie, showing "Cannot find any VM in Java
> +Home". Fixing it. (Closes: #856560)
> 
> That sounds a little overly verbose, personally - it's reasonably
> obvious that the package is broken in some way and you're fixing it,
> otherwise the update wouldn't exist in the first place.
> 
> In any case, please update the changelog to mention the fact that the
> fix is specifically related to ppc64el, and feel free to upload.

Uploaded and flagged for acceptance.

Regards,

Adam

Bug#856665: jessie-pu: package commons-daemon/1.0.15-6

[Adam D. Barratt]

Control: tags -1 + confirmed

On Fri, 2017-03-03 at 17:07 -0300, Breno Leitao wrote:
> Hi Adam,
> 
> On Fri, Mar 03, 2017 at 05:41:51PM +, Adam D. Barratt wrote:
> > On 2017-03-03 17:29, Breno Leitao wrote:
> > >Package jsvc (commons-daemon) contains a patch enabling ppc64el on
> > >version 1.0.15-6 (currently on Jessie), but it does not work. A new
> > >patch required to add functional support for commons-daemon on ppc64el.
> > >
> > >This patch is already commited on upstream as in Stretch.
> > >
> > >You can find more information about this bug at #856560.
> > >
> > >Let me know if I can update a fixed package in stable.
> > 
> > We'd need to see a source debdiff for the proposed (and tested) package
> > first, please.
> 
> Sure. This is the debdiff I have. I tested it and it solves the problem
> on ppc64el, and it does not seem cause any regression on amd64.

+  * This package is broken on Jessie, showing "Cannot find any VM in Java
+Home". Fixing it. (Closes: #856560)

That sounds a little overly verbose, personally - it's reasonably
obvious that the package is broken in some way and you're fixing it,
otherwise the update wouldn't exist in the first place.

In any case, please update the changelog to mention the fact that the
fix is specifically related to ppc64el, and feel free to upload.

Regards,

Adam

Bug#856665: jessie-pu: package commons-daemon/1.0.15-6

[Breno Leitao]

Hi Adam,

On Fri, Mar 03, 2017 at 05:41:51PM +, Adam D. Barratt wrote:
> On 2017-03-03 17:29, Breno Leitao wrote:
> >Package jsvc (commons-daemon) contains a patch enabling ppc64el on
> >version 1.0.15-6 (currently on Jessie), but it does not work. A new
> >patch required to add functional support for commons-daemon on ppc64el.
> >
> >This patch is already commited on upstream as in Stretch.
> >
> >You can find more information about this bug at #856560.
> >
> >Let me know if I can update a fixed package in stable.
> 
> We'd need to see a source debdiff for the proposed (and tested) package
> first, please.

Sure. This is the debdiff I have. I tested it and it solves the problem
on ppc64el, and it does not seem cause any regression on amd64.
diff -Nru commons-daemon-1.0.15/debian/changelog 
commons-daemon-1.0.15/debian/changelog
--- commons-daemon-1.0.15/debian/changelog  2014-11-11 10:01:45.0 
-0500
+++ commons-daemon-1.0.15/debian/changelog  2017-03-03 13:47:51.0 
-0500
@@ -1,3 +1,11 @@
+commons-daemon (1.0.15-6+deb8u1) jessie; urgency=medium
+
+  * Team upload.
+  * This package is broken on Jessie, showing "Cannot find any VM in Java
+Home". Fixing it. (Closes: #856560)
+
+ -- Breno Leitao   Fri, 03 Mar 2017 13:47:51 -0500
+
 commons-daemon (1.0.15-6) unstable; urgency=medium
 
   * Team upload.
diff -Nru commons-daemon-1.0.15/debian/patches/ppc64el.diff 
commons-daemon-1.0.15/debian/patches/ppc64el.diff
--- commons-daemon-1.0.15/debian/patches/ppc64el.diff   2014-11-11 
10:01:45.0 -0500
+++ commons-daemon-1.0.15/debian/patches/ppc64el.diff   2017-03-03 
13:34:37.0 -0500
@@ -1,7 +1,7 @@
 Description: Add ppc64el support
 Author: Colin Watson 
-Forwarded: https://issues.apache.org/jira/browse/DAEMON-326
-Last-Update: 2014-11-06
+Forwarded: https://issues.apache.org/jira/browse/DAEMON-358
+Last-Update: 2017-03-02
 
 Index: b/src/native/unix/configure
 ===
@@ -12,9 +12,9 @@
  HOST_CPU=aarch64
  ;;
 +  powerpc64le)
-+CFLAGS="$CFLAGS -DCPU=\\\"powerpc64le\\\""
-+supported_os="powerpc64le"
-+HOST_CPU=powerpc64le
++CFLAGS="$CFLAGS -DCPU=\\\"ppc64le\\\""
++supported_os="ppc64le"
++HOST_CPU=ppc64le
 +;;
*)
  echo "$as_me:$LINENO: result: failed" >&5
@@ -28,9 +28,9 @@
  HOST_CPU=aarch64
  ;;
 +  powerpc64le)
-+CFLAGS="$CFLAGS -DCPU=\\\"powerpc64le\\\""
-+supported_os="powerpc64le"
-+HOST_CPU=powerpc64le
++CFLAGS="$CFLAGS -DCPU=\\\"ppc64le\\\""
++supported_os="ppc64le"
++HOST_CPU=ppc64le
 +;;
*)
  AC_MSG_RESULT([failed])


pgpo1GLhts7v8.pgp
Description: PGP signature

Bug#856665: jessie-pu: package commons-daemon/1.0.15-6

[Adam D. Barratt]

Control: tags -1 + moreinfo

On 2017-03-03 17:29, Breno Leitao wrote:

Package jsvc (commons-daemon) contains a patch enabling ppc64el on
version 1.0.15-6 (currently on Jessie), but it does not work. A new
patch required to add functional support for commons-daemon on ppc64el.

This patch is already commited on upstream as in Stretch.

You can find more information about this bug at #856560.

Let me know if I can update a fixed package in stable.


We'd need to see a source debdiff for the proposed (and tested) package 
first, please.


Regards,

Adam

Bug#856665: jessie-pu: package commons-daemon/1.0.15-6

[Breno Leitao]

Package: release.debian.org
Severity: normal
Tags: jessie
User: release.debian@packages.debian.org
Usertags: pu

Dear Release team,

Package jsvc (commons-daemon) contains a patch enabling ppc64el on
version 1.0.15-6 (currently on Jessie), but it does not work. A new
patch required to add functional support for commons-daemon on ppc64el.

This patch is already commited on upstream as in Stretch.

You can find more information about this bug at #856560.

Let me know if I can update a fixed package in stable.

Thank you,
Breno

-- System Information:
Debian Release: 9.0
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing'), (1, 'experimental')
Architecture: ppc64el (ppc64le)
Foreign Architectures: powerpc

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