Remove the -1 option from the installer.
authorMatthias Schmidt <matthias@dragonflybsd.org>
Tue, 16 Sep 2008 11:21:31 +0000 (16 11:21 +0000)
committerMatthias Schmidt <matthias@dragonflybsd.org>
Tue, 16 Sep 2008 11:21:31 +0000 (16 11:21 +0000)
This option is specific to the old, now removed, ISC dhcp client.  The new
OpenBSD dhcp client does not support this option.  If a user wants to set an
IP address with the installer prior to this commit, it fails and the user sees
an error message.

The meaning of the -1 option was: "Try to get a lease only once.  If it fails,
quit."  Due to this commit, this behaviour changes.  I think its better to have
an installer which tries to get a lease more than once that having an installer
which does not try to get a lease at all.

contrib/bsdinstaller-1.1.6/src/backend/installer/fn_configure.c

index 10206d3..f3160b3 100644 (file)
@@ -970,7 +970,7 @@ fn_assign_ip(struct i_fn_args *a)
                cmd = command_add(cmds, "%s%s dhclient",
                    a->os_root, cmd_name(a, "KILLALL"));
                command_set_failure_mode(cmd, COMMAND_FAILURE_IGNORE);
-               command_add(cmds, "%s%s -1 %s",
+               command_add(cmds, "%s%s %s",
                    a->os_root, cmd_name(a, "DHCLIENT"),
                    interface);
                if (commands_execute(a, cmds)) {