Re: [PATCH] Allow building lxc without virt-login-shell

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 20.11.2015 16:42, Cedric Bosdonnat wrote:
> On Fri, 2015-11-20 at 16:29 +0100, Michal Privoznik wrote:
>> On 10.11.2015 00:28, Cédric Bosdonnat wrote:
>>> Add a configure option to disable virt-login-shell build even if lxc is
>>> enabled.
>>> ---
>>>  configure.ac      | 14 ++++++++++++++
>>>  tools/Makefile.am | 12 ++++++------
>>>  2 files changed, 20 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/configure.ac b/configure.ac
>>> index f481c50..c766351 100644
>>> --- a/configure.ac
>>> +++ b/configure.ac
>>> @@ -1074,6 +1074,19 @@ if test "$with_lxc" = "yes" ; then
>>>  fi
>>>  AM_CONDITIONAL([WITH_LXC], [test "$with_lxc" = "yes"])
>>>  
>>> +AC_ARG_WITH([login_shell],
>>> +  [AS_HELP_STRING([--with-login-shell],
>>> +    [build virt-login-shell @<:@default=yes@:>@])])
>>> +m4_divert_text([DEFAULTS], [with_login_shell=yes])
>>> +
>>> +if test "$with_lxc" != "yes" ; then
>>> +    with_login_shell="no"
>>
>> what if somebody calls ./configure --without-lxc --with-login-shell?
>> Should we fail explicitly instead of silently disabling login shell?
> 
> No idea what is the best thing to do... but surely an error would be
> more explicit indeed.

I vote for explicit erroring out on conflicting arguments. So if you fix
that you got my ACK.

Michal

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]