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

Karl MacMillan kmacmill at redhat.com
Thu Oct 18 19:09:38 UTC 2007


On Thu, 2007-10-18 at 15:03 -0400, William Jon McCann wrote:
> On 10/18/07, Kevin McCarthy <kmccarth at redhat.com> wrote:
> > Karl MacMillan wrote:
> > > On Thu, 2007-10-18 at 10:37 -0400, William Jon McCann wrote:
> > > > On 10/18/07, William Jon McCann <mccann at jhu.edu> wrote:
> > > > > 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
> > > >
> > > > Seems like "hg diff" doesn't report locally added files by default.
> > > > But hg diff --git does.  Weird.  What should I be using?
> > > >
> > >
> > > Oh, and hg export preserves checkin comments and, I think, mode changes
> > > where as hg diff does not.
> >
> > Use 'hg export -g' to create a patch that prevserves binary files and
> > file permissions.
> 
> Ok, thanks!  Could we add that to
> http://freeipa.org/page/Contribute#Submitting_Changes or is there a
> way I can edit the wiki?  I can't figure out how to create a new user.
> 

Made the change, adding info about how to get a wiki account, and I'll
send you info about your wiki account separately.

Karl




More information about the Freeipa-devel mailing list