Bug 1420399

Summary:	switch Requires: /etc/httpd/conf.d to Requires: httpd-filesystem
Product:	[Fedora] Fedora	Reporter:	Igor Gnatenko <ignatenko>
Component:	perl-HTML-Mason	Assignee:	Emmanuel Seyman <emmanuel>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	emmanuel, perl-devel, rc040203, steve
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	perl-HTML-Mason-1.56-8.fc26	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-02-12 09:31:18 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Igor Gnatenko 2017-02-08 14:51:23 UTC

Unfortunately at this point we don't have guidelines which prohibit adding directory dependencies, but I will work on this.

It's quite common that other packages own same directory, there's no problem for it. Which means you will end up with different packages installed on system...

For example, gallery3 co-owns /etc/httpd/conf.d which makes perl-HTML-Mason to pull it instead of httpd-filesystem.

Comment 1 Ralf Corsepius 2017-02-08 15:16:03 UTC

(In reply to Igor Gnatenko from comment #0)
> Unfortunately at this point we don't have guidelines which prohibit adding
> directory dependencies, but I will work on this.
We do have guidelines on this. Multiple ownerships are explicitly allowed.
 
> It's quite common that other packages own same directory, there's no problem
> for it.
Correct.

> Which means you will end up with different packages installed on
> system...
... which means the installer doesn't handle such situations correctly.

> For example, gallery3 co-owns /etc/httpd/conf.d which makes perl-HTML-Mason
> to pull it instead of httpd-filesystem.
I think a viable solution in this case would be to remove this requires and let perl-HTML-Mason also own it.

Comment 2 Igor Gnatenko 2017-02-08 15:32:12 UTC

(In reply to Ralf Corsepius from comment #1)
> (In reply to Igor Gnatenko from comment #0)
> > Unfortunately at this point we don't have guidelines which prohibit adding
> > directory dependencies, but I will work on this.
> We do have guidelines on this. Multiple ownerships are explicitly allowed.
You don't read sentence. I was pointing on Requires on directory, not co-owning directories.
>  
> > It's quite common that other packages own same directory, there's no problem
> > for it.
> Correct.
> 
> > Which means you will end up with different packages installed on
> > system...
> ... which means the installer doesn't handle such situations correctly.
No.
> 
> > For example, gallery3 co-owns /etc/httpd/conf.d which makes perl-HTML-Mason
> > to pull it instead of httpd-filesystem.
> I think a viable solution in this case would be to remove this requires and
> let perl-HTML-Mason also own it.
or change it to Req: httpd-filesystem. Completely up to you. I don't know what this particular package does with it.

Comment 3 Emmanuel Seyman 2017-02-12 09:31:18 UTC

(In reply to Igor Gnatenko from comment #2)
>
> or change it to Req: httpd-filesystem.

Done.

Comment 4 Ralf Corsepius 2017-02-12 19:02:10 UTC

This change is naive and the cause of a regression: Nothing guarantees http-filesystem will provide /etc/httpd/conf.d.

Comment 5 Ralf Corsepius 2017-02-12 19:07:40 UTC

In protest against this change and against  Igor Gnatenko, I am stepping down as maintainer of this package.