Control: tags -1 + pending
Hi again,
2017-11-20 20:49 Helmut Grohne:
On Mon, Nov 20, 2017 at 01:23:07AM +0100, Manuel A. Fernandez Montecelo wrote:
https://anonscm.debian.org/cgit/collab-maint/libvoikko.git/commit/?id=94e7ce125739e59241c04034f5b0615e93c3cc49
Please review. Explanations follow.
I don't think that this split makes sense as is.
Either you mark the new package M-A:foreign or you shouldn't split. If
some other package needs those tools, it certainly wants to execute
them. Thus it will need the build architecture version. In
build-depends, this can be achieved by annotating with :native or by
marking the package M-A:foreign (and the latter is wrong here). Thus
rdeps will have to put up with :native and that means you can simply
keep them together.
So marking the new package M-A:foreign may be an option. I'm not sure
whether it is correct. To figure out, someone should check whether the
tools *behave* the same on different architectures. If you feed them the
same input (input files, command line flags, environment, etc.), will
they have the same output? If the answer is "always yes", then
M-A:foreign is appropriate.
If M-A:foreign ends up being wrong, I'd not split at all, but simply
mark the -dev package M-A:no and update the relevant rdeps to use
:native.
So I think that this is what we're going to do now as the safest route,
I plan to NMU with this change.
If M-A:foreign ends up being correct, libvoikko-dev should actually
depend on the new package. Thus the full interface formerly provided by
libvoikko-dev keeps being provided by virtue of the dependency. But the
M-A headers will ensure that the library is for the host architecture
and the tools are for the build architecture. Consequently, I'd name the
package "libvoikko-dev-bin" and describe it as an implementation detail
of libvoikko-dev that you shouldn't use directly.
I think that it's worth trying that as it's a better solution, but it
needs understanding from the maintainer or someone who really groks the
tools and knows how the reverse-depends use them.
I reported another bug #882273 for that purpose.
Cheers.
--
Manuel A. Fernandez Montecelo <[email protected]>
