Bug#770762: libinline-java-perl: Build dependencies are too loose

[Peter Pentchev]

Control: tags -1 + confirmed patch

On Sun, Nov 23, 2014 at 01:12:06PM -0800, Daniel Schepler wrote:
> Source: libinline-java-perl
> Version: 0.53-3
> Severity: serious
> 
> If I try building src:libinline-java-perl in a chroot with gcj-jdk installed
> but not default-jdk, then that satisfies the Build-Depends.  However, the
> build then fails:
> 
>  debian/rules build
[snip]
> Java 2 SDK installation directory '/usr/lib/jvm/default-java' does not exist.
> /usr/share/cdbs/1/class/perl-makemaker.mk:44: recipe for target 'Makefile' failed
> make: *** [Makefile] Error 1
> dpkg-buildpackage: error: debian/rules build gave error exit status 2

Thanks for the bug report!

Attached is a patch to the version in Git that should fix the build by
depending unconditionally on default-jdk.  This should be all right,
since the Debian Java policy says so :)

If there are no objections, I intend to commit this patch to the pkg-perl
Git repository later today.

G'luck,
Peter

-- 
Peter Pentchev  roam at ringlet.net roam at FreeBSD.org p.penchev at storpool.com
PGP key:        http://people.FreeBSD.org/~roam/roam.key.asc
Key fingerprint 2EE7 A7A5 17FC 124C F115  C354 651E EFB0 2527 DF13
-------------- next part --------------
From d020bed987ec62fbca9f5e80d6a7feedf813f6f8 Mon Sep 17 00:00:00 2001
From: Peter Pentchev <roam at ringlet.net>
Date: Mon, 24 Nov 2014 17:48:20 +0200
Subject: [PATCH 3/3] Build unconditionally with default-jdk.

This complies with section 2.2 of the Debian policy for Java.

Closes:		#770762
Reported by:	Daniel Schepler <dschepler at gmail.com>
---
 debian/control | 2 +-
 debian/rules   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/debian/control b/debian/control
index cf15224..b0d8b01 100644
--- a/debian/control
+++ b/debian/control
@@ -8,7 +8,7 @@ Build-Depends: cdbs,
  dh-buildinfo,
  libinline-perl (>= 0.68),
  libinline-c-perl,
- default-jdk | java-compiler,
+ default-jdk,
  chrpath
 Maintainer: Debian Perl Group <pkg-perl-maintainers at lists.alioth.debian.org>
 Uploaders: Jonas Smedegaard <dr at jones.dk>
diff --git a/debian/rules b/debian/rules
index 22b091b..c6b67ca 100755
--- a/debian/rules
+++ b/debian/rules
@@ -30,7 +30,7 @@ DEB_UPSTREAM_URL = http://www.cpan.org/modules/by-module/Inline
 DEB_UPSTREAM_TARBALL_MD5 = 23f96c2ac4915302c3647c0abc92916f
 
 # Needed both by upstream build and at runtime
-deps = libinline-perl (>= 0.68), libinline-c-perl, default-jdk | java-compiler
+deps = libinline-perl (>= 0.68), libinline-c-perl, default-jdk
 
 # Needed by upstream build
 b-depends = chrpath
-- 
2.1.3

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-perl-maintainers/attachments/20141124/653532fa/attachment.sig>