Debian package names seem not to follow Debian naming standards

[gavanderhoorn]

Not sure this isn't intentional, so I didn't want to open an issue immediately.

Debian packages are supposed to follow a rather strict naming scheme, otherwise various tools related to packaging and repository management can't process them correctly.

In practice, this means <PackageName>_<VersionNumber>-<DebianRevisionNumber>_<DebianArchitecture>.deb.

But the .debs distributed as part of the Tesseract Release archives don't appear to adhere to this. Example: tesseract-motion-planners-trajopt-ifopt_amd64_linux_0.29.1.deb.

The parts are there (except the OS in the version), but not in the right order and the 'punctuation' isn't the same.

It should be: tesseract-motion-planners-trajopt-ifopt_0.29.1-0jammy_amd64.deb.

Is the current naming the result of using CPack in a certain way to package for both Windows and Linux? Could it perhaps be tweaked to generate 'correct' Debian package names?

I ran into this when trying to import the .debs into a multi-distribution APT repository which didn't work because the package names for different distributions are identical.

[gavanderhoorn]

It's not just the filenames actually. The control file lists the Version as 0.29.1 (for esseract-motion-planners-trajopt-ifopt), instead of 0.29.1-0jammy.

It's not strictly necessary to include the codename in the version I believe, as long as the versions are unique (ie: 0.29.1 can't be the version on two different OS).

But Debian versioning is complex, so I may be wrong.

[marip8]

I think this issue actually stems from the macros for using CPack defined in ros_industrial_cmake_boilerplate here and here

[gavanderhoorn]

Right. This would indeed be responsible for overriding the generated filename.

The default value would already seem to come closer to what's standard.

[Levi-Armstrong]

@marip8 and @gavanderhoorn Should we just remove that line from ros_industrial_cmake_boilerplate and let it auto generate it?

[gavanderhoorn]

Maybe, although perhaps this line would be better. That sets CPACK_DEBIAN_PACKAGE_NAME.

I haven't checked whether that improves anything though.