Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ubuntu 18.04 has different pkg-config name for appindicator #15

Open
tekknolagi opened this issue May 13, 2020 · 2 comments
Open

Ubuntu 18.04 has different pkg-config name for appindicator #15

tekknolagi opened this issue May 13, 2020 · 2 comments

Comments

@tekknolagi
Copy link

On Ubuntu, this won't compile unless I change the Makefile a little.

maple% lsb_release -irc
Distributor ID:	Ubuntu
Release:	18.04
Codename:	bionic

Apparently appindicator3-0.1 is not a valid pkg config.

diff --git a/Makefile b/Makefile
index 483c0aa..d9a1da4 100644
--- a/Makefile
+++ b/Makefile
@@ -2,8 +2,8 @@ ifeq ($(OS),Windows_NT)
        TRAY_CFLAGS := -DTRAY_WINAPI=1
        TRAY_LDFLAGS :=
 else ifeq ($(shell uname -s),Linux)
-       TRAY_CFLAGS := -DTRAY_APPINDICATOR=1 $(shell pkg-config --cflags appindicator3-0.1)
-       TRAY_LDFLAGS := $(shell pkg-config --libs appindicator3-0.1)
+       TRAY_CFLAGS := -DTRAY_APPINDICATOR=1 $(shell pkg-config --cflags appindicator-0.1)
+       TRAY_LDFLAGS := $(shell pkg-config --libs appindicator-0.1)
 else ifeq ($(shell uname -s),Darwin)
        TRAY_CFLAGS := -DTRAY_APPKIT=1
        TRAY_LDFLAGS := -framework Cocoa

I don't think I know enough to make a pull request -- just tossing this info your way.

Cheers on a very cool project!

@SapuSeven
Copy link

SapuSeven commented Nov 5, 2020

Same here on current Arch Linux.

Update:

I did some more searching and found out that appindicator-0.1 is the GTK2 version.
You need to install appindicator-gtk3 to use the GTK3 version (appindicator3-0.1).

@chym95
Copy link

chym95 commented Aug 29, 2022

On Ubuntu 22.04.1 I also need GTK3 :

-       TRAY_CFLAGS := -DTRAY_APPINDICATOR=1 $(shell pkg-config --cflags appindicator3-0.1)
-       TRAY_LDFLAGS := $(shell pkg-config --libs appindicator3-0.1)
+       TRAY_CFLAGS := -DTRAY_APPINDICATOR=1 $(shell pkg-config --cflags appindicator-0.1) $(shell pkg-config --cflags --libs gtk+-3.0)
+       TRAY_LDFLAGS := $(shell pkg-config --libs appindicator-0.1) $(shell pkg-config --libs gtk+-3.0)

prabhugopal pushed a commit to prabhugopal/tray that referenced this issue Jun 14, 2024
The tooltip member should be a const char* since the string data is just used as a source for strncpy(). This allows callers to initialize the tooltip member with a string literal without triggering a -Wwrite-strings warning.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants