[Mesa-dev] [PATCH] build: Disable building of d3d1x

[Jose Fonseca]

Matt,

I see you went ahead and just disabled it. Please remove it all together.

Touching code that is not built nor tested ends just silently introduces bugs, so keeping this around won't help bring it back one day in any way.

Jose

----- Original Message -----
> Cristoph Bumiller was the last person to commit any changes to this,
> last December.
> 
> d3d1x is not built through scons either. If it is really broken and
> unmaintained then lets just remove it from the tree, as code will
> just rot. If an interested party comes along they can always refer
> to git history.
> 
> Jose
> 
> ----- Original Message -----
> > It's broken and unmaintained, and I'm tired of seeing bug reports
> > about
> > it.
> > ---
> >  configure.ac |   11 ++++++-----
> >  1 files changed, 6 insertions(+), 5 deletions(-)
> > 
> > diff --git a/configure.ac b/configure.ac
> > index 82fe225..5ea0752 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -563,11 +563,12 @@ AC_ARG_ENABLE([xa],
> >          [enable build of the XA X Acceleration API
> >          @<:@default=no@:>@])],
> >      [enable_xa="$enableval"],
> >      [enable_xa=no])
> > -AC_ARG_ENABLE([d3d1x],
> > -    [AS_HELP_STRING([--enable-d3d1x],
> > -        [enable support for Direct3D 10 & 11 low-level API
> > @<:@default=no@:>@])],
> > -    [enable_d3d1x="$enableval"],
> > -    [enable_d3d1x=no])
> > +dnl d3d1x is broken and unmaintained.
> > +dnlAC_ARG_ENABLE([d3d1x],
> > +dnl    [AS_HELP_STRING([--enable-d3d1x],
> > +dnl        [enable support for Direct3D 10 & 11 low-level API
> > @<:@default=no@:>@])],
> > +dnl    [enable_d3d1x="$enableval"],
> > +dnl    [enable_d3d1x=no])
> >  AC_ARG_ENABLE([gbm],
> >     [AS_HELP_STRING([--enable-gbm],
> >           [enable gbm library @<:@default=auto@:>@])],
> > --
> > 1.7.8.6
> > 
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> >