Hello,
On Fri, Feb 19, 2021 at 10:45:41AM +0100, Sven Mueller wrote:
> Package: xapp
> Version: 2.0.6-1
> Severity: serious
>
> xapps-common is tagged as architecture:all, but the generated
> xapp-sn-watcher.desktop included in it depends on the architecture it was
> built on.
[...]
> 1) Move the autostart file into the respective libxapp1 packages (with the
> binary)
[...]
Seems like Fabio Fantoni went with this solution and while I agree
that desktop files should be shipped in the same package as the binary
they launch in general, I also think it's very wrong to ship
non-SO-verioned files in a libfooN package!
(You'll most likely cause a file conflict bug when you later move
to libxapp2 in the future.... the entire reason we put the SO version
in the package name is to make the different versions co-installable,
but if you put non-versioned files in the package then you're breaking
that....)
The executable and desktop file should really be in a different
(possibly newly added) package!
Given we're in freeze it might be too late to introduce a new binary
package (if that's needed), but fixing one bug by doing something
wrong doesn't feel like debian style either. Wouldn't it be better
to just ask the release team which solution they prefer and possibly
get an exception for introducing a new binary package if needed?
Regards,
Andreas Henriksson