Bug 1138599 - cgit requieres httpd instead of any webserver
Summary: cgit requieres httpd instead of any webserver
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora EPEL
Classification: Fedora
Component: cgit
Version: epel7
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Todd Zullinger
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-09-05 09:34 UTC by tim
Modified: 2017-07-08 17:42 UTC (History)
7 users (show)

Fixed In Version: cgit-0.10.2-4.el7
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1468839 (view as bug list)
Environment:
Last Closed: 2015-03-08 22:45:38 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description tim 2014-09-05 09:34:53 UTC
Description of problem:
I've installed nginx for serving several sites. Now I wanted to add cgit. It has a dependency to httpd instead of any webserver so yum installs httpd which I do not need.

Version-Release number of selected component (if applicable):
CentOS7, cgit 0.10.2 on x86_64

How reproducible:
Install any webserver except httpd, then install cgit and you will get an httpd

Steps to Reproduce:
1. yum install nginx
2. yum install cgit
3.

Actual results:
getting useless second webserver

Expected results:
just cgit as I've already a working webserver

Additional info:
IRC log from #epel:
bastelfreak | hi guys. I'm trying to install cgit via the epel repository on centos 7. unfortuantely it has a dependency to httpd. I have currently installed nginx and do not want httpd, also cgit is running fine with any webserver. How can I report this issue?
wolfy | bastelfreak: bugzilla.redhat.com
bastelfreak | thx
wolfy | bastelfreak: let me check something first 
wolfy | bastelfreak: ask them to require webserver instead of httpd 
wolfy | bastelfreak: but it's tricky because it comes with predefined configs for httpd ( /etc/httpd/conf.d/cgit.conf ) which , in the case of the change , would better be placed in a different package like cgit-httpd 
wolfy | which in turn should require httpd because otherwise  /etc/httpd/conf.d/ would not be owned...
bastelfreak | yeah right

Comment 1 Pavel Raiskup 2014-09-12 15:02:33 UTC
Thanks, good point.  Probably the following is needed:

* make the httpd configuration file %doc file

* README.SELinux rename to README.rpm-dist
    .. saying, among other things, that (a) one webserver should be installed
    and (b) where the appropriate configuration examples are

* I am not sure that 'Requires: webserver'.  Isn't it better to say
  in docs that somebody should install a webserver?

Don't you have nginx configuration needed to run cgit properly?  It would be
nice documentation file.

Thanks, Pavel

Comment 2 neal 2015-02-08 12:46:32 UTC
there is the same problem with php and nagios, both depend on httpd

wouldn't it be possible to simply have nginx provide httpd?

Comment 3 Pavel Raiskup 2015-02-08 13:22:21 UTC
IMO thats what the 'Provides: webserver' is for.

Comment 4 Pavel Raiskup 2015-02-08 14:38:20 UTC
For F21+ it seems to be easy now, there is 'httpd-filesystem' subpackage we
could require (in parallel with 'webserver').

For EPEL7 the situation is worse.  Would anybody object if cgit owned also
/etc/httpd/ and /etc/httpd/conf.d?  According to package guidelines [1], it is
acceptable.  It seems to be ideal to compatibly wait for httpd-filesystem is a
part of EPEL8.  I don't think subpackaging httpd configuration makes sense for
cgit.

Also, in future we could use weak dependencies somehow.  For that no
subpackage is needed.

[1] git and bzr example at https://fedoraproject.org/wiki/Packaging:Guidelines#File_and_Directory_Ownership

Comment 5 Kevin Fenzi 2015-02-08 16:32:43 UTC
I have no objection to just owning those in epel7 for now until a better solution comes in 8. :)

Comment 6 Fedora Update System 2015-02-09 07:42:44 UTC
cgit-0.10.2-4.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/cgit-0.10.2-4.el7

Comment 7 Pavel Raiskup 2015-02-09 07:47:32 UTC
FWIW, epel7 commit:
http://pkgs.fedoraproject.org/cgit/cgit.git/commit/?h=epel7&id=802dbc490f3e

Fedora 22 (rawhide) commit:
http://pkgs.fedoraproject.org/cgit/cgit.git/commit/?id=d552c4627b04b

Still, if there is somebody willing to share nginx configuration for cgit, I
would be happy to package and test that.  In this case, please open separate
issue, possibly with link to upstream proposal (at least regarding
documentation).  Thanks.

Comment 8 Fedora Update System 2015-02-14 02:51:22 UTC
Package cgit-0.10.2-4.el7:
* should fix your issue,
* was pushed to the Fedora EPEL 7 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=epel-testing cgit-0.10.2-4.el7'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-EPEL-2015-0736/cgit-0.10.2-4.el7
then log in and leave karma (feedback).

Comment 9 Fedora Update System 2015-03-08 22:45:38 UTC
cgit-0.10.2-4.el7 has been pushed to the Fedora EPEL 7 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 10 defanor 2017-07-08 17:42:50 UTC
The issue is present again, apparently since 2015-09-05.


Note You need to log in before you can comment on or make changes to this bug.