[Freeipa-devel] Re: [PATCH] build fixes

[William Jon McCann]

On 10/18/07, Karl MacMillan <kmacmill at redhat.com> wrote:
> This patch fixes the dist and local-dist targets from the recent
> autoconf import. It also makes some minor fixes to the spec files.
>
> One remaining question: when I make the source tarballs for distribution
> I'm running autogen.sh. However, that runs configure which, I think, is
> not typically run before distributing the source. Should I just remove
> the configure step from autogen.sh?

No.  At least in my experience the proper way to do a release is to:

Pull a fresh tree  or run make maintainer-clean (does not have
configure or Makefile)
Run autogen.sh (which builds configure, runs configure, creates Makefile)
Run "make distcheck" (which requires Makfile)

Also, looks like parts of the autotools patches didn't get committed
and needs fixing up:
 * autogen.sh should have +x mode
 * Running autogen.sh gives:
   Makefile.am: required file `./NEWS' not found
   Makefile.am: required file `./README' not found
   Makefile.am: required file `./AUTHORS' not found

Jon