FS#19353 - [git] Could install gitweb.cgi, please?

Attached to Project: Arch Linux
Opened by Peter Simons (peti) - Tuesday, 04 May 2010, 09:10 GMT
Last edited by Dan McGee (toofishes) - Thursday, 22 July 2010, 18:51 GMT
Task Type Feature Request
Category Packages: Extra
Status Closed
Assigned To Dan McGee (toofishes)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

The PKGBUILD for git does not install gitweb.cgi. Without it, however, git instaweb doesn't work. Generally speaking, it would be nice if gitweb.cgi were available as a port of this package.
This task depends upon

Closed by  Dan McGee (toofishes)
Thursday, 22 July 2010, 18:51 GMT
Reason for closing:  Fixed
Additional comments about closing:  Installed by default anyway in 1.7.2
Comment by Dan McGee (toofishes) - Tuesday, 04 May 2010, 17:07 GMT
I'm not following your logic on the git instaweb part. It works absolutely fine for me, did you try it? What was the result? The entire thing is packaged up in /usr/lib/git-core/git-instaweb.

With that said, we have already had bugs in the past saying this *shouldn't* be included. See  FS#14508 .
Comment by Peter Simons (peti) - Friday, 07 May 2010, 09:43 GMT
You are right, git instaweb does have an embedded version of the CGI script. I didn't realize that. I am sorry.

Regarding  FS#14508 , you seem to misunderstood what that bug report was about. gitweb.cgi is the build product that's generated from gitweb.perl, and that is the script I'd like to have installed. Previous versions of the git package, however, installed gitweb.perl. That was a mistake, IMHO, and I reported that. Instead, gitweb.cgi should be installed. I still believe that this is the right thing to do.
Comment by Peter Simons (peti) - Thursday, 24 June 2010, 07:45 GMT
So ... I guess this task is going to be ignored for decades to come, right?
Comment by Dan McGee (toofishes) - Wednesday, 30 June 2010, 02:14 GMT
The next major version of git is going to always install this automatically, so I'm just going to wait for that to be released and also try and address  FS#19822  after that.

Loading...