Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the CGI CPAN distribution.

Report information
The Basics
Id: 42954
Status: rejected
Priority: 0/
Queue: CGI

People
Owner: MARKSTOS [...] cpan.org
Requestors: jnareb [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



CC: git [...] vger.kernel.org, Junio C Hamano <gitster [...] pobox.com>, bug-CGI.pm [...] rt.cpan.org
MIME-Version: 1.0
In-Reply-To: <1233365512-7130-4-git-send-email-giuseppe.bilotta [...] gmail.com>
X-Spam-Status: No, hits=0.0 required=8.0 tests=DK_SIGNED,SPF_PASS
Content-Disposition: inline
References: <1233365512-7130-1-git-send-email-giuseppe.bilotta [...] gmail.com> <1233365512-7130-3-git-send-email-giuseppe.bilotta [...] gmail.com> <1233365512-7130-4-git-send-email-giuseppe.bilotta [...] gmail.com>
Message-ID: <200902012237.46837.jnareb [...] gmail.com>
content-type: text/plain; charset="utf-8"
Received: from la.mx.develooper.com (x1.develooper.com [63.251.223.170]) by diesel.bestpractical.com (Postfix) with SMTP id 12F7E4D8015 for <bug-CGI.pm [...] rt.cpan.org>; Sun, 1 Feb 2009 16:38:03 -0500 (EST)
Received: (qmail 26676 invoked by uid 103); 1 Feb 2009 21:38:03 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 1 Feb 2009 21:38:03 -0000
Received: from fg-out-1718.google.com (HELO fg-out-1718.google.com) (72.14.220.155) by 16.mx.develooper.com (qpsmtpd/0.43rc1) with ESMTP; Sun, 01 Feb 2009 13:37:59 -0800
Received: by fg-out-1718.google.com with SMTP id 19so556493fgg.15 for <bug-CGI.pm [...] rt.cpan.org>; Sun, 01 Feb 2009 13:37:55 -0800 (PST)
Received: by 10.86.97.7 with SMTP id u7mr703983fgb.8.1233524275290; Sun, 01 Feb 2009 13:37:55 -0800 (PST)
Received: from ?192.168.1.13? (abwv210.neoplus.adsl.tpnet.pl [83.8.245.210]) by mx.google.com with ESMTPS id 3sm2607984fge.52.2009.02.01.13.37.53 (version=TLSv1/SSLv3 cipher=RC4-MD5); Sun, 01 Feb 2009 13:37:54 -0800 (PST)
Delivered-To: cpan-bug+CGI.pm [...] diesel.bestpractical.com
Subject: [PATCH 4/3] gitweb: Update README that gitweb works better with PATH_INFO
User-Agent: KMail/1.9.3
Return-Path: <jnareb [...] gmail.com>
Domainkey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:subject:date:user-agent:cc:references:in-reply-to :mime-version:content-type:content-transfer-encoding :content-disposition:message-id; b=Dh+sE40gRkWVJWPQA3CxjD6uS8FKux4UgA4RqN+qalkTxB+bi2AY0yE05L6oFunTn5 Whq3jUSFacjwQEKUBi2XnqlLvJOq5IW97JV7FsVwhOsUimt7jrAV7KlSrBfBgqcjl8mG j8o3xpT3CA6tU9ZJCEjn/f5APom1Nkf9YOnZU=
X-Original-To: bug-CGI.pm [...] rt.cpan.org
X-Spam-Check-BY: 16.mx.develooper.com
Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:from:to:subject:date :user-agent:cc:references:in-reply-to:mime-version:content-type :content-transfer-encoding:content-disposition:message-id; bh=2BgXnK5fMVVLerIuDyCkMCYy5d5VV36BVugPSSrMvec=; b=lzaUYarCSF3KlhQlb/MMX7kwuwsDLqihhgwSWWM6HVAbalBjt/x2bw3FikHRDgpa4p FlIYoyRzZ8gGiq8iVfLjjhk/XeRcPjGtIcEiOhnMy5nGkx+2kBx7CbMBvJiQGE/Ae/O7 uMb3vs/4Wo1z+jf1E06kili5+wG5+HHnAJG8E=
Date: Sun, 1 Feb 2009 22:37:45 +0100
X-Spam-Level: *
To: Giuseppe Bilotta <giuseppe.bilotta [...] gmail.com>
Content-Transfer-Encoding: 7bit
From: Jakub Narebski <jnareb [...] gmail.com>
X-RT-Original-Encoding: iso-8859-2
Content-Length: 2474
Download (untitled) / with headers
text/plain 2.4k
One had to configure gitweb for it to find static files (stylesheets, images) when using path_info URLs. Now that it is not necessary thanks to adding BASE element to HTML head if needed, update README to reflect this fact. Signed-off-by: Jakub Narebski <jnareb@gmail.com> --- This is update of gitweb/README to fit state after Giuseppe patches. There is a little problem either with terminology, or with CGI.pm module itself. RFC 1808 "Relative Uniform Resource Locators" says that: http://git.example.com/cgi-bin/gitweb.cgi is absolute URL /cgi-bin/gitweb.cgi is relative URL but absolute path gitweb.cgi is relative URL and relative path while CGI.pm (or, to be more exact, its url() method) says: url() or url(-full) returns http://git.example.com/cgi-bin/gitweb.cgi url(-absolute) returns /cgi-bin/gitweb.cgi url(-relative) returns gitweb.cgi (or /cgi-bin/gitweb.cgi) That is why I used "Full URL" for $my_url = $cgi->url(); and "absolute URL" (as before) for $my_uri = $cgi->url(-absolute); in gitweb/README. gitweb/README | 10 ++++------ 1 files changed, 4 insertions(+), 6 deletions(-) diff --git a/gitweb/README b/gitweb/README index 52ad88b..a9dc2e5 100644 --- a/gitweb/README +++ b/gitweb/README @@ -162,14 +162,12 @@ not include variables usually directly set during build): $GITWEB_LIST during installation. If empty, $projectroot is used to scan for repositories. * $my_url, $my_uri - URL and absolute URL of gitweb script; you might need to set those - variables if you are using 'pathinfo' feature: see also below. + Full URL and absolute URL of gitweb script; + in earlier versions of gitweb you might have need to set those + variables, now there should be no need to do it. * $home_link Target of the home link on top of all pages (the first part of view - "breadcrumbs"). By default set to absolute URI of a page; you might - need to set it up to [base] gitweb URI if you use 'pathinfo' feature - (alternative format of the URLs, with project name embedded directly - in the path part of URL). + "breadcrumbs"). By default set to absolute URI of a page ($my_uri). * @stylesheets List of URIs of stylesheets (relative to base URI of a page). You might specify more than one stylesheet, for example use gitweb.css -- 1.6.1
MIME-Version: 1.0
In-Reply-To: <200902012237.46837.jnareb [...] gmail.com>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Charset: utf8
References: <1233365512-7130-1-git-send-email-giuseppe.bilotta [...] gmail.com> <1233365512-7130-3-git-send-email-giuseppe.bilotta [...] gmail.com> <1233365512-7130-4-git-send-email-giuseppe.bilotta [...] gmail.com> <200902012237.46837.jnareb [...] gmail.com>
Content-Type: text/plain
Message-ID: <rt-3.6.HEAD-6413-1248485380-824.42954-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 2871
Download (untitled) / with headers
text/plain 2.8k
This email was CC'ed to the CGI.pm bug tracker, but doesn't appear to report a bug in CGI.pm. I'm marking it as "rejected", but please reply if you feel there is a related bug in CGI.pm that still needs to be changed. Mark On Sun Feb 01 16:38:11 2009, jnareb@gmail.com wrote: Show quoted text
> One had to configure gitweb for it to find static files (stylesheets, > images) when using path_info URLs. Now that it is not necessary > thanks to adding BASE element to HTML head if needed, update README to > reflect this fact. > > Signed-off-by: Jakub Narebski <jnareb@gmail.com> > --- > This is update of gitweb/README to fit state after Giuseppe patches. > > There is a little problem either with terminology, or with CGI.pm > module itself. RFC 1808 "Relative Uniform Resource Locators" says > that: > http://git.example.com/cgi-bin/gitweb.cgi is absolute URL > /cgi-bin/gitweb.cgi is relative URL > but absolute path > gitweb.cgi is relative URL > and relative path > > while CGI.pm (or, to be more exact, its url() method) says: > > url() or url(-full) returns http://git.example.com/cgi- > bin/gitweb.cgi > url(-absolute) returns /cgi-bin/gitweb.cgi > url(-relative) returns gitweb.cgi (or /cgi-bin/gitweb.cgi) > > That is why I used "Full URL" for $my_url = $cgi->url(); > and "absolute URL" (as before) for $my_uri = $cgi->url(-absolute); > in gitweb/README. > > gitweb/README | 10 ++++------ > 1 files changed, 4 insertions(+), 6 deletions(-) > > diff --git a/gitweb/README b/gitweb/README > index 52ad88b..a9dc2e5 100644 > --- a/gitweb/README > +++ b/gitweb/README > @@ -162,14 +162,12 @@ not include variables usually directly set > during build): > $GITWEB_LIST during installation. If empty, $projectroot is used > to scan for repositories. > * $my_url, $my_uri > - URL and absolute URL of gitweb script; you might need to set those > - variables if you are using 'pathinfo' feature: see also below. > + Full URL and absolute URL of gitweb script; > + in earlier versions of gitweb you might have need to set those > + variables, now there should be no need to do it. > * $home_link > Target of the home link on top of all pages (the first part of > view > - "breadcrumbs"). By default set to absolute URI of a page; you > might > - need to set it up to [base] gitweb URI if you use 'pathinfo' > feature > - (alternative format of the URLs, with project name embedded > directly > - in the path part of URL). > + "breadcrumbs"). By default set to absolute URI of a page > ($my_uri). > * @stylesheets > List of URIs of stylesheets (relative to base URI of a page). You > might specify more than one stylesheet, for example use gitweb.css


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.