commit c577c2614e79ff5e15d716690efce7d3cca2b5f9
Author: Aleksandrs Stier <aleks.stier_AT_icloud.com>
Date: Sat Jun 4 01:43:38 2022 +0200
[st][netwmicon] remove glib-dependency; update index.md
diff --git a/st.suckless.org/patches/netwmicon/index.md b/st.suckless.org/patches/netwmicon/index.md
index 8695a5e8..c9ac83af 100644
--- a/st.suckless.org/patches/netwmicon/index.md
+++ b/st.suckless.org/patches/netwmicon/index.md
_AT_@ -3,59 +3,56 @@ netwmicon
Description
-----------
-Enables to set *\_NET\_WM\_ICON* with a png-image.
+Enables to set \_NET\_WM\_ICON with a png-image.
Generally the icon of an application is defined by its desktop-entry. The patch
[desktopentry](../desktopentry) serves this purpose. Unfortunately, some
programs like [tint2](
https://gitlab.com/o9000/tint2) or
[alttab](
https://github.com/sagb/alttab) can't make use of the desktop-entry and
rely instead on a hardcoded icon which has to be defined by the application
-itself with the window-propery *\_NET\_WM\_ICON*. Since st doesn't define
-*\_NET\_WM\_ICON* this programs can't display the correct icon for st even if a
+itself with the window-propery \_NET\_WM\_ICON. Since st doesn't define
+\_NET\_WM\_ICON this programs can't display the correct icon for st even if a
desktop-entry exists. This patch solves this problem.
Dependencies
------------
- gd
-- glib
Defining an icon
----------------
By default each time st starts it will search for a file with the name *st.png*
under */usr/local/share/pixmaps/*. If you put an image with this name in the
-root-directory of the st-repository and call *make install* the image will be
+root-directory of the st-repository and call `make install` the image will be
installed in */usr/local/share/pixmaps/* automatically. Otherwise you have to
-put the file there manually. You can try it out with the following icon (credit:
-[flat-remix](
https://github.com/daniruiz/flat-remix)).
-[st.png](st.png "Example Icon")
+put the file there manually. You can try it out with this icon [st.png](st.png)
+(credit: [flat-remix](
https://github.com/daniruiz/flat-remix)).
Using with desktopentry
-----------------------
If you use the [desktopentry](../desktopentry)-patch you should adjust the icon
-in the file *st.desktop* by replacing the line *Icon=utilities-terminal* with
-*Icon=st*.
+in the file *st.desktop* by replacing the line `Icon=utilities-terminal` with
+`Icon=st`.
Most programs which use the desktop-file to determine the icon should be able to
find *st.png* under */usr/local/share/pixmaps/*. If not report it to me and try
-changing in *confing.mk* the line *ICONPREFIX = $(PREFIX)/share/pixmaps* to
-*ICONPREFIX = $(PREFIX)/share/icons/hicolor/256x256/apps/* and install the icon
+changing in *confing.mk* the line `ICONPREFIX = $(PREFIX)/share/pixmaps` to
+`ICONPREFIX = $(PREFIX)/share/icons/hicolor/256x256/apps/` and install the icon
there.
Call to action
--------------
Maybe somebody would like to create an icon with the st-logo so we could add it
-here as the official icon. If it already exists tell me.
+here instead of using the icon from flat-remix.
Credits
-------
-The code for loading and transforming the image to an appropriate format is,
-except for minor modifications, from
-[xseticon](
https://www.leonerd.org.uk/code/xseticon/) which is developed by Paul
-Evans.
+The code for loading and transforming the image to an appropriate format is in
+large part from [xseticon](
https://www.leonerd.org.uk/code/xseticon/) which is
+developed by Paul Evans.
Download
--------
-* [st-netwmicon-0.8.5.diff](st-netwmicon-0.8.5.diff)
+* [st-netwmicon-0.8.5-v2.diff](st-netwmicon-0.8.5-v2.diff)
* [st-netwmicon-0.8.4.diff](st-netwmicon-0.8.4.diff) (Deprecated)
* [netwmicon.sh](netwmicon.sh) (Deprecated. Used for 0.8.4-patch. Look at
patch-description to understand how its supposed to work. It seems to distort
diff --git a/st.suckless.org/patches/netwmicon/st-netwmicon-0.8.5.diff b/st.suckless.org/patches/netwmicon/st-netwmicon-0.8.5-v2.diff
similarity index 50%
rename from st.suckless.org/patches/netwmicon/st-netwmicon-0.8.5.diff
rename to st.suckless.org/patches/netwmicon/st-netwmicon-0.8.5-v2.diff
index 56fb9220..b02d9dfc 100644
--- a/st.suckless.org/patches/netwmicon/st-netwmicon-0.8.5.diff
+++ b/st.suckless.org/patches/netwmicon/st-netwmicon-0.8.5-v2.diff
_AT_@ -1,46 +1,14 @@
-From 2bab59f760fbb510b0a8f0ba1c26671c8839304e Mon Sep 17 00:00:00 2001
-From: aleks <aleks.stier_AT_icloud.com>
-Date: Tue, 31 May 2022 00:16:17 +0200
-Subject: [PATCH] Enable to set _NET_WM_ICON with a png-image
-
-Generally the icon of an application is defined by its desktop-entry.
-The patch desktopentry serves this purpose. Unfortunately, some programs
-like tint2 (
https://gitlab.com/o9000/tint2) or alttab
-(
https://github.com/sagb/alttab) can't make use of the desktop-entry and
-rely instead on a hardcoded icon which has to be defined by the
-application itself with the window-propery _NET_WM_ICON. Since st
-doesn't define _NET_WM_ICON this programs can't display the correct icon
-for st even if a desktop-entry exists. This patch solves this problem.
-
-The dependencies are gd and glib.
-
-By default each time st starts it will search for a file with the name st.png
-under /usr/local/share/pixmaps/. If you put an image with this name in the
-root-directory of the st-repository and call make install the image will be
-installed in /usr/local/share/pixmaps/ automatically. Otherwise you have to
-put the file there manually.
-
-If you use the desktopentry-patch you should adjust the icon
-in the file st.desktop by replacing the line *Icon=utilities-terminal* with
-*Icon=st*.
-
-Most programs which use the desktop-file to determine the icon should be able to
-find st.png under /usr/local/share/pixmaps/. If not report it to me and try
-changing in confing.mk the line *ICONPREFIX = $(PREFIX)/share/pixmaps* to
-*ICONPREFIX = $(PREFIX)/share/icons/hicolor/256x256/apps/* and install the icon
-there.
-
-The code for loading and transforming the image to an appropriate format
-is, except for minor modifications, from xseticon
-(
https://www.leonerd.org.uk/code/xseticon/) which is developed by Paul
-Evans.
+From 8b0128e8b295fc97bfa3bc5fb4b5e64856d4e3cb Mon Sep 17 00:00:00 2001
+From: Aleksandrs Stier <aleks.stier_AT_icloud.com>
+Date: Sat, 4 Jun 2022 01:24:07 +0200
+Subject: [PATCH] Set _NET_WM_ICON with a png-image
---
Makefile | 3 +++
- config.mk | 8 ++++++--
- st.h | 5 +++--
- x.c | 38 +++++++++++++++++++++++++++++++++++++-
- 4 files changed, 49 insertions(+), 5 deletions(-)
+ config.mk | 6 ++++--
+ st.h | 2 ++
+ x.c | 37 ++++++++++++++++++++++++++++++++++++-
+ 4 files changed, 45 insertions(+), 3 deletions(-)
diff --git a/Makefile b/Makefile
index 470ac86..96e27e3 100644
_AT_@ -60,7 +28,7 @@ index 470ac86..96e27e3 100644
.PHONY: all options clean dist install uninstall
diff --git a/config.mk b/config.mk
-index 4c4c5d5..7af8ab7 100644
+index 4c4c5d5..f8fc780 100644
--- a/config.mk
+++ b/config.mk
_AT_@ -6,6 +6,8 @@ VERSION = 0.8.5
_AT_@ -72,16 +40,12 @@ index 4c4c5d5..7af8ab7 100644
X11INC = /usr/X11R6/include
X11LIB = /usr/X11R6/lib
-_AT_@ -14,14 +16,16 @@ PKG_CONFIG = pkg-config
-
- # includes and libs
+_AT_@ -16,12 +18,12 @@ PKG_CONFIG = pkg-config
INCS = -I$(X11INC) \
-+ `$(PKG_CONFIG) --cflags glib-2.0` \
`$(PKG_CONFIG) --cflags fontconfig` \
`$(PKG_CONFIG) --cflags freetype2`
-LIBS = -L$(X11LIB) -lm -lrt -lX11 -lutil -lXft \
+LIBS = -L$(X11LIB) -lm -lrt -lX11 -lutil -lXft -lgd \
-+ `$(PKG_CONFIG) --libs glib-2.0` \
`$(PKG_CONFIG) --libs fontconfig` \
`$(PKG_CONFIG) --libs freetype2`
_AT_@ -92,36 +56,23 @@ index 4c4c5d5..7af8ab7 100644
STLDFLAGS = $(LIBS) $(LDFLAGS)
diff --git a/st.h b/st.h
-index 519b9bd..0f6e7d4 100644
+index 519b9bd..c10af86 100644
--- a/st.h
+++ b/st.h
-_AT_@ -3,9 +3,10 @@
+_AT_@ -3,6 +3,8 @@
#include <stdint.h>
#include <sys/types.h>
+#include <gd.h>
-+#include <glib.h>
+
/* macros */
--#define MIN(a, b) ((a) < (b) ? (a) : (b))
--#define MAX(a, b) ((a) < (b) ? (b) : (a))
- #define LEN(a) (sizeof(a) / sizeof(a)[0])
- #define BETWEEN(x, a, b) ((a) <= (x) && (x) <= (b))
- #define DIVCEIL(n, d) (((n) + ((d) - 1)) / (d))
+ #define MIN(a, b) ((a) < (b) ? (a) : (b))
+ #define MAX(a, b) ((a) < (b) ? (b) : (a))
diff --git a/x.c b/x.c
-index 8a16faa..84a1e31 100644
+index 8a16faa..169e833 100644
--- a/x.c
+++ b/x.c
-_AT_@ -77,6 +77,8 @@ typedef XftDraw *Draw;
- typedef XftColor Color;
- typedef XftGlyphFontSpec GlyphFontSpec;
-
-+typedef unsigned long int CARD32;
-+
- /* Purely graphic info */
- typedef struct {
- int tw, th; /* tty width and height */
-_AT_@ -93,7 +95,7 @@ typedef struct {
+_AT_@ -93,7 +93,7 @@ typedef struct {
Window win;
Drawable buf;
GlyphFontSpec *specbuf; /* font spec buffer used for rendering */
_AT_@ -130,34 +81,35 @@ index 8a16faa..84a1e31 100644
struct {
XIM xim;
XIC xic;
-_AT_@ -1204,6 +1206,40 @@ xinit(int cols, int rows)
+_AT_@ -1204,6 +1204,41 @@ xinit(int cols, int rows)
xw.netwmiconname = XInternAtom(xw.dpy, "_NET_WM_ICON_NAME", False);
XSetWMProtocols(xw.dpy, xw.win, &xw.wmdeletewin, 1);
+ /* use a png-image to set _NET_WM_ICON */
+ FILE* file = fopen(ICON, "r");
+ if (file) {
-+ /* inititialize variables */
++ /* load image in rgba-format */
+ const gdImagePtr icon_rgba = gdImageCreateFromPng(file);
+ fclose(file);
-+ const int width = gdImageSX(icon_rgba);
++ /* declare icon-variable which will store the image in argb-format */
++ const int width = gdImageSX(icon_rgba);
+ const int height = gdImageSY(icon_rgba);
+ const int icon_n = width * height + 2;
-+ CARD32 *icon_argb = g_new0(CARD32, icon_n);
++ long icon_argb[icon_n];
+ /* set width and height of the icon */
+ int i = 0;
+ icon_argb[i++] = width;
+ icon_argb[i++] = height;
-+ /* RGBA -> ARGB */
++ /* rgba -> argb */
+ for (int y = 0; y < height; y++) {
+ for (int x = 0; x < width; x++) {
+ const int pixel_rgba = gdImageGetPixel(icon_rgba, x, y);
-+ guint8* pixel_argb = (guint8*) &icon_argb[i++];
++ unsigned char *pixel_argb = (unsigned char *) &icon_argb[i++];
+ pixel_argb[0] = gdImageBlue(icon_rgba, pixel_rgba);
+ pixel_argb[1] = gdImageGreen(icon_rgba, pixel_rgba);
+ pixel_argb[2] = gdImageRed(icon_rgba, pixel_rgba);
+ /* scale alpha from 0-127 to 0-255 */
-+ const int alpha = 127 - gdImageAlpha(icon_rgba, pixel_rgba);
++ const unsigned char alpha = 127 - gdImageAlpha(icon_rgba, pixel_rgba);
+ pixel_argb[3] = alpha == 127 ? 255 : alpha * 2;
+ }
+ }
_AT_@ -165,7 +117,7 @@ index 8a16faa..84a1e31 100644
+ /* set _NET_WM_ICON */
+ xw.netwmicon = XInternAtom(xw.dpy, "_NET_WM_ICON", False);
+ XChangeProperty(xw.dpy, xw.win, xw.netwmicon, XA_CARDINAL, 32,
-+ PropModeReplace, (uchar *)icon_argb, icon_n);
++ PropModeReplace, (uchar *) icon_argb, icon_n);
+ }
+
xw.netwmpid = XInternAtom(xw.dpy, "_NET_WM_PID", False);
Received on Sat Jun 04 2022 - 01:45:21 CEST