Re: [ptxdist] iproute2 dependencies

2016-10-17 Thread Clemens Gruber
On Mon, Oct 17, 2016 at 03:21:08PM +0200, Michael Olbrich wrote:
> Clemens: I think a patch that adds xtables-version.h is the cleanest
> solution. The resulting m_xt.so will be broken because it doesn't link to
> libiptables but we don't install that right now.

Good idea! I'll send a patch shortly. (on top of the 4.8 version bump)

Thanks,
Clemens

___
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] iproute2 dependencies

2016-10-17 Thread Michael Olbrich
Hi,

On Mon, Oct 17, 2016 at 03:13:49PM +0200, Michael Olbrich wrote:
> On Mon, Oct 17, 2016 at 02:54:51PM +0200, Uwe Kleine-König wrote:
> > On Mon, Oct 17, 2016 at 12:48:37PM +0200, Clemens Gruber wrote:
> > > What's also interesting to me is that Debian does not depend on
> > > libxtables11, it is only recommended.
> > Looking at iproute2 4.6.0-4, it build-depends on iptables-dev though. So
> > maybe only the headers are needed, but not the library?!
> > 
> > In the changelog there is:
> > 
> > iproute (20121001-2) experimental; urgency=low
> > 
> >   * Generate recommends for optional module dependencies
> > This makes the optional module dependency on libxtables more
> > visible, to hopefully get iproute rebuilt whenever libxtables bumps
> > soname.
> > Thanks to iptables maintainer for splitting libxtables9 into
> > a separate package in version 1.4.16.3-2.
> > 
> > Maybe this is also related?
> 
> Somewhat. The situation is like this:
> iproute has plugins that are loaded at runtime with dlopen. Some of them
> need libxtables. So unless these plugins are actually used libxtables is
> not needed. So libxtables a optional _runtime_ dependency but to build a
> working plugin its a required _buildtime_ dependency.
> 
> The current code is rather broken. It has some imported iptables headers
> (from version 1.6.0), but those are useless because xtables-version.h is
> missing...

Clemens: I think a patch that adds xtables-version.h is the cleanest
solution. The resulting m_xt.so will be broken because it doesn't link to
libiptables but we don't install that right now.

If someone wants it we can add the dependency and install the plugin.

Michael

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

___
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] iproute2 dependencies

2016-10-17 Thread Michael Olbrich
Hi,

On Mon, Oct 17, 2016 at 02:54:51PM +0200, Uwe Kleine-König wrote:
> On Mon, Oct 17, 2016 at 12:48:37PM +0200, Clemens Gruber wrote:
> > What's also interesting to me is that Debian does not depend on
> > libxtables11, it is only recommended.
> Looking at iproute2 4.6.0-4, it build-depends on iptables-dev though. So
> maybe only the headers are needed, but not the library?!
> 
> In the changelog there is:
> 
> iproute (20121001-2) experimental; urgency=low
> 
>   * Generate recommends for optional module dependencies
> This makes the optional module dependency on libxtables more
> visible, to hopefully get iproute rebuilt whenever libxtables bumps
> soname.
> Thanks to iptables maintainer for splitting libxtables9 into
> a separate package in version 1.4.16.3-2.
> 
> Maybe this is also related?

Somewhat. The situation is like this:
iproute has plugins that are loaded at runtime with dlopen. Some of them
need libxtables. So unless these plugins are actually used libxtables is
not needed. So libxtables a optional _runtime_ dependency but to build a
working plugin its a required _buildtime_ dependency.

The current code is rather broken. It has some imported iptables headers
(from version 1.6.0), but those are useless because xtables-version.h is
missing...

Michael

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

___
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] iproute2 dependencies

2016-10-17 Thread Uwe Kleine-König
Hello,

On Mon, Oct 17, 2016 at 12:48:37PM +0200, Clemens Gruber wrote:
> What's also interesting to me is that Debian does not depend on
> libxtables11, it is only recommended.
Looking at iproute2 4.6.0-4, it build-depends on iptables-dev though. So
maybe only the headers are needed, but not the library?!

In the changelog there is:

iproute (20121001-2) experimental; urgency=low

  * Generate recommends for optional module dependencies
This makes the optional module dependency on libxtables more
visible, to hopefully get iproute rebuilt whenever libxtables bumps
soname.
Thanks to iptables maintainer for splitting libxtables9 into
a separate package in version 1.4.16.3-2.

Maybe this is also related?

Best regards
Uwe

-- 
Pengutronix e.K.   | Uwe Kleine-König|
Industrial Linux Solutions | http://www.pengutronix.de/  |

___
ptxdist mailing list
ptxdist@pengutronix.de