Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 206653 - sys-kernel/kerneloops-0.10 - missing dependency on x11-libs/libnotify
Summary: sys-kernel/kerneloops-0.10 - missing dependency on x11-libs/libnotify
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: x86 Linux
: High normal
Assignee: Greg Kroah-Hartman (RETIRED)
URL:
Whiteboard:
Keywords:
: 222419 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-01-19 16:18 UTC by Peter Gantner (a.k.a. nephros)
Modified: 2009-07-29 22:31 UTC (History)
8 users (show)

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


Attachments
build log showing the exact error (sys-kernel_kerneloops-0.10_20080119-161636.log,4.55 KB, text/plain)
2008-01-21 09:22 UTC, Peter Gantner (a.k.a. nephros)
Details
the patch for makefile (makefile-nogtk-0.10.patch,583 bytes, patch)
2008-02-11 22:46 UTC, Michal Terepeta
Details | Diff
the patch for ebuild (ebuild.patch,327 bytes, patch)
2008-02-11 22:49 UTC, Michal Terepeta
Details | Diff
The exact error, kerneloops-0.11 (kerneloops_failure.log,4.73 KB, text/plain)
2008-04-29 22:27 UTC, Michal Terepeta
Details
kerneloops-makefile.patch (kerneloops-makefile.patch,1.09 KB, patch)
2008-06-20 08:44 UTC, Paweł Hajdan, Jr. (RETIRED)
Details | Diff
kerneloops-0.11.ebuild.patch (kerneloops-0.11.ebuild.patch,439 bytes, patch)
2008-06-20 08:45 UTC, Paweł Hajdan, Jr. (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Gantner (a.k.a. nephros) 2008-01-19 16:18:29 UTC
+++ This bug was initially created as a clone of Bug #205386 +++

kerneloops has missing dependency on x11-libs/libnotify

>>> Source unpacked.
>>> Compiling source in /tmp/portage/sys-kernel/kerneloops-0.10/work/kerneloops-0.10 ...
Package libnotify was not found in the pkg-config search path.
Perhaps you should add the directory containing `libnotify.pc'
to the PKG_CONFIG_PATH environment variable
No package 'libnotify' found
... (alot of glib errors follow)

After emerging libnotify kerneloops compiles fine
Comment 1 Greg Kroah-Hartman (RETIRED) gentoo-dev 2008-01-21 06:54:36 UTC
ugh, libnotify?  I'll add it to the list of dependancies for the next release,

but is this really needed if you don't build the X portion of the package?
Comment 2 Peter Gantner (a.k.a. nephros) 2008-01-21 08:52:11 UTC
I had and have USE="-gtk" set for kerneloops, and can still reproduce this by unmerging libnotify and trying to emerge kerneloops.

I think the reason is that even "make kerneloops" (which is called with USE="-gtk") will execute pkg-config (lines 21,22 in the Makefile).

But I'm not very good at reading Makefiles, so that might be wrong.

Comment 3 Peter Gantner (a.k.a. nephros) 2008-01-21 09:22:47 UTC
Created attachment 141442 [details]
build log showing the exact error
Comment 4 Michal Terepeta 2008-02-11 22:44:29 UTC
(In reply to comment #2)
> I had and have USE="-gtk" set for kerneloops, and can still reproduce this by
> unmerging libnotify and trying to emerge kerneloops.
> 
> I think the reason is that even "make kerneloops" (which is called with
> USE="-gtk") will execute pkg-config (lines 21,22 in the Makefile).
> 
> But I'm not very good at reading Makefiles, so that might be wrong.

You're almost right - it is about the line 14 in the Makefile, where MY_CFLAGS is defined by calling pkg-config with libnotify and gtk+-2.0 as arguments. Therefore even "make kerneloops" will fail if there is no libnotify and gtk-2.0 on the system. 
I'm not sure if that is the correct way to do it, but simply changing a bit the the above mentioned line solved the problem for me. I'm attaching a patch for the makefile and for the ebuild.
Comment 5 Michal Terepeta 2008-02-11 22:46:36 UTC
Created attachment 143246 [details, diff]
the patch for makefile
Comment 6 Michal Terepeta 2008-02-11 22:49:17 UTC
Created attachment 143247 [details, diff]
the patch for ebuild
Comment 7 Greg Kroah-Hartman (RETIRED) gentoo-dev 2008-04-28 15:25:20 UTC
That kind of patch will not work as we shouldn't be changing the makefile like this as it will break the "normal" build.

I'm wondering if this is even possible, can you look at the 0.11 release and see if that works better?
Comment 8 Michal Terepeta 2008-04-29 22:25:46 UTC
(In reply to comment #7)
> That kind of patch will not work as we shouldn't be changing the makefile like
> this as it will break the "normal" build.
> 
> I'm wondering if this is even possible, can you look at the 0.11 release and
> see if that works better?
> 

Unfortunately it still fails. I will attach a log with the exact error. But it is easy to notice that during compilation "pkg-config --cflags libnotify gtk+-2.0" is called and as I do not have the libnotify installed, pkg-config can't find it and fails.
Comment 9 Michal Terepeta 2008-04-29 22:27:23 UTC
Created attachment 151371 [details]
The exact error, kerneloops-0.11
Comment 10 Carsten Lohrke (RETIRED) gentoo-dev 2008-05-17 00:44:10 UTC
*** Bug 222419 has been marked as a duplicate of this bug. ***
Comment 11 Xake 2008-06-12 19:26:20 UTC
Easy found problem in the Makefile:

.c.o wants:
       $(CC) $(CFLAGS) $(MY_CFLAGS) -c -o $@ $<

but as:
MY_CFLAGS := `pkg-config --cflags libnotify gtk+-2.0`

then it tries to find libnotify and gtk+-2.0 no matter what...
Comment 12 Paweł Hajdan, Jr. (RETIRED) gentoo-dev 2008-06-20 08:44:34 UTC
Created attachment 157705 [details, diff]
kerneloops-makefile.patch

Here is my suggested Makefile patch, which should not break normal (+gtk) builds.
It looks like it could be quite easily adopted by upstream.
Comment 13 Paweł Hajdan, Jr. (RETIRED) gentoo-dev 2008-06-20 08:45:23 UTC
Created attachment 157707 [details, diff]
kerneloops-0.11.ebuild.patch

And here is a patch for the ebuild (please note changes in dependencies).
Comment 14 Toralf Förster gentoo-dev 2008-08-04 20:32:33 UTC
(In reply to comment #13)
> Created an attachment (id=157707) [edit]
> kerneloops-0.11.ebuild.patch
> 
> And here is a patch for the ebuild (please note changes in dependencies).
> 

Now it gives :
claration -Wdeclaration-after-statement `pkg-config --cflags glib-2.0 dbus-glib-1` -c -o submit.o submit.c
Package dbus-glib-1 was not found in the pkg-config search path.
Perhaps you should add the directory containing `dbus-glib-1.pc'
to the PKG_CONFIG_PATH environment variable
No package 'dbus-glib-1' found
Package dbus-glib-1 was not found in the pkg-config search path.
Perhaps you should add the directory containing `dbus-glib-1.pc'
to the PKG_CONFIG_PATH environment variable
No package 'dbus-glib-1' found
kerneloops.c:34:18: error: glib.h: No such file or directory
Comment 15 Paweł Hajdan, Jr. (RETIRED) gentoo-dev 2008-08-05 12:26:01 UTC
(In reply to comment #14)
> Package dbus-glib-1 was not found in the pkg-config search path.

Oh, I forgot to add dev-libs/dbus-glib to DEPEND. Thanks for noticing!
Comment 16 Toralf Förster gentoo-dev 2008-08-05 12:48:17 UTC
(In reply to comment #15)
> (In reply to comment #14)
> > Package dbus-glib-1 was not found in the pkg-config search path.
> 
> Oh, I forgot to add dev-libs/dbus-glib to DEPEND. Thanks for noticing!
> 
yep, now it compiles and installs fine
Comment 17 Greg Kroah-Hartman (RETIRED) gentoo-dev 2008-09-15 22:41:02 UTC
argh, this is just a mess now.  I've commited the 0.12 version and took out -gtk support.

If someone can create a 0.12 ebuild with the needed changes for this, and if someone wants it, please create a new bug and assign it to me with it.
Comment 18 James M Leddy 2009-07-29 22:31:37 UTC
(In reply to comment #17)
> argh, this is just a mess now.  I've commited the 0.12 version and took out
> -gtk support.
> 
> If someone can create a 0.12 ebuild with the needed changes for this, and if
> someone wants it, please create a new bug and assign it to me with it.
> 
donezo:

bug 276412