Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 717910 - [kde overlay] kde-misc/latte-dock-9999: app/settings/dialogs/settingsdialog.cpp:35:10: fatal error: ../../liblatte2/types.h: No such file or directory
Summary: [kde overlay] kde-misc/latte-dock-9999: app/settings/dialogs/settingsdialog.c...
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo KDE team
URL:
Whiteboard:
Keywords: UPSTREAM
Depends on:
Blocks:
 
Reported: 2020-04-17 15:29 UTC by FlyingWaffle
Modified: 2020-04-18 18:28 UTC (History)
0 users

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


Attachments
live ebuild for latte-dock (latte-dock-9999.ebuild,2.03 KB, text/plain)
2020-04-17 15:29 UTC, FlyingWaffle
Details
failed build's log (build.log,166.95 KB, text/plain)
2020-04-17 16:21 UTC, FlyingWaffle
Details
diff for fix (latte-dock.diff,911 bytes, text/plain)
2020-04-17 16:22 UTC, FlyingWaffle
Details
reduced diff (latte-dock.diff,413 bytes, text/plain)
2020-04-17 17:47 UTC, FlyingWaffle
Details

Note You need to log in before you can comment on or make changes to this bug.
Description FlyingWaffle 2020-04-17 15:29:55 UTC
Created attachment 633418 [details]
live ebuild for latte-dock

Hello,

I noticed that there is no live ebuild for kde-misc/latte-dock on the official overlay, and the one on the 'kde' overlay is broken, so I made my own!

You can find the new ebuild attached.

Let me know  if I need to make any changes to it!

-FlyingWaffle
Comment 1 Andreas Sturmlechner gentoo-dev 2020-04-17 15:36:44 UTC
Live ebuilds of KDE Proj packages are maintained in KDE overlay.


(In reply to FlyingWaffle from comment #0)
> Created attachment 633418 [details]
> live ebuild for latte-dock
> 
> Hello,
> 
> I noticed that there is no live ebuild for kde-misc/latte-dock on the
> official overlay, and the one on the 'kde' overlay is broken, so I made my
> own!

If you want to report a bug, then fine, please attach a build.log and submit a *unified* diff to fix the issue.
Comment 2 FlyingWaffle 2020-04-17 16:21:34 UTC
Created attachment 633422 [details]
failed build's log

'kde' overlay live build for latte-dock doesn't take into acount a hard coded reference to a file relative to the build directory
Comment 3 FlyingWaffle 2020-04-17 16:22:39 UTC
Created attachment 633424 [details]
diff for fix

Here is diff with my fix
Comment 4 Andreas Sturmlechner gentoo-dev 2020-04-17 16:26:52 UTC
That is an upstream out-of-source build issue that should be reported exactly there, upstream. We don't change our ebuild whenever there is a temporary upstream issue.

Your fix is just a workaround and contains wholly unrelated changes that are unacceptable for use.
Comment 5 FlyingWaffle 2020-04-17 17:47:42 UTC
Created attachment 633426 [details]
reduced diff

(In reply to Andreas Sturmlechner from comment #4)
> That is an upstream out-of-source build issue that should be reported
> exactly there, upstream. We don't change our ebuild whenever there is a
> temporary upstream issue.
> 
> Your fix is just a workaround and contains wholly unrelated changes that are
> unacceptable for use.

Could you please clarify?  Ebuild documentation is sparse and not centrally located, so I would appreciate if you could point out what unrelated changes you are referring to.

I also don't see how this is an upstream issue; the upstream build instructions specify that the build directory be placed in the source directory and when doing so manually and when using my ebuild (which also follows these instructions) I encounter no issues.  If there is a different way to relocate the build directory that I should be using please let me know.

I've attached a reduced diff that complies with the upstream build instructions.
Comment 6 Andreas Sturmlechner gentoo-dev 2020-04-17 18:16:20 UTC
(In reply to FlyingWaffle from comment #5)
> Could you please clarify?  Ebuild documentation is sparse and not centrally
> located, so I would appreciate if you could point out what unrelated changes
> you are referring to.
It doesn't seem I need to clarify as you reduced the diff already yourself. src_configure() still looks redundant though.

(In reply to FlyingWaffle from comment #5)
> I also don't see how this is an upstream issue;
If their build system relies on the exact directory placement one level below source dir then it is broken. It usually does not, as the ebuild has worked like that until very recently, as does *any* other kde.org hosted ECM based package.

So the correct fix is to identify the breaking change, fix cmake, and submit it to upstream.
Comment 7 FlyingWaffle 2020-04-18 16:36:40 UTC
Filed an upstream bug report; bug confirmed and fixed; tested emerge again with ebuild from 'kde' overlay, issue has been resolved!

(Should I mark this as 'resolved fixed' or 'resolved upstream'?)
Comment 8 Andreas Sturmlechner gentoo-dev 2020-04-18 18:28:40 UTC
(In reply to FlyingWaffle from comment #7)
> Filed an upstream bug report; bug confirmed and fixed; tested emerge again
> with ebuild from 'kde' overlay, issue has been resolved!
Thank you very much!

(In reply to FlyingWaffle from comment #7)
> (Should I mark this as ... 'resolved upstream'?)
Perfect.