[Mesa-dev] robust-tarballs branch (Was: Error building on Windows with SCons)
dbn.lists at gmail.com
Tue Jul 12 05:55:56 PDT 2011
On Sat, Jul 9, 2011 at 6:03 AM, Jose Fonseca <jfonseca at vmware.com> wrote:
> I heard no concerns so I went ahead and made a branch where:
> - I removed GLUT
> - derived Mesa tarballs' file list from git ls-files.
> I've confirmed that both automake and scons+crossmingw32 build correctly on Linux.
> I'd like to merge to main if there are no objections.
Having a brief look through the commits, it looks like a lot of nice
cleanup. A couple comments.
Using the variable DIRECTORY is a little confusing since it's a
symlink. DIRLINK may be better.
- src/mesa/depend is in EXTRA_FILES. This seems like something that
shouldn't be shipped.
- You're passing "-x autogen.sh" to not ship autogen.sh, but is that
the only thing that shouldn't be disted? Any chance you could compare
an old tarball to a new tarball and see if any unintended new files
- This might be for another commit, but it would be nice to move that
to a separate variable. To start it could be hacky with the arguments
IGNORE_FILES = \
-x autogen.sh \
- $(LIBNAME).zip should depend on manifest.txt
Looks good otherwise, though.
More information about the mesa-dev