Bug 310961

Summary:	www-apps/cgit-0.8.3.1-r1 without cgi use flag is pointless
Product:	Gentoo Linux	Reporter:	parafin <help>
Component:	Current packages	Assignee:	Lance Albertson (RETIRED) <ramereth>
Status:	RESOLVED FIXED
Severity:	normal	CC:	nikoli, pva, web-apps
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---

Description parafin 2010-03-23 18:29:42 UTC

If I emerge www-apps/cgit-0.8.3.1-r1 with cgi use flag disabled nothing executable gets installed, so what's exactly the point of this use flag?
Also another problem: /var/cache/cgit/ dir is created during install, but default permissions don't allow anybody but root to write there, so caching doesn't work even if enabled in config file. Probably some notice about changing ownership of this dir should be added somewhere.
Speaking of config file, default css and logo settings rarely will work, so IMHO there should be some warning about that too.

Comment 1 Sebastian Pipping gentoo-dev

2010-05-22 21:31:34 UTC

Just quick note: I plan on using cgit with nginx and fastcgi and therefore do not want use cgi enabled. I still need to check details about it, please excuse my hurry.

Comment 2 Lance Albertson (RETIRED) gentoo-dev

2010-06-21 00:53:18 UTC

I agree with sping in comment #1 where the cgi, and the use flag is enabled by default for the reason you stated. 

As for the einfo additions, I think adding the cache dir permission is a good idea. We probably should install a better default config file if it doesn't work well out of the box. Do you have any good recommendations on what changes should be made?

Comment 3 Nikoli 2010-06-21 08:46:48 UTC

cgit supports _only_ cgi, fork with fastcgi support exist http://github.com/Dundee/Fast-CGIT

p.s. CGIT 0.8.3.2 released http://hjemli.net/git/cgit/tag/?id=v0.8.3.2

Comment 4 Nikoli 2010-06-24 21:28:46 UTC

bug #325411 has new ebuild

Comment 5 parafin 2010-08-02 17:15:47 UTC

New ebuild hasn't fixed cache permissions problem - see comment #3 to bug #325411.
But it got rid of cgi use flag, that's good.
About cgitrc, I propose these defaults:
css=/cgit/cgit.css
logo=/cgit/cgit.png
(They will work if you run webapp-config with "-d cgit" option)

Comment 6 Peter Volkov (RETIRED) gentoo-dev

2011-04-29 19:03:32 UTC

Fixed in cgit-0.8.3.5.ebuild and above. Thank you for report parafin!