Re: [PATCH] Adding svn-push rev. 2

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.

BR, Jani

-- 
Jani Averbach
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org