[Mesa-dev] robust-tarballs branch (Was: Error building on Windows with SCons)

Dan Nicholson dbn.lists at gmail.com
Tue Jul 12 05:55:56 PDT 2011

Hi Jose,

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.
> http://cgit.freedesktop.org/mesa/mesa/log/?h=robust-tarballs
> 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
slipped in?
- 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

        -x autogen.sh \
        -x otherunwantedfile

- $(LIBNAME).zip should depend on manifest.txt

Looks good otherwise, though.


More information about the mesa-dev mailing list