Hrm, good catch. I'm not sure how I missed that. I'm removing this patch for my next round.
Thanks! Phillip On Fri, Feb 5, 2021 at 6:56 AM Thomas Huth <th...@redhat.com> wrote: > On 04/02/2021 17.25, phillip.en...@gmail.com wrote: > > From: Phillip Tennen <phil...@axleos.com> > > > > Although we already have CONFIG_BSD, I added this flag to be sure that > we could rely on various macOS-specific subsystems, such as vmnet.framework. > > > > Signed-off-by: Phillip Tennen <phil...@axleos.com> > > --- > > configure | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/configure b/configure > > index 87de49e2c2..4afd22bdf5 100755 > > --- a/configure > > +++ b/configure > > @@ -5825,6 +5825,10 @@ if [ "$bsd" = "yes" ] ; then > > echo "CONFIG_BSD=y" >> $config_host_mak > > fi > > > > +if [ "$darwin" = "yes" ] ; then > > + echo "CONFIG_DARWIN=y" >> $config_host_mak > > +fi > > Hi! > > We already have these lines in "configure": > > if test "$darwin" = "yes" ; then > echo "CONFIG_DARWIN=y" >> $config_host_mak > fi > > ... so your patch here looks pretty redundant? > > Thomas > >