Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#9209 closed defect (fixed)

webkitgtk-2.16.1 appears to require libnotify

Reported by: ken@… Owned by: Douglas R. Reno
Priority: normal Milestone: 8.1
Component: BOOK Version: SVN
Severity: normal Keywords:
Cc:

Description

libnotify is shown as optional for webkitgtk, but appears to be required.

Doing a test build with minimal deps, cmake failed:

-- Looking for IceConnectionNumber in ICE - found
-- Found X11: /usr/lib/libX11.so
-- Could NOT find LibNotify (missing:  LIBNOTIFY_INCLUDE_DIRS LIBNOTIFY_LIBRARIES) 
CMake Error at Source/cmake/OptionsGTK.cmake:415 (message):
  libnotify is needed for USE_LIBNOTIFY.
Call Stack (most recent call first):
  Source/cmake/WebKitCommon.cmake:42 (include)
  CMakeLists.txt:137 (include)


-- Configuring incomplete, errors occurred!

Change History (8)

comment:1 by Douglas R. Reno, 5 years ago

Ken,

I can do this if you want, but if you're already there, feel free.

comment:2 by Douglas R. Reno, 5 years ago

Owner: changed from blfs-book@… to Douglas R. Reno
Status: newassigned

comment:3 by Armin K, 5 years ago

Seems it's not strictly required: -DUSE_LIBNOTIFY=OFF might disable it.

comment:4 by ken@…, 5 years ago

Thanks, it looks as if that works (running make now, cmake was happy). That will make it recommended. I'll report back. Douglas, if you haven't prepared it, you can pass it back to me if you wish.

comment:5 by ken@…, 5 years ago

Yeah, it builds - I had not realised Douglas is now working on 1.16.2.

comment:6 by Douglas R. Reno, 5 years ago

Resolution: fixed
Status: assignedclosed

Fixed at r18714

Ken, I added an option for disabling libnotify for you.

comment:7 by bdubbs@…, 5 years ago

Milestone: 8.1m8.1

Milestone renamed

comment:8 by bdubbs@…, 5 years ago

Milestone: m8.18.1

Milestone renamed

Note: See TracTickets for help on using tickets.