Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 648594

Summary: kde5-functions.eclass: punt_bogus_dep() breaks if ${T}/bogus${dep} is empty
Product: Gentoo Linux Reporter: Sven Eden <sven.eden>
Component: EclassesAssignee: Gentoo KDE team <kde>
Status: RESOLVED INVALID    
Severity: normal CC: prefix
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: fix_kde5-functions.eclass_for_cygwin.patch

Description Sven Eden 2018-02-23 09:49:31 UTC
Created attachment 520670 [details, diff]
fix_kde5-functions.eclass_for_cygwin.patch

I have never seen this when working on a normal Gentoo system.

But while merging Konsole and KCalc in a Gentoo Prefix on Cygwin, merges would fail because ${T}/bogus${dep} ended up empty which is not handled.

I am very sorry that I do not remember the packages that caused the failings, but at least the fix is easy.

With the attached patch all packages merged just fine.
Comment 1 Michael Palimaka (kensington) gentoo-dev 2018-02-23 10:42:13 UTC
${T}/bogus${dep} should never be empty AFAIK, so I suspect the root cause to be something else.

Are you able to try running the earlier pcregrep command within the prefix to see if that works manually?
Comment 2 Sven Eden 2018-03-08 09:17:15 UTC
(In reply to Michael Palimaka (kensington) from comment #1)
> ${T}/bogus${dep} should never be empty AFAIK, so I suspect the root cause to
> be something else.
> 
> Are you able to try running the earlier pcregrep command within the prefix
> to see if that works manually?

I have rebuild my prefix, and am at least far enough to start an "emerge kcalc".
So I'll revert my fix locally and see what breaks when. Then I'll report back.
Comment 3 Sven Eden 2018-03-16 11:05:41 UTC
@Michael : You were right. The cause for ${T}/bogus${dep} being empty must have been somewhere else.

I have just finished merging in kcalc and konsole, and had reverted my patch before starting to merge any kde related packages. What should I say? Everything went through just fine.

So the only thing left to do for is is closing this bug.