After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 476076 - fix libnotifymm includedirs
fix libnotifymm includedirs
Status: RESOLVED FIXED
Product: gnomemm
Classification: Deprecated
Component: build
unspecified
Other All
: Normal critical
: ---
Assigned To: gtkmm-forge
gtkmm-forge
Depends on:
Blocks:
 
 
Reported: 2007-09-12 02:28 UTC by Yaakov Selkowitz
Modified: 2007-09-20 15:54 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
includedir patch (4.34 KB, patch)
2007-09-12 02:30 UTC, Yaakov Selkowitz 		none Details | Review

Description Yaakov Selkowitz 2007-09-12 02:28:01 UTC 
The headers of libnotifymm are installed all over the place, and the .pc file doesn't even point to the right directory.  I'm attaching a patch to fix this.
Comment 1 Yaakov Selkowitz 2007-09-12 02:30:25 UTC 
Created attachment 95397 [details] [review]
includedir patch

* libnotify/Makefile.am:
* libnotify/libnotifymm/private/Makefile.am:
* libnotify/libnotifymm.pc.in:
Fix includedirs, and make .pc file point there.  Fixes bug 476076.
Comment 2 Murray Cumming 2007-09-12 09:20:26 UTC 
Thanks. I have committed a similar patch, and release a new tarball.
Comment 3 Yaakov Selkowitz 2007-09-20 12:04:24 UTC 
In 0.6.0, AFAICS there's still one inconsistency: 

--- tools/m4/Makefile.am.orig 
+++ tools/m4/Makefile.am
@@ -5,5 +5,5 @@
 
 # Install the .m4, files:
 
-tools_m4_includedir = $(libdir)/libnotifymm/proc/m4
+tools_m4_includedir = $(libdir)/libnotifymm-1.0/proc/m4
 tools_m4_include_HEADERS = $(files_tools_m4)
Comment 4 Murray Cumming 2007-09-20 13:26:32 UTC 
Done. Thanks. Do you need a new tarball?
Comment 5 Yaakov Selkowitz 2007-09-20 15:54:10 UTC 
I don't; my release includes the patch.