[Bug 2053254] Re: Update urdfdom, dart and ignition-physics for Noble (Open Robotics packages)

[Lucas Kanashiro]

Hi Jose,

Thanks for the updated packages! I think you also noticed that two
packages there, in the PPA, are failing to build from source in s390x,
right? We need to fix that because s390x is a officially supported
architecture. Could you please take a look to see if you can identify
any issue?

For dart we have some failing tests in s390x as you can see below:

93% tests passed, 5 tests failed out of 71

Total Test time (real) =   6.01 sec

The following tests FAILED:
	 25 - test_MjcfParser (Subprocess aborted)
	 26 - test_SdfParser (SEGFAULT)
	 29 - test_DartLoader (Failed)
	 30 - test_IkFast (Failed)
	 42 - test_ForwardKinematics (SEGFAULT)

For ignition-physics, we have a missing build dependency:

Missing build dependencies: libignition-common-dev (>= 4.0.0)

If you check the version of this package in Noble you will notice that
it should be satisfiable:

 libignition-common-dev | 4.7.0+ds2-2ubuntu1 | noble/universe  | amd64,
arm64, armhf, ppc64el, riscv64

However, src:ignition-common which provides this library is FTBFSing in
s390x:

https://launchpad.net/ubuntu/+source/ignition-common
https://launchpad.net/ubuntu/+source/ignition-common/4.7.0+ds2-2ubuntu1/+build/27670232

I am not sure how exactly this migrated to the release pocket but this
require some extra investigation. Ideally, everything should be building
on s390x. Could you please try to understand what is happening here?

Regarding the dart armhf unsupportability, if upstream says it is not
supported, you should change the Architecture fields in debian/control
to reflect that. If you leave arch:any then it will be built in all
architectures, you'd need to exclude armhf.

I did not do a in-deep review but I think the above is already enough
for you (and the main points to get them uploaded). I see some other
things like a description in the source paragraph in debian/control
which seems weird to me but those are more "cosmetic" things, since this
is also in Debian I'd not say this is a blocker.

I am now unsubscribing ~ubuntu-sponsors. Once you addressed the comments
above and you think they are ready for another round of review, please,
subscribe ~ubuntu-sponsors again.

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/2053254

Title:
  Update urdfdom, dart and ignition-physics for Noble (Open Robotics
  packages)

Status in dart package in Ubuntu:
  New

Bug description:
  Versions bump for the Open Robotics Community packages:
   - urdfdom to 4.0.0-0ubuntu1
   - dart to 6.13.1+ds-0ubuntu2

  As a side effect ignition-physics package needs also to be patch updated:
   - ignition-physics to 5.1.0+ds1-4.1ubuntu2

  Package PPA:
   https://launchpad.net/~j-rivero/+archive/ubuntu/urdfdom4-noble/+packages
   (let me know if you prefer the code in bzr)

  [Other info]

  I have both version bumps ready on Debian but the 64bit_time transition[1] is preventing the
  packages to go through the lib transition since two weeks ago and counting. As we want to
  have these new versions in 24.04 (feature freeze is coming soon), we do the version bump
  directly on Ubuntu.

  [Justification]

  Important updates in a couple of packages used by the Open Robotics
  Community, particularly affecting ROS (Robot Operative System) and the
  ignition/Gazebo simulator.

   * urdfdom is changing the ABI/API completely by replacing tinyxml with tinyxml2
   * Dart current version 6.9.2 was released in .. 2019 :(

  Both packages should be lintian clean.

  [Transition details]

  The PPA includes all direct dependencies of urfdom and DART (ignition-
  physics and ros-urdf).

   * ignition-physics include a patch to work with new DART adding conditional code to deal with
     minor DART API renames.

  All the previous architectures where the software was building before
  are also supported in the update with an important exception: new DART
  upstream releases do not support 32 bits, so armhf is lost from the
  list.

  [1] https://lists.debian.org/debian-devel-
  announce/2024/02/msg00000.html

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/dart/+bug/2053254/+subscriptions