Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 591556 - sci-mathematics/singular-4.0.2 file collision with media-gfx/paint-mono-0.1.63
Summary: sci-mathematics/singular-4.0.2 file collision with media-gfx/paint-mono-0.1.63
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Thomas Kahle (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-17 22:05 UTC by Joost Ruis
Modified: 2016-11-09 10:46 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Joost Ruis 2016-08-17 22:05:18 UTC
* Detected file collision(s):
 * 
 *      /usr/lib64/pkgconfig/resources.pc
 * 
 * Searching all installed packages for file collisions...
 * 
 * Press Ctrl-C to Stop
 * 
 * media-gfx/paint-mono-0.1.63:0::gentoo
 *      /usr/lib64/pkgconfig/resources.pc


Reproducible: Always
Comment 1 Thomas Kahle (RETIRED) gentoo-dev 2016-08-21 05:08:09 UTC
I wonder why this does not also apply to the corresponding libraries?  Why is paint-mono installing resources.pc without installing libresources.so in /usr/lib64?  In any case, it is a case of too generally named libraries.

Does pkg-config search in subdirectories of /usr/lib/pkg-config ?  That would give a simple solution.
Comment 2 Thomas Kahle (RETIRED) gentoo-dev 2016-08-22 20:22:17 UTC
I can't even build paint-mono (for bug 581140).
Comment 3 Sławomir Nizio 2016-08-26 07:05:17 UTC
quick but dirty way:

src_install() {
	default
	local lst=(
		Singular.pc
		omalloc.pc
		resources.pc
		factory.pc
		libpolys.pc
	)
	local f
	for f in "${lst[@]}"; do
		rm -v "${ED}usr/$(get_libdir)/pkgconfig/${f}" || die
	done
}
Comment 4 Thomas Kahle (RETIRED) gentoo-dev 2016-08-26 07:45:27 UTC
(In reply to SN (Enlik) from comment #3)
> quick but dirty way:
> 
> src_install() {
> 	default
> 	local lst=(
> 		Singular.pc
> 		omalloc.pc
> 		resources.pc
> 		factory.pc
> 		libpolys.pc
> 	)
> 	local f
> 	for f in "${lst[@]}"; do
> 		rm -v "${ED}usr/$(get_libdir)/pkgconfig/${f}" || die
> 	done
> }

I know how to not install the .pc files.  The point is that I don't want to remove them without good reason.  People who install singular may want to use those.  I first need to be convinced that paint-mono needs theirs more than singular users do.  For that paint-mono should at least be installable.  Is it maintained at all?
Comment 5 David Seifert gentoo-dev 2016-08-26 07:50:04 UTC
(In reply to Thomas Kahle from comment #4)
> (In reply to SN (Enlik) from comment #3)
> > quick but dirty way:
> > 
> > src_install() {
> > 	default
> > 	local lst=(
> > 		Singular.pc
> > 		omalloc.pc
> > 		resources.pc
> > 		factory.pc
> > 		libpolys.pc
> > 	)
> > 	local f
> > 	for f in "${lst[@]}"; do
> > 		rm -v "${ED}usr/$(get_libdir)/pkgconfig/${f}" || die
> > 	done
> > }
> 
> I know how to not install the .pc files.  The point is that I don't want to
> remove them without good reason.  People who install singular may want to
> use those.  I first need to be convinced that paint-mono needs theirs more
> than singular users do.  For that paint-mono should at least be installable.
> Is it maintained at all?

I think the proper way is to do it like systemd: prepend ${PN} to all .pc files (e.g. singular-factory.pc) and then upstream the change. There are ways to deal with multiple global filenames for the semantically same .pc files (https://autotools.io/pkgconfig/pkg_check_modules.html Section 3.5 "Alternatives"). Deleting .pc files is absolutely unacceptable.
Comment 6 Sławomir Nizio 2016-08-26 18:23:50 UTC
(In reply to Thomas Kahle from comment #4)
> I know how to not install the .pc files.

(In reply to David Seifert from comment #5)
> Deleting .pc files is absolutely unacceptable.

No worries, I just posted something I labelled "quick and dirty" as an example and in case it helps someone with anything. Also, thanks for the interest.
Comment 7 Pacho Ramos gentoo-dev 2016-11-09 10:36:40 UTC
paint-mono removed
Comment 8 Thomas Kahle (RETIRED) gentoo-dev 2016-11-09 10:46:56 UTC
(In reply to Pacho Ramos from comment #7)
> paint-mono removed

Huh??? This bug was fixed long time ago in commit
5853a6f2a64c85ae4398c2a6c79abd12c6c21767
Singular does not collide anymore... I thought I had reported back to this bug.