[PATCH] pciaccess: remove xorg macros dependency

Eric Anholt eric at anholt.net
Wed Jul 29 16:02:28 PDT 2009


On Fri, 2009-07-17 at 02:05 +1000, Daniel Stone wrote:
> On Thu, Jul 16, 2009 at 08:27:12AM -0700, Eric Anholt wrote:
> > On Thu, 2009-07-16 at 14:18 +0300, Tiago Vignatti wrote:
> > > -m4_ifndef([XORG_MACROS_VERSION], [AC_FATAL([must install xorg-macros 1.2 or later before running autoconf/autogen])])
> > > -XORG_MACROS_VERSION(1.2)
> > >  AM_CONFIG_HEADER(config.h)
> > >  
> > >  # Check for progs
> > >  AC_PROG_CC
> > >  AC_PROG_LIBTOOL
> > > -XORG_CWARNFLAGS
> > >  
> > >  pciids_path=/usr/share/hwdata
> > >  AC_ARG_WITH(pciids-path, AC_HELP_STRING([--with-pciids-path=PCIIDS_PATH],
> > > @@ -108,9 +104,6 @@ AC_CHECK_MEMBER([struct pci_io.pi_sel.pc_domain],
> > >  AC_SUBST(PCIACCESS_CFLAGS)
> > >  AC_SUBST(PCIACCESS_LIBS)
> > >  		  
> > > -XORG_RELEASE_VERSION
> > > -XORG_CHANGELOG
> > > -
> > >  AC_OUTPUT([Makefile
> > >  	   src/Makefile
> > >             pciaccess.pc])
> > 
> > NAK.
> > 
> > Reducing the functionality of the package for no reason.
> 
> Agreed, but shouldn't XORG_RELEASE_VERSION be removed in general?

Yeah, XORG_RELEASE_VERSION is terrible, and I think XORG_CHANGELOG is
pretty bogus, too.  Removing the warning cflags is what I don't like.

-- 
Eric Anholt
eric at anholt.net                         eric.anholt at intel.com


-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
Url : http://lists.x.org/archives/xorg-devel/attachments/20090729/2f078e86/attachment.pgp 


More information about the xorg-devel mailing list