Review: libcentres-java 1.4-1
Package Information
| Description | Stereochemistry labelling according to Cahn-Ingold-Prelog priority rules Centres is an Open Source Java library for that allows perception and labelling of stereogenic centres in chemical structures using the Cahn-Ingold-Prelog priority rules. |
|---|---|
| Maintainer | Debian Java Maintainers <pkg-java-maintainers@lists.alioth.debian.org> |
| Changed By | Andrius Merkys <merkys@debian.org> |
| Distribution | unstable |
| Architecture | all |
| VCS | git: https://salsa.debian.org/java-team/libcentres-java.git (browse) |
| Closes | #1150236 |
| Tracker | https://tracker.debian.org/pkg/libcentres-java |
| Uploaded | 29 days ago |
New Package Report
.changes
| Source | libcentres-java |
|---|---|
| Version | 1.4-1 |
| Changed-By | Andrius Merkys <merkys@debian.org> |
| Architecture | source all |
| Distribution | unstable |
| Date | Fri, 09 Oct 2026 04:38:03 -0400 |
Changelog
libcentres-java (1.4-1) unstable; urgency=medium . * Initial release (Closes: #1150236)
.dsc
| Priority | optional |
|---|---|
| Component | main |
| Package-List | libcentres-java deb java optional arch=all |
| Section | java |
debian/copyright
Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/ Upstream-Name: centres Source: http://johnmay.github.com/centres Files: * Copyright: 2026, John Mayfield License: BSD-2-clause Files: cdk/pom.xml cdk/src/test/resources/uk/ac/ebi/centres/cdk/CHEBI_15645.xml Copyright: 2012, 2014, John May License: LGPL-2.1+ Files: debian/* Copyright: 2026, Andrius Merkys <merkys@debian.org> License: BSD-2-clause License: BSD-2-clause Redistribution and use in source and binary forms, with or without modification, are permitted provided that the following conditions are met: . 1. Redistributions of source code must retain the above copyright notice, this list of conditions and the following disclaimer. 2. Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the following disclaimer in the documentation and/or other materials provided with the distribution. . THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS License: LGPL-2.1+ On Debian systems, the full text of the LGPL-2.1 license can be found in the file '/usr/share/common-licenses/LGPL-2.1'
Review Information
rejected — allocated to awm 26 days ago, started 26 days ago, completed 26 days ago.
Final Comment
Hi,
Several of these issues might be incorrect: please advise if they are wrong so I can ignore them on a future upload. In particular I did not attempt to build the package, so the assertions about FTBFS on current buildds may be incorrect however the licensing issues #3 and #4 below need fixing before I can accept.
- [packaging] Build fails: pom pins javac source/target 1.7, but default-jdk is OpenJDK 25
- Files:
pom.xml:64-66,debian/control:11 - Evidence:
pom.xmlmaven-compiler-pluginconfiguration<source>1.7</source><target>1.7</target>;Build-Depends: default-jdk; apt-cache shows default-jdk -> openjdk-25-jdk. - Problem: javac since JDK 20 errors with "Source option 7 is no longer supported. Use 8 or later." The parent pom's compiler configuration propagates to the
core,cdkandopsinmodules, somvn packagecannot compile any main source on the current unstable toolchain (Build-Dependsdefault-jdk). maven-debian-helper (2.6.9) does not rewrite pom compiler settings, anddebian/maven.propertiesonly setsmaven.test.skip=true. This is a build failure on current archive buildds. - Fix: Add a quilt patch raising the compiler
source/target(or usemaven.compiler.release) to 8 or higher, and bump the compiler plugin as needed.
- [packaging] cdk module's shade plugin builds an uberjar that embeds the whole CDK
- Files:
cdk/pom.xml:94-107,debian/libcentres-java.poms,debian/clean:1 - Evidence:
maven-shade-pluginexecution bound topackagewith<finalName>centres</finalName>(noshadedArtifactAttached), so it replaces the module's main artifact;debian/cleancontainscdk/dependency-reduced-pom.xml(a shade by-product), proving the plugin runs during the Debian build;libcentres-java.pomsinstallscdk/pom.xml's jar with no option to install the unshaded artifact. - Problem: The installed centres-cdk jar would bundle a full copy of CDK (LGPL-2.1 code) even though CDK is already packaged as
libcdk-javaand is a Build-Depends/Runtime-Depends of this package. That contradicts Policy 4.13 (embedded code copies) and makes the BSD-2 attribution indebian/copyrightfor that jar incomplete, since the embedded CDK classes are not covered by it. - Fix: Disable the shade plugin for the Debian build (or build and install the unshaded jar) so the package relies on the CDK classes from
libcdk-java; verify the installed jar's contents.
- [licensing] BSD-2-clause License stanza in debian/copyright is truncated
- Files:
debian/copyright:36 - Evidence: Stanza ends
...ARISING IN ANY WAY OUT OF THE USE OF THIS; the LICENSE file (lines 22-23) continues...OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. - Problem: BSD-2-clause is not in
/usr/share/common-licenses, so the complete text must be reproduced in
debian/copyright(copyright-format 6.7) and thus in the binary package's copyright file. The last sentence of the warranty disclaimer is missing. - Fix: Append the missing final sentence to the
License: BSD-2-clausestanza.
- [licensing] Root pom.xml declares LGPL-3.0 while actual license is BSD-2
- Files:
pom.xml:26-31 - Evidence:
<licenses><license><name>GNU Lesser General Public License, Version 3.0</name>while the top-levelLICENSE(BSD-2) andREADME.mdsay "BSD 2-Clause", anddebian/copyrightattributespom.xmlto BSD-2-clause. - Problem: Leftover upstream metadata: this pom is installed by maven-debian-helper (
libcentres-java.poms), so the shipped/usr/share/maven-repo/.../centres-1.4.pomadvertises LGPL-3.0, contradicting the package's copyright and the actual license grant. - Fix: Fix the
<licenses>block upstream to BSD-2 (or document the mismatch with a Comment indebian/copyrightand contact upstream).
- [packaging] Outdated Standards-Version (4.7.0, current is 4.7.4.1)
- Files:
debian/control:15 - Evidence:
Standards-Version: 4.7.0; Policy tool reports Debian Policy version 4.7.4.1. - Problem: The Standards-Version does not reflect the current Policy; section 4.1 requires a maintainer to aim for conformance with the latest Policy, and newer 4.7.x releases contain relevant clarifications.
- Fix: Bump Standards-Version to 4.7.4.1 after checking the A10 upgrading checklist.
Thanks!
Other Reviews of this Package
| Version | Hash | Allocated | Completed | Reviewer | Status | Details |
|---|---|---|---|---|---|---|
| 1.4-2 | 32fcba00… | 2026-10-14 21:38 | 2026-10-14 22:29 | awm | accepted | VIEW |