Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 28 Oct 2009 10:01:09 -0400
From:      Wesley Shields <wxs@FreeBSD.org>
To:        stef@memberwebs.com
Cc:        freebsd-ports@freebsd.org
Subject:   Re: [patch] Make git dependencies optional.
Message-ID:  <20091028140109.GB55026@atarininja.org>
In-Reply-To: <4AE60885.1040103@memberwebs.com>
References:  <4AE60885.1040103@memberwebs.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Oct 26, 2009 at 02:37:25PM -0600, Stef Walter wrote:
> Thanks for your work on the git port.
> 
> When using git as a deployment mechanism, I'd like to be able to have
> minimal dependencies installed on the production system. The git
> developers have made git so every dependency is optional.
> 
> This patch adds options to the port, so that with 'make config' you can
> choose to not install any dependencies. The following options were added:
> 
>     PERL "Build perl based git tools" on \
>     ICONV "Support for multiple character encodings" on \
>     CURL "Support HTTP push and pull" on \
> 
> The options default to the previous state of affairs, so ports users
> will by default get a git that works exactly like before.
> 
> Some points about the patch:
> 
> * When git is built without perl support, stubs are installed for all
>   perl dependent commands, so the packaging does not change greatly
>   with or without perl.
> * When git is built without curl support, three commands are not
>   installed. Packaging changes slightly.
> * When git is built without curl support, the expat dependency is no
>   longer needed.
> * When git is built without iconv support, nothing changes in the
>   packaging. And obviously UTF-8 support is still present.
> 
> PR: http://www.freebsd.org/cgi/query-pr.cgi?pr=139993
> 
> Thanks in advance for considering this patch. Please let me know if
> additional changes are necessary. I'd love to help you get this into
> FreeBSD ports.

I will review this and commit if appropriate with the next version
update (which there is a PR for already).

-- WXS



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20091028140109.GB55026>