Hi Jose,
On Sat, Jul 9, 2011 at 6:03 AM, Jose Fonseca <jfons...@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.
26edecac589819f0d0efe2165ab748dbc4e53394:
Using the variable DIRECTORY is a little confusing since it's a
symlink. DIRLINK may be better.
34983337f9d7db984e9f0117808274106d262110:
- 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
embedded:
IGNORE_FILES = \
-x autogen.sh \
-x otherunwantedfile
- $(LIBNAME).zip should depend on manifest.txt
Looks good otherwise, though.
--
Dan
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
