[libvirt] [PATCH v3] BSD: implement virNetDev(Set|Clear)IPv4Address
Roman Bogorodskiy
bogorodskiy at gmail.com
Thu Aug 22 14:25:33 UTC 2013
Roman Bogorodskiy wrote:
> Eric Blake wrote:
>
> > On 08/11/2013 07:54 AM, Roman Bogorodskiy wrote:
> > > Provide an implementation of virNetDev(Set|Clear)IPv4Address based on
> > > BSD ifconfig tool in addition to 'ip' from Linux iproute2 package.
> > > ---
> > > configure.ac | 15 +++++++++++++++
> > > src/util/virnetdev.c | 24 ++++++++++++++++++++++++
> > > 2 files changed, 39 insertions(+)
> > >
> >
> > >
> > > if test $with_freebsd = yes; then
> > > + want_ifconfig=yes
> > > +
> > > with_firewalld=no
> > > fi
> > >
> > > @@ -2429,6 +2435,15 @@ AC_CHECK_DECLS([BRDGSFD, BRDGADD, BRDGDEL],
> > > #include <net/if_bridgevar.h>
> > > ])
> > >
> > > +# Check if we need to look for ifconfig
> > > +if test "$want_ifconfig" = "yes"; then
> > > + AC_PATH_PROG([IFCONFIG_PATH], [ifconfig])
> > > + if test -z "$IFCONFIG_PATH"; then
> > > + AC_MSG_ERROR([Failed to find ifconfig.])
> >
> > This means that configure will fail if ifconfig is not installed but
> > want_ifconfig was set. Are you certain enough that FreeBSD is likely to
> > have ifconfig by default, and thus not hit this failure except on
> > extremely unlikely configurations?
>
> ifconfig is a part of FreeBSD base system. And appears it's not even
> possible to disable installing it via make.conf [1]. So I doubt there
> are a lot of systems without ifconfig, and even if they exist I'm not
> sure we need to support them because, as you said, it's extremely
> unlikely configuration.
>
> 1: http://www.freebsd.org/cgi/man.cgi?query=make.conf&sektion=5
One correction:
The proper man would be:
http://www.freebsd.org/cgi/man.cgi?query=src.conf
It's possible to build using WITHOUT_INET to disable IPv4 networking,
but again, I doubt that we should support such systems.
Roman Bogorodskiy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130822/218da4db/attachment-0001.sig>
More information about the libvir-list
mailing list