<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/3.26.0">
</HEAD>
<BODY>
On Tue, 2011-01-11 at 13:37 +0000, Jon TURNEY wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
On 05/01/2011 21:05, Gaetan Nadon wrote:
&gt; On Wed, 2011-01-05 at 20:30 +0000, Jon TURNEY wrote:
&gt; 
&gt;&gt; On 05/01/2011 14:25, Daniel Stone wrote:
&gt;&gt;&gt; Previously the XWin build required a live internet connection (and
&gt;&gt;&gt; Khronos's servers to not be down for maintenance, etc) in order to
&gt;&gt;&gt; download the (W)GL specs to autogenerate files from.  Even if you
&gt;&gt;&gt; weren't building XWin, this broke distcheck.
&gt;&gt;&gt;
&gt;&gt;&gt; It's a fairly upsetting diffstat, but not being able to distcheck on
&gt;&gt;&gt; the train makes me sad-in-pants.
&gt;&gt;&gt;
&gt;&gt;&gt; Signed-off-by: Daniel Stone &lt;daniel-rLtY4a/8tF1rovVCs/<A HREF="mailto:uTlw-XMD5yJDbdMToUJ55TTLPZA@public.gmane.org.org">uTlw-XMD5yJDbdMToUJ55TTLPZA@public.gmane.org.org</A>&gt;
&gt;&gt;&gt; ---
&gt;&gt;&gt;
&gt;&gt;&gt; Note that this is not the real patch: SMTP doesn't allow even 1024-character
&gt;&gt;&gt; lines, let alone 31488 characters (sigh).  The real patch can be found at:
&gt;&gt;&gt; git://people.freedesktop.org/~daniels/xserver in the master branch
&gt;&gt;&gt;
&gt;&gt;&gt; But you get the idea.
&gt;&gt;
&gt;&gt; Yes, this is pretty bad atm.  I'm not sure why I did things this way in the
&gt;&gt; first place.  I think perhaps I was concerned about that I didn't know the
&gt;&gt; license terms for those files and didn't want to get stuck with manually
&gt;&gt; updating them.
&gt; 
&gt; It looks promising as the terms of the license are copied from the X11
&gt; license.
&gt; <A HREF="http://oss.sgi.com/projects/FreeB/SGIFreeSWLicB.2.0.pdf">http://oss.sgi.com/projects/FreeB/SGIFreeSWLicB.2.0.pdf</A>.
&gt; 
&gt; The build reproducibility is important. Various distros building X must
&gt; do so from the same source code.
&gt; Those building from tarballs expect the source code to be complete.
&gt; One should be able to rebuild a 2 year old server with the same 2 year
&gt; old gl spec.

I'm not sure I totally buy this argument.  We don't include stdio.h to ensure
that we can build with the version that was installed 2 years ago :-)

(Although I'm sure this problem is solved by a package build system which
audits the build dependencies)

</PRE>
</BLOCKQUOTE>
It's all a matter of risk evaluation, how much you trust where the source code it.<BR>
A download as you go approach is not practical for many due to network issues.<BR>
In addition, you don't have versioning, so you can never be sure to be able<BR>
to rebuild previous versions. You can't even be sure there will be anything to download.<BR>
<BR>
Having said this, I am not in a position to evaluate the risks and decide if they are acceptable<BR>
or not. The patch you attached makes sense, it is similar to so many &quot;dev&quot; packages that<BR>
are required to build some corners of xorg.<BR>
<BR>
I appreciate you have taken the time to evaluate these issues.<BR>
<BR>
Acked-by: <TT><FONT COLOR="#1a1a1a">Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt;</FONT></TT>
<BLOCKQUOTE TYPE=CITE>
<PRE>
&gt;&gt; Perhaps it would be better to simply remove these files, and I should just
&gt;&gt; create a package for cygwin containing these files, and then make that a
&gt;&gt; build-time requirement?

It makes me very uncomfortable to add a copy of those files, so here is an
alternative patch which removes the downloading of the files, which hopefully
fixes the distcheck problem, and the ~2.5 people who actually build XWin can
ensure they are made available in an appropriate way.

(I had a look at how Wine does this, as it needs to generate these same
wrappers with the opposite thunking, to see if it did this in a better way,
but unfortunately, that downloads the files also :-( [1] )

[1] <A HREF="http://source.winehq.org/git/wine.git/?a=tree;f=dlls/opengl32">http://source.winehq.org/git/wine.git/?a=tree;f=dlls/opengl32</A>
_______________________________________________
<A HREF="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</A>: X.Org development
Archives: <A HREF="http://lists.x.org/archives/xorg-devel">http://lists.x.org/archives/xorg-devel</A>
Info: <A HREF="http://lists.x.org/mailman/listinfo/xorg-devel">http://lists.x.org/mailman/listinfo/xorg-devel</A>
</PRE>
</BLOCKQUOTE>
</BODY>
</HTML>