-
Notifications
You must be signed in to change notification settings - Fork 37
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
Which are the (build?) dependencies? #491
Comments
Why? GdkPixbuf is a GTK3 dependency. |
Well, there are two separate calls of elementary-xfce/svgtopng/Makefile Lines 10 to 11 in 1080fd5
|
Hm. If I understood correctly, |
@Skif-off , so, the question is wider? Which are the (build?) dependencies? |
Thanks for the report! So it looks like gdk-pixbuf is a dependency of gtk-3. So I think just having gtk-3 listed is okay. The gtk-3 dev package does seem to be required though, so maybe that detail should be added. I tried building on Fedora Xfce Live and, in addition to gtk3-devel and optipng, it did need Unfortunately these dependencies install a ton of other dependencies:
This feels like a lot for just an icon theme. Personally I'd really like to move away from requiring all of these dependencies. Gtk is a big dependency. Especially for those on non-gtk systems, they would need way more packages to be installed. Upstream elementary moved to using
This also seems a little more platform/desktop agnostic. |
Thanks! |
gdk-pixbuf
the dependency?
But the reason is unclear (gtk-3 dev has too many dependencies). |
Well, my test in Xubuntu 22.04. Simple patch: --- elementary-xfce.orig/svgtopng/Makefile
+++ elementary-xfce/svgtopng/Makefile
@@ -7,7 +7,6 @@ all: svgtopng
svgtopng:
${CC} -Wall -Werror -O0 -pipe \
svgtopng.c -o svgtopng \
- `pkg-config --libs --cflags gtk+-3.0` \
`pkg-config --libs --cflags gdk-pixbuf-2.0`
convert: svgtopng | $(ICONDIR)
--- elementary-xfce.orig/svgtopng/svgtopng.c
+++ elementary-xfce/svgtopng/svgtopng.c
@@ -28,7 +28,7 @@
#include <unistd.h>
#include <errno.h>
-#include <gtk/gtk.h>
+#include <gdk-pixbuf/gdk-pixbuf.h>
static void and now: |
https://github.com/shimmerproject/elementary-xfce/blob/v0.20/README.md?plain=1#L28-L30
Should be
gdk-pixbuf
in this list?The text was updated successfully, but these errors were encountered: