On Monday 22 September 2003 00:21, you wrote:
> On 2003-09-21 22:19+0300, Shlomi Fish wrote:
> > This patch adds the svn-push utility. It builds by default, but does not
> > install it. Note that I needed to add a new install meta-target
> > "buildonly" and add it to configure.in at the appropriate place. I talked
> > with sussman about it and he and I don't know if this is the proper way
> > to do it.
>
> IMHO, anything under contrib should not be built by default. There would
> be 'make contrib' target, but this should not be executed by default.
>
> My rationale for that is following:
>
> The contrib is supposed to be "a less maintained" area, and there is
> only few or one maintainer for project or feature under contrib. So if
> Subversion's main trunk evolves too far, the build of contrib could be
> broken until the maintainer of that broken contib project catch up
> with the head of the rest of subversion. In this meantime, I
> absolutely like and want that ./configure && make of subversion will
> run without errors.
>
Actually, the point of making this so, is to make sure svn-push builds
correctly after an API change. This paradigm can later be applied to other
programs like the minimal client. So, a maintainer will have to modify it as
well, once a patch is included.
But I don't mind building it as "make contrib".
Regards,
Shlomi Fish
> BR, Jani
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Sep 22 14:12:59 2003