Anthony -

I've committed 3/4 of your mentioned issues. Personally, I'd love it if you
could make "make dist" work right. I'll try to keep an eye out for a patch
on this topic.

-Bryan

On Sun, May 2, 2010 at 12:52 PM, Anthony Molinaro <
anthonym@alumni.caltech.edu> wrote:

> I've noticed that as well, I even looked into it but haven't patched it.
> The issue is the compiler would like to announce what it's version it so
> it needs that version in a header file.  Currently the system relies
> on make to build that header (see compiler/ccp/Makefile.am).
>
> Instead you want to have the version set in one place configure.ac and
> have everything else trigger of that at configure time.  This would
> probably mean deprecating print_version.sh as it is only used to
> create that one file (which would also mean deprecating the addition
> of svn/git revision number in that file as well, but that really shouldn't
> be a problem, now that you are onto real releases).
>
> I'd be willing to fix this stuff up for the project, if there's a chance
> the patch might make it through (none of my other 4 patches for the
> build system THRIFT-646, THRIFT-647, THRIFT-649, THRIFT-650 every made
> it through).
>
> Other oddness which could probably be done away with is the cleanup.sh
> script, which 'make maintainer-clean' should basically be doing for you.
>
> Anyway, let me know if there's interest in a patch to allow './bootstrap.sh
> &&
> ./configure && make dist' to work.
>
> Cheers,
>
> -Anthony
>
> On Sat, May 01, 2010 at 08:57:01PM -0700, Bryan Duxbury wrote:
> > If I do "./bootstrap.sh && ./configure && make dist", I get errors:
> >
> > [bryanduxbury thrift-0.3]$make dist
> > { test ! -d thrift-0.3.0 || { find thrift-0.3.0 -type d ! -perm -200
> -exec
> > chmod u+w {} ';' && rm -fr thrift-0.3.0; }; }
> > test -d thrift-0.3.0 || mkdir thrift-0.3.0
> > list='compiler/cpp lib test'; for subdir in $list; do \
> >   if test "$subdir" = .; then :; else \
> >     test -d "thrift-0.3.0/$subdir" \
> >     || ./install-sh -c -d "thrift-0.3.0/$subdir" \
> >     || exit 1; \
> >     distdir=`CDPATH="${ZSH_VERSION+.}:" && cd thrift-0.3.0 && pwd`; \
> >     top_distdir=`CDPATH="${ZSH_VERSION+.}:" && cd thrift-0.3.0 && pwd`; \
> >     (cd $subdir && \
> >       make  \
> >         top_distdir="$top_distdir" \
> >         distdir="$distdir/$subdir" \
> >         am__remove_distdir=: \
> >         am__skip_length_check=: \
> >         distdir) \
> >       || exit 1; \
> >   fi; \
> > done
> > \
> > /bin/sh ../../ylwrap `test -f 'src/thriftl.ll' || echo
> './'`src/thriftl.ll
> > lex.yy.c thriftl.cc -- flex
> > \
> > /bin/sh ../../ylwrap `test -f 'src/thrifty.yy' || echo
> './'`src/thrifty.yy
> > y.tab.c thrifty.cc y.tab.h thrifty.h y.output thrifty.output -- bison -y
>  -d
> > updating thrifty.h
> > make[1]: *** No rule to make target `version.h', needed by `src/main.cc'.
> > Stop.
> > make: *** [distdir] Error 1
> >
> > On Sat, May 1, 2010 at 8:12 PM, Todd Lipcon <todd@cloudera.com> wrote:
> >
> > > The tarball seems to include compiled .o files.
> > >
> > > I think the build process is just make dist, without the preceding
> make.
> > >
> > > -Todd
> > >
> > > On Sat, May 1, 2010 at 10:45 AM, Bryan Duxbury <bryan@rapleaf.com>
> wrote:
> > >
> > > > I bootstrapped on my Mac.
> > > >
> > > > On Sat, May 1, 2010 at 9:09 AM, David Reiss <dreiss@facebook.com>
> wrote:
> > > >
> > > > > What system did you run bootstrap on?
> > > > >
> > > > >
> > > > > --David (mobile)
> > > > >
> > > > > On May 1, 2010, at 6:53 AM, "Bryan Duxbury" <bryan@rapleaf.com>
> wrote:
> > > > >
> > > > > > Hi all,
> > > > > >
> > > > > > I'd like to propose the following tarball be accepted as the
> > > > > > official Thrift
> > > > > > 0.3.0 release:
> > > > > >
> > > > > > http://people.apache.org/~bryanduxbury/thrift-0.3.0-rc1.tar.gz<http://people.apache.org/%7Ebryanduxbury/thrift-0.3.0-rc1.tar.gz>
> <http://people.apache.org/%7Ebryanduxbury/thrift-0.3.0-rc1.tar.gz>
> > > <
> > > > http://people.apache.org/%7Ebryanduxbury/thrift-0.3.0-rc1.tar.gz>
> > > > > <http://people.apache.org/%7Ebryanduxbury/thrift-0.3.0-rc1.tar.gz
> > > > > > >
> > > > > >
> > > > > > with GPG signature:
> > > > > >
> http://people.apache.org/~bryanduxbury/thrift-0.3.0-rc1.tar.gz.asc<http://people.apache.org/%7Ebryanduxbury/thrift-0.3.0-rc1.tar.gz.asc>
> <http://people.apache.org/%7Ebryanduxbury/thrift-0.3.0-rc1.tar.gz.asc>
> > > <
> > > > http://people.apache.org/%7Ebryanduxbury/thrift-0.3.0-rc1.tar.gz.asc
> >
> > > > > <
> http://people.apache.org/%7Ebryanduxbury/thrift-0.3.0-rc1.tar.gz.asc
> > > > > > >
> > > > > > and MD5 sum: 677607c0b5acb89fb35386169c7ffa8d
> > > > > >
> > > > > > This tarball was built by checking out
> > > > > > http://svn.apache.org/repos/asf/incubator/thrift/tags/0.3.0 and
> > > > > > running
> > > > > > "./boostrap.sh && ./configure && make && make dist".
> > > > > >
> > > > > > Please download, untar, configure, and build to verify that
> > > > > > everything works
> > > > > > as expected. Sorry about the false start.
> > > > > >
> > > > > > -Bryan
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Todd Lipcon
> > > Software Engineer, Cloudera
> > >
>
> --
> ------------------------------------------------------------------------
> Anthony Molinaro                           <anthonym@alumni.caltech.edu>
>