Re: [PATCH] instaweb: make the perl path configurable

From: Jakub Narebski <jnareb@gmail.com>
Date: Jun 12, 2013, 14:00 UTC
Message-ID: <loom.20130612T155755-338@post.gmane.org>
In-Reply-To: <20130611201400.GA28010@compy.Home>

Charles McGarvey <chazmcgarvey <at> brokenzipper.com> writes:

Show 7 quoted lines

> It is convenient for the user to be able to customize the path to perl if they
> do not want to use the system perl.  This may be the case, for example, if the
> user wants to use the plackup httpd but its extra dependencies are not
> installed in the system perl; they can set the perl path to a perl that they
> install and have control over in their own home directory.
> 
> Signed-off-by: Charles McGarvey <chazmcgarvey <at> brokenzipper.com>

Is it really necessary? There is always PERL5LIB if one wants to use Perl modules installed in one's own home directory. If one is using local::lib one has it "for free".

If they do not want to use system perl there is always perlbrew.

Show 17 quoted lines

> ---
>  Documentation/config.txt | 4 ++++
>  git-instaweb.sh          | 4 +++-
>  2 files changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/config.txt b/Documentation/config.txt
> index 6e53fc5..e103594 100644
> --- a/Documentation/config.txt
> +++ b/Documentation/config.txt
>  <at>  <at>  -1549,6 +1549,10  <at>  <at>  instaweb.modulepath::
>  	instead of /usr/lib/apache2/modules.  Only used if httpd
>  	is Apache.
> 
> +instaweb.perlpath::
> +	The path to the perl executable used by linkgit:git-instaweb[1] to
> +	run gitweb and/or verify that the HTTP daemon is running.
> +

[...]

-- 
Jakub Narębski
(via GMane)