After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 679428 - dhcp: do not require dhclient-4.x to be built with DHCPv6
dhcp: do not require dhclient-4.x to be built with DHCPv6
Status: RESOLVED FIXED
Product: NetworkManager
Classification: Platform
Component: general
0.9.x
Other Linux
: Normal normal
: ---
Assigned To: Pavel Simerda
Dan Williams
Depends on:
Blocks:
 
 
Reported: 2012-07-05 02:24 UTC by Alexandre Rostovtsev
Modified: 2012-12-27 03:35 UTC
See Also:
GNOME target: ---
GNOME version: 3.3/3.4


Attachments
proposed patch (1.20 KB, patch)
2012-07-05 02:25 UTC, Alexandre Rostovtsev
none Details | Review

Description Alexandre Rostovtsev 2012-07-05 02:24:22 UTC
(As reported downstream at https://bugs.gentoo.org/show_bug.cgi?id=335147)

Currently, NM calls dhclient with the "-6" flag when requesting an ipv6 address, and with the "-4" flag when requesting ipv4.

However, if dhclient-4.x was built with --enable-dhcpv6, the "-4" flag is the default behavior, and does not need to be specified manually. And if dhclient-4.x was built with --disable-dhcpv6, the "-4" flag is invalid, and results in an error.

So NM should simply never use the "-4" flag when calling dhclient; this will allow it to support dhclient-4.x whether or not it was built with DHCPv6 support.
Comment 1 Alexandre Rostovtsev 2012-07-05 02:25:08 UTC
Created attachment 218067 [details] [review]
proposed patch
Comment 2 Pavel Simerda 2012-07-25 17:17:15 UTC
“And if dhclient-4.x was built with --disable-dhcpv6, the "-4" flag is invalid, and results in an error.”

AFAIK this should be fixed in dhclient and not NetworkManager as we are using
valid command-line interface.
Comment 3 Pavel Simerda 2012-07-25 17:23:18 UTC
Information about reporting bugs on dhclient:

https://www.isc.org/software/dhcp/news
Comment 4 Dan Williams 2012-09-07 21:59:18 UTC
I'd agree that it should be fixed upstream in dhclient, but if they really, really don't want to do that, I'm open to taking the proposed patch.
Comment 5 Pavel Simerda 2012-11-04 14:35:50 UTC
Changed my mind after speaking with Michal Hrušecký and Amy Winston. When -4 is redundant anyway, and as I have no information that dhclient would be changing
its default, it might be safe to remove the -4 flag and make life easier to
those distributions.

I don't have a strong opinion on this in any direction.

Patch here:

http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/net-misc/networkmanager/files/networkmanager-0.9.4.0-dhclient-ipv6.patch?view=markup
Comment 6 Pavel Simerda 2012-11-07 22:35:34 UTC
Added to 'pavlix/dhcp' and will be merged eventually.
Comment 7 Dan Williams 2012-12-27 03:35:22 UTC
a52fba46fecd214ad1ae3378c44328565240776a