Your message dated Tue, 05 Oct 2021 19:33:45 +0000
with message-id <[email protected]>
and subject line Bug#995406: fixed in bbmap 38.94+dfsg-1
has caused the Debian Bug report #995406,
regarding bbmap: package does not ship resource files
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
995406: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=995406
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: bbmap
Version: 38.93+dfsg-1
Severity: normal
Tags: patch
X-Debbugs-Cc: [email protected]

Dear Maintainer,

The bbmap package does not ship the needed resource files which causes some of
the included tools not to work, e.g. bbduk when trying to process some fastq
data, crashes with output like [1].

I don't have a census which other bbtools require resource files to be usable,
but bbduk is not usable at all in the current state.

My suggested fix (see patch [2]) puts the resource files to the root of
the built jar file.  This is were dna.Data.findPath() will indeed find
these files, which can be seen when looking at the verbose output [3]
for findPath() (by setting vb=true in current/dna.Data.java near line
1204).  By contrast, the current packaging tries (but also fails) to put
the files under resources/.  I don't see a way to trick jh_build with
the JH_JAR_EXTRA mechanism to do what is needed (without changes to
jh_build).  The patch applies agaiinst the bbmap source package's
current git master branch and the package builds and run for me on
bullseye.

thanks
(Robert)


[1] error message of bbduk tool:


$ pwd
/tmp/bbduk_test
$ ls
fwd.fastq  rev.fastq
$ bbduk.sh in1=fwd.fastq in2=rev.fastq ktrim=r k=21 mink=8 hdist=2 ftm=5 tpe 
tbo threads=48 out=out.fastq
java -ea -Xmx76702m -Xms76702m -cp /usr/share/java/bbmap.jar jgi.BBDuk 
in1=fwd.fastq in2=rev.fastq ktrim=r k=21 mink=8 hdist=2 ftm=5 tpe tbo 
threads=48 out=out.fastq
Executing jgi.BBDuk [in1=fwd.fastq, in2=rev.fastq, ktrim=r, k=21, mink=8, 
hdist=2, ftm=5, tpe, tbo, threads=48, out=out.fastq]
Version 38.90

Set threads to 48
maskMiddle was disabled because useShortKmers=true
Warning!  Cannot find primes.txt.gz 
/tmp/bbduk_test/file:/usr/share/java/bbmap.jar!/primes.txt.gz
java.lang.Exception
        at dna.Data.findPath(Data.java:1247)
        at dna.Data.findPath(Data.java:1194)
        at shared.Primes.fetchPrimes(Primes.java:167)
        at shared.Primes.<clinit>(Primes.java:177)
        at kmer.ScheduleMaker.<clinit>(ScheduleMaker.java:155)
        at jgi.BBDuk.<init>(BBDuk.java:964)
        at jgi.BBDuk.main(BBDuk.java:78)
Exception in thread "main" java.lang.ExceptionInInitializerError
        at kmer.ScheduleMaker.<clinit>(ScheduleMaker.java:155)
        at jgi.BBDuk.<init>(BBDuk.java:964)
        at jgi.BBDuk.main(BBDuk.java:78)
Caused by: java.lang.NullPointerException
        at fileIO.ByteFile.<init>(ByteFile.java:43)
        at fileIO.ByteFile1.<init>(ByteFile1.java:98)
        at fileIO.ByteFile1.<init>(ByteFile1.java:94)
        at shared.Primes.fetchPrimes(Primes.java:169)
        at shared.Primes.<clinit>(Primes.java:177)
        ... 3 more



[2] proposed patch


diff --git a/debian/rules b/debian/rules
index a44c8e7..479739f 100755
--- a/debian/rules
+++ b/debian/rules
@@ -8,7 +8,6 @@ PKGSHAREDIR    := debian/$(DEB_SOURCE)/usr/share/$(DEB_SOURCE)
 export JAVA_HOME  := /usr/lib/jvm/default-java
 DEBJAR    := /usr/share/java
 export CLASSPATH := $(DEBJAR)/mpi.jar
-export JH_JAR_EXTRA := resources/*

 %:
        dh $@ --with javahelper
@@ -23,7 +22,8 @@ override_dh_clean:

 override_dh_auto_build:
        cd jni && make -f makefile.linux
-       JH_JAR_EXTRA=`$(CURDIR)/resources/*` jh_build $(DEB_SOURCE).jar current
+       jh_build $(DEB_SOURCE).jar current
+       cd $(CURDIR)/resources && jar uf $(CURDIR)/$(DEB_SOURCE).jar $(subst 
resources/,,$(wildcard resources/*))

 override_dh_install:
        dh_install



[3] bbduk output like [1] but with proposed patch and verbose output for 
findPath() applied:


$ bbduk.sh in1=fwd.fastq in2=rev.fastq ktrim=r k=21 mink=8 hdist=2 ftm=5 tpe 
tbo threads=48 out=out.fastq
java -ea -Xmx76702m -Xms76702m -cp /usr/share/java/bbmap.jar jgi.BBDuk 
in1=fwd.fastq in2=rev.fastq ktrim=r k=21 mink=8 hdist=2 ftm=5 tpe tbo 
threads=48 out=out.fastq
Executing jgi.BBDuk [in1=fwd.fastq, in2=rev.fastq, ktrim=r, k=21, mink=8, 
hdist=2, ftm=5, tpe, tbo, threads=48, out=out.fastq]
Version 38.93

Set threads to 48
maskMiddle was disabled because useShortKmers=true
Did not find primes.txt.gz at 
/tmp/bbduk_test/file:/usr/share/java/bbmap.jar!/primes.txt.gz
Did not find primes.txt.gz at 
/tmp/bbduk_test/file:/usr/share/java/resources/primes.txt.gz
Considering fixing /tmp/bbduk_test/file:/usr/share/java/bbmap.jar!/primes.txt.gz
true
Did not find primes.txt.gz at file:/usr/share/java/bbmap.jar!/primes.txt.gz
Considering getResource
Found URL jar:file:/usr/share/java/bbmap.jar!/primes.txt.gz
Found primes.txt.gz at jar:file:/usr/share/java/bbmap.jar!/primes.txt.gz
0.032 seconds.
Initial:
Memory: max=80429m, total=80429m, free=80346m, used=83m
...



-- System Information:
Debian Release: 11.0
  APT prefers stable-security
  APT policy: (990, 'stable-security'), (990, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.15.0-132-generic (SMP w/96 CPU threads)
Locale: LANG=C, LC_CTYPE=C (charmap=UTF-8) (ignored: LC_ALL set to 
en_US.UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /bin/dash
Init: unable to detect

Versions of packages bbmap depends on:
ii  libmpj-java  0.44+dfsg-4

Versions of packages bbmap recommends:
ii  pigz  2.6-1

bbmap suggests no packages.

-- no debconf information

--- End Message ---
--- Begin Message ---
Source: bbmap
Source-Version: 38.94+dfsg-1
Done: Étienne Mollier <[email protected]>

We believe that the bug you reported is fixed in the latest version of
bbmap, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Étienne Mollier <[email protected]> (supplier of updated bbmap package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Tue, 05 Oct 2021 21:06:02 +0200
Source: bbmap
Architecture: source
Version: 38.94+dfsg-1
Distribution: unstable
Urgency: medium
Maintainer: Debian Med Packaging Team 
<[email protected]>
Changed-By: Étienne Mollier <[email protected]>
Closes: 995406
Changes:
 bbmap (38.94+dfsg-1) unstable; urgency=medium
 .
   * Team upload.
 .
   [ Andreas Tille ]
   * Include resources files
     Closes: #995406
 .
   [ Étienne Mollier ]
   * New upstream version.
   * add autopkgtest, following discussion on #995406; thanks Robert Hein for
     the help with testing bbduk.sh!
   * d/control: add bbmap dependency to a JRE.
Checksums-Sha1:
 5260550cd8af34aa23d10236fb0bc897bd743052 2119 bbmap_38.94+dfsg-1.dsc
 4ae94aebdf632d4a1c12224e374b8ef8075a3162 4515308 bbmap_38.94+dfsg.orig.tar.xz
 4b656e92ea86d2dc24ba968406442c3ff3c9caf6 24356 bbmap_38.94+dfsg-1.debian.tar.xz
 d9fdb074c16be6f20af7eefb8624409911bc321e 11134 
bbmap_38.94+dfsg-1_amd64.buildinfo
Checksums-Sha256:
 807a3ff94a586457189d1b7c775da0118282cd618994780d6949321d379eab65 2119 
bbmap_38.94+dfsg-1.dsc
 86e180b639d07feb9aa1eda7fe4ed66f9fa6367d5751b363bb36ca1b192f877a 4515308 
bbmap_38.94+dfsg.orig.tar.xz
 0a5916da67ad27a4dff9e2bbba1f16c98dc97a08cf57492893b8d57c1df8acdd 24356 
bbmap_38.94+dfsg-1.debian.tar.xz
 416720ccd5944e046652ff30dc574eebf309baeb6911d204e7d0d62d322f738d 11134 
bbmap_38.94+dfsg-1_amd64.buildinfo
Files:
 7d6beacb23e0c6ff17a3e5a0feac4a9d 2119 science optional bbmap_38.94+dfsg-1.dsc
 7383f5564bea87d801286cab42536260 4515308 science optional 
bbmap_38.94+dfsg.orig.tar.xz
 1978750954f8591e0734207daac5b9e6 24356 science optional 
bbmap_38.94+dfsg-1.debian.tar.xz
 42eca40cb65fd92436b8854c6f8468fa 11134 science optional 
bbmap_38.94+dfsg-1_amd64.buildinfo

-----BEGIN PGP SIGNATURE-----

iQJIBAEBCgAyFiEEj5GyJ8fW8rGUjII2eTz2fo8NEdoFAmFco4sUHGVtb2xsaWVy
QGRlYmlhbi5vcmcACgkQeTz2fo8NEdpOvw//RwUBF1XVQV0RF+ASuo5VXWAN81vC
PwQ7VlyK0MKPjo1e7JQu1KAStOodLNlVq66IKwWJH3hzffH/v547T3cWlw25Iebx
6c02mjZ4KS8JwRnNYuNoDktSqU0YcK1zHkWKy46jMrNMj8vB9HSz0lIRnDBRym4O
sTOk8CPYwG2x3/3u7LBGUQBZIvXZOvUniP3gyCqgG1645y8hIbAjLsPNpTCSwf3K
7H5f+xuNpnAKtaz3bUb+ymUJpD6EOFfLWOT7keMguau5ZlPPYOysdd1L0Z3yn5W0
Z93aF0ikZQ32B2aDi7mePR6QSVwHTkySEA3YIosHh5gO47mcVG46QNl4mjqwYzqm
xoqDEN6naWW0SlC2LuJAXvckzYcOM/z6VVyb/tmFTbIfmgiR28IBCBLttbmzFKKL
d2d6DAsSi3/jciiLyMuTomokVS1OVRuxJnhxaWT0ThtDjMpqo+lOaP+RA99TzBI0
XND123pQXGHNf9B1vKPzysl98cdJzzXGoAvFkT4a2cWOFddUdkROLj7FYBbdyaaD
OdxoIvG9Me6NDcZ0Z4uhZKZNt71ZLfJxkfULb4Ls9aLGYeaVi1gTz+Wlz0kjsJyi
yZkEoKpEZOhuYkSid0b4OYyt3JsnGmfdnBXx3B9LqZULEK+VnODulyMotRQeuiCS
SFHV0LHrS6sLFv0=
=R+Lt
-----END PGP SIGNATURE-----

--- End Message ---

Reply via email to