Bug#923958: libmessaging-menu-dev: wrong filenames in pkg-config file

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Bug#923958: libmessaging-menu-dev: wrong filenames in pkg-config file

Luca Boccassi-3
Package: libmessaging-menu-dev
Version: 0.6.0-2
Severity: important

Dear Maintainer,

The pkg-config file shipped by libmessaging-menu-dev adds a non-
existing directory to the include path via cflags and a non-existing
linking flag:

$ pkg-config --cflags messaging-menu
-pthread -I/usr/include/ayatana-messaging-menu -I/usr/include/gio-unix-
2.0 -I/usr/include/libmount -I/usr/include/blkid -I/usr/include/uuid
-I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include
$ pkg-config --libs messaging-menu
-layatana-messaging-menu -lgio-2.0 -lgobject-2.0 -lglib-2.0

$ dpkg -L libmessaging-menu-dev | grep include
/usr/include
/usr/include/messaging-menu
/usr/include/messaging-menu/messaging-menu-app.h
/usr/include/messaging-menu/messaging-menu-message.h
/usr/include/messaging-menu/messaging-menu.h

$ dpkg -L libmessaging-menu-dev | grep so
/usr/lib/x86_64-linux-gnu/libmessaging-menu.so

/usr/include/ayatana-messaging-menu does not exist, the actual
directory is /usr/include/messaging-menu. libayatana-messaging-menu.so
does not exist, the actual library is libmessaging-menu.so.

This makes pkg-config unusable as the real include directory and
linker flags are missing.

--
Kind regards,
Luca Boccassi

signature.asc (499 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Bug#923958: [Pkg-ayatana-devel] Bug#923958: libmessaging-menu-dev: wrong filenames in pkg-config file

Mike Gabriel-4
Hi Luca,

On  Do 07 Mär 2019 17:24:28 CET, Luca Boccassi wrote:

> Package: libmessaging-menu-dev
> Version: 0.6.0-2
> Severity: important
>
> Dear Maintainer,
>
> The pkg-config file shipped by libmessaging-menu-dev adds a non-
> existing directory to the include path via cflags and a non-existing
> linking flag:
>
> $ pkg-config --cflags messaging-menu
> -pthread -I/usr/include/ayatana-messaging-menu -I/usr/include/gio-unix-
> 2.0 -I/usr/include/libmount -I/usr/include/blkid -I/usr/include/uuid
> -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include
> $ pkg-config --libs messaging-menu
> -layatana-messaging-menu -lgio-2.0 -lgobject-2.0 -lglib-2.0
>
> $ dpkg -L libmessaging-menu-dev | grep include
> /usr/include
> /usr/include/messaging-menu
> /usr/include/messaging-menu/messaging-menu-app.h
> /usr/include/messaging-menu/messaging-menu-message.h
> /usr/include/messaging-menu/messaging-menu.h
>
> $ dpkg -L libmessaging-menu-dev | grep so
> /usr/lib/x86_64-linux-gnu/libmessaging-menu.so
>
> /usr/include/ayatana-messaging-menu does not exist, the actual
> directory is /usr/include/messaging-menu. libayatana-messaging-menu.so
> does not exist, the actual library is libmessaging-menu.so.
>
> This makes pkg-config unusable as the real include directory and
> linker flags are missing.
I am also receiving some patches on the upstream side of the project  
at the moment.

Do you think, you can file a pull request against the upstream Git repo [1]?

Or send a patch to this bug tracker instead, if you don't want to use Github?

Thanks,
Mike

[1] https://github.com/AyatanaIndicators/ayatana-indicator-messages/
--

DAS-NETZWERKTEAM
c\o Technik- und Ökologiezentrum Eckernförde
Mike Gabriel, Marienthaler str. 17, 24340 Eckernförde
mobile: +49 (1520) 1976 148
landline: +49 (4354) 8390 139

GnuPG Fingerprint: 9BFB AEE8 6C0A A5FF BF22  0782 9AF4 6B30 2577 1B31
mail: [hidden email], http://das-netzwerkteam.de


attachment0 (868 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Bug#923958: [Pkg-ayatana-devel] Bug#923958: libmessaging-menu-dev: wrong filenames in pkg-config file

Luca Boccassi-3
On Fri, 2019-03-08 at 22:44 +0000, Mike Gabriel wrote:

> Hi Luca,
>
> On  Do 07 Mär 2019 17:24:28 CET, Luca Boccassi wrote:
>
> > Package: libmessaging-menu-dev
> > Version: 0.6.0-2
> > Severity: important
> >
> > Dear Maintainer,
> >
> > The pkg-config file shipped by libmessaging-menu-dev adds a non-
> > existing directory to the include path via cflags and a non-
> > existing
> > linking flag:
> >
> > $ pkg-config --cflags messaging-menu
> > -pthread -I/usr/include/ayatana-messaging-menu -I/usr/include/gio-
> > unix-
> > 2.0 -I/usr/include/libmount -I/usr/include/blkid
> > -I/usr/include/uuid
> > -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-
> > 2.0/include
> > $ pkg-config --libs messaging-menu
> > -layatana-messaging-menu -lgio-2.0 -lgobject-2.0 -lglib-2.0
> >
> > $ dpkg -L libmessaging-menu-dev | grep include
> > /usr/include
> > /usr/include/messaging-menu
> > /usr/include/messaging-menu/messaging-menu-app.h
> > /usr/include/messaging-menu/messaging-menu-message.h
> > /usr/include/messaging-menu/messaging-menu.h
> >
> > $ dpkg -L libmessaging-menu-dev | grep so
> > /usr/lib/x86_64-linux-gnu/libmessaging-menu.so
> >
> > /usr/include/ayatana-messaging-menu does not exist, the actual
> > directory is /usr/include/messaging-menu. libayatana-messaging-
> > menu.so
> > does not exist, the actual library is libmessaging-menu.so.
> >
> > This makes pkg-config unusable as the real include directory and
> > linker flags are missing.
>
> I am also receiving some patches on the upstream side of the
> project  
> at the moment.
>
> Do you think, you can file a pull request against the upstream Git
> repo [1]?
>
> Or send a patch to this bug tracker instead, if you don't want to use
> Github?
>
> Thanks,
> Mike
>
> [1] https://github.com/AyatanaIndicators/ayatana-indicator-messages/
Hi,

Sure no problem, it's a simple fix:

https://github.com/AyatanaIndicators/ayatana-indicator-messages/pull/5

There's another issue - the fork from Ubuntu winded the version back -
0.6.0 when in Ubuntu it was 13.xx, so existing applications that check
for a minimum version, like the pidgin-libnotify patch that ships in
Ubuntu, fail the check now. Not sure how it's best to deal with this.

--
Kind regards,
Luca Boccassi

signature.asc (499 bytes) Download Attachment