[PATCH:macros] doc: add XORG_CHECK_SGML_DOCTOOLS to detect xorg-sgml-doctools

[Gaetan Nadon]

On Fri, 2010-04-09 at 06:11 -0700, Dan Nicholson wrote:

> On Thu, Apr 8, 2010 at 11:22 PM, Yaakov (Cygwin/X)
> <yselkowitz at users.sourceforge.net> wrote:
> > On 2010-04-08 08:14, Dan Nicholson wrote:
> >>
> >> Huh, I hadn't noticed that. I think we should demand that you have the
> >> version with the .pc file once there's a release. For the very few
> >> people who are going to attempt to build the sgml documentation, I
> >> think they can be bothered to grab a newer doctools package.
> >
> > That would be easier, and indeed the first version of my patch did just
> > that, but Gaetan pointed out that this would cause a regression if macros is
> > updated but not doctools.  Whether or not that is acceptable as a tradeoff
> > for the functionality of using pkg-config needs to be decided.
> 
> As long as someone makes a release of the doctools, I'd be willing to
> make that sacrifice.
> 
> 1. The number of people actually building the sgml docs is likely
> extremely small and probably is encompassed by x developers trying to
> make a katamari release and a handful of distro packagers. I think
> they can be bothered to update one more package that has 0
> dependencies.
> 
> 2. The gain in proper versioning outweighs the loss of backwards
> compatibility in my mind.
> 
> 3. The time when people will be attempting to build the docs is likely
> just before or after the next Xorg release. They'll be wanting to
> download the new doctools for the updated entities anyway.
> 

I don't have objections in this case to somewhat break compatibility. I
routinely test for such a scenario when making changes as I have been
reminded that changes and new features in macros should not adversely
affect current or previous versions of clients. There is some leeway
here as long as it does not just plain break and is easy to figure out.
It's a judgement call. 



> --
> Dan
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100409/f944f411/attachment.htm>
-------------- 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/20100409/f944f411/attachment.pgp>