Bug 177487 - evolution doesn't compile against libnotify
Summary: evolution doesn't compile against libnotify
Alias: None
Product: Fedora
Classification: Fedora
Component: evolution
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Dave Malcolm
QA Contact:
Depends On: 177666 177667 177670
Blocks: FC5Blocker
TreeView+ depends on / blocked
Reported: 2006-01-11 01:30 UTC by Dave Malcolm
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2006-01-27 19:28:57 UTC
Type: ---

Attachments (Terms of Use)

System ID Priority Status Summary Last Updated
GNOME Bugzilla 325629 None None None Never

Description Dave Malcolm 2006-01-11 01:30:09 UTC
Description of problem:
If you have libnotify [1] installed when trying to rebuild evolution, the build
fails owing to some new stuff added to notify.h which upstream are making use of:
Error is in calendar/gui/alarm-notify:
alarm-queue.c: In function 'popup_notification':

alarm-queue.c:1463: error: 'NotifyIcon' undeclared (first use in this function)

alarm-queue.c:1463: error: (Each undeclared identifier is reported only once

alarm-queue.c:1463: error: for each function it appears in.)

alarm-queue.c:1463: error: 'icon' undeclared (first use in this function)

alarm-queue.c:1474: error: implicit declaration of function

alarm-queue.c:1514: error: implicit declaration of function

HEAD appears to contain the necessary API stuff: 

but 0.3.0 doesn't:

So it looks like upstream are working with svn snapshots of libnotify; I'll try
 to patch around this within evolution, or else we could use a more
bleeding-edge libnotify 

[1] as opposed to libnotify-devel; I filed bug 177484 about this

Version-Release number of selected component (if applicable):

How reproducible:

Should also add the necessary Requires/BuildRequires to evolution so that we're
deterministic about which notification implementation gets built

Comment 1 Ray Strode [halfline] 2006-01-11 15:36:04 UTC
Okay, so we can either disable libnotify support or port to the new api.

We gotta do one or the other before fc5 if evolution isn't building.

Comment 2 John (J5) Palmieri 2006-01-11 16:24:16 UTC
Port it.  Dave if you send me the file(s) I'll do the initial diff then we can
test together.

Comment 3 Dave Malcolm 2006-01-11 22:12:28 UTC
Have committed a patch to package CVS; building as evolution-2.5.4-4; ought to
at least compile, though the calendar's crashing quite a lot for me ATM which
makes it difficult to test this part of the code, alas.

Comment 5 Dave Malcolm 2006-01-12 20:02:36 UTC
I've reported this upstream here: http://bugzilla.gnome.org/show_bug.cgi?id=326757

Comment 6 Dave Malcolm 2006-01-12 20:34:54 UTC
Upstream report was marked as a dup of
which has a slightly different patch from Ubuntu attached

Comment 7 Ray Strode [halfline] 2006-01-27 19:28:57 UTC
This has been fixed upstream. We will get the fix in the next evolution update.

Closing ->UPSTREAM

Note You need to log in before you can comment on or make changes to this bug.