Bug#1103164: release.debian.org: Fixing a policy violation

2025-04-21 Thread Sebastian Ramacher
Control: tags -1 confirmed

On 2025-04-16 20:47:20 +0200, Leopold Palomo-Avellaneda wrote:
> Hi,
> 
> El 15/4/25 a les 8:42, Sebastian Ramacher ha escrit:
> [...]
> > 
> > This sounds good in principle, but please provide a debdiff between the
> > version in testing and what you are proposing to upload.
> 
> 
> Here, it is.

Please go ahead

Cheers
-- 
Sebastian Ramacher



Bug#1103164: release.debian.org: Fixing a policy violation

2025-04-16 Thread Leopold Palomo-Avellaneda

Hi,

El 15/4/25 a les 8:42, Sebastian Ramacher ha escrit:
[...]


This sounds good in principle, but please provide a debdiff between the
version in testing and what you are proposing to upload.



Here, it is.

Cheers,
diff -Nru ompl-1.6.0+ds1/debian/changelog ompl-1.6.0+ds1/debian/changelog
--- ompl-1.6.0+ds1/debian/changelog 2025-04-04 20:18:39.0 +0200
+++ ompl-1.6.0+ds1/debian/changelog 2025-04-12 14:33:05.0 +0200
@@ -1,3 +1,11 @@
+ompl (1.6.0+ds1-3~exp1) experimental; urgency=medium
+
+  * Bump ABI version. Upstream defined it as 17, not 16.
+  * Added autopkgtests.
+  * Pushed to experimental.
+
+ -- Leopold Palomo-Avellaneda   Sat, 12 Apr 2025 14:33:05 
+0200
+
 ompl (1.6.0+ds1-2) unstable; urgency=medium
 
   * Added patch to solve wrong Python detection on arm64 (Closes: #1091159)
diff -Nru ompl-1.6.0+ds1/debian/control ompl-1.6.0+ds1/debian/control
--- ompl-1.6.0+ds1/debian/control   2025-04-04 20:18:39.0 +0200
+++ ompl-1.6.0+ds1/debian/control   2025-04-12 14:33:05.0 +0200
@@ -32,9 +32,12 @@
 Package: libompl-dev
 Architecture: any
 Section: libdevel
-Depends: libompl16t64 (= ${binary:Version}),
- ${misc:Depends},
- libboost-dev,
+Depends: libompl17 (= ${binary:Version}),
+ ${misc:Depends}, 
+libopenmpi-dev,
+libboost-serialization-dev,
+libboost-filesystem-dev,
+libboost-system-dev,
  libeigen3-dev,
 libode-dev
 Suggests: pkgconf
@@ -47,10 +50,10 @@
  additional components. This package contains the development files
  needed to use the library.
 
-Package: libompl16t64
+Package: libompl17
 Provides: ${t64:Provides}
-Replaces: libompl16
-Breaks: libompl16 (<< ${source:Version})
+Replaces: libompl16t64
+Breaks: libompl16t64 (<< ${source:Version})
 Multi-Arch: same
 Architecture: any
 Depends: ${shlibs:Depends},
diff -Nru ompl-1.6.0+ds1/debian/libompl16t64.install 
ompl-1.6.0+ds1/debian/libompl16t64.install
--- ompl-1.6.0+ds1/debian/libompl16t64.install  2025-04-03 20:04:39.0 
+0200
+++ ompl-1.6.0+ds1/debian/libompl16t64.install  1970-01-01 01:00:00.0 
+0100
@@ -1,2 +0,0 @@
-usr/lib/*/lib*.so.*
-
diff -Nru ompl-1.6.0+ds1/debian/libompl17.install 
ompl-1.6.0+ds1/debian/libompl17.install
--- ompl-1.6.0+ds1/debian/libompl17.install 1970-01-01 01:00:00.0 
+0100
+++ ompl-1.6.0+ds1/debian/libompl17.install 2025-04-12 14:33:05.0 
+0200
@@ -0,0 +1,2 @@
+usr/lib/*/lib*.so.*
+
diff -Nru ompl-1.6.0+ds1/debian/tests/autopkgtest 
ompl-1.6.0+ds1/debian/tests/autopkgtest
--- ompl-1.6.0+ds1/debian/tests/autopkgtest 1970-01-01 01:00:00.0 
+0100
+++ ompl-1.6.0+ds1/debian/tests/autopkgtest 2025-04-12 14:33:05.0 
+0200
@@ -0,0 +1,26 @@
+#!/bin/sh
+set -e
+
+# Create a simple C++ source file that uses OMPL.
+cat <<'EOF' > test_ompl.cpp
+#include 
+#include 
+#include 
+
+int main() {
+// Create a 3-dimensional state space
+auto space = std::make_shared(3);
+std::cout << "Successfully created a " 
+  << space->getDimension() 
+  << "-dimensional state space." << std::endl;
+return 0;
+}
+EOF
+
+# Compile the test program.
+# It is best practice to use pkg-config so that the proper include and library 
flags are set.
+g++ -o test_ompl test_ompl.cpp $(pkg-config --cflags --libs ompl)
+echo "build: OK"
+[ -x test_ompl ]
+./test_ompl
+echo "run: OK"
diff -Nru ompl-1.6.0+ds1/debian/tests/control 
ompl-1.6.0+ds1/debian/tests/control
--- ompl-1.6.0+ds1/debian/tests/control 1970-01-01 01:00:00.0 +0100
+++ ompl-1.6.0+ds1/debian/tests/control 2025-04-12 14:33:05.0 +0200
@@ -0,0 +1,2 @@
+tests: autopkgtest
+depends: @, libompl-dev, g++


Bug#1103164: release.debian.org: Fixing a policy violation

2025-04-14 Thread Sebastian Ramacher
Control: tag -1 moreinfo

On 2025-04-14 20:35:02 +0200, Leopold Palomo-Avellaneda wrote:
> Package: release.debian.org
> Severity: normal
> 
> The package ompl is of a library. There was a mistake in the name of the
> package library assigning 16 (libompl16t64) when the ABI number was 17
> It is a policy violation because the package should be libompl17.
> 
> The library package has no others packages that depend on it, so no
> transition is necessary. I would like to ask for exception and let the
> version in experimental to go to unstable.

This sounds good in principle, but please provide a debdiff between the
version in testing and what you are proposing to upload.

Cheers
-- 
Sebastian Ramacher



Bug#1103164: release.debian.org: Fixing a policy violation

2025-04-14 Thread Leopold Palomo-Avellaneda

Package: release.debian.org
Severity: normal

The package ompl is of a library. There was a mistake in the name of the
package library assigning 16 (libompl16t64) when the ABI number was 17
It is a policy violation because the package should be libompl17.

The library package has no others packages that depend on it, so no
transition is necessary. I would like to ask for exception and let the
version in experimental to go to unstable.

Best regards

--
--
Linux User 152692 GPG: 05F4A7A949A2D9AA
Catalonia
-
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing in e-mail?