Tuesday, April 15, 2025

Re: update: x11/dunst to 1.12.2

Friendly ping.

I am using the updated port since the day I have first submitted it
here. Seems to work fine. Or is there anything left to do?

On Mon Mar 24, 2025 at 8:30 PM CET, Alvar Penning wrote:
> On Mon Mar 24, 2025 at 10:13 AM CET, Stuart Henderson wrote:
>> The completions files would be better in the standard locations
>> (probably easiest to mv in post-install).
>
> Thanks for the feedback. I have attached an updated patch.
>
> Best,
> Alvar
>
>
> diff --git Makefile Makefile
> index 775e475ea5c..03413c0556d 100644
> --- Makefile
> +++ Makefile
> @@ -2,8 +2,7 @@ COMMENT = customizable and lightweight notification-daemon
>
> GH_ACCOUNT = dunst-project
> GH_PROJECT = dunst
> -GH_TAGNAME = v1.9.2
> -REVISION = 2
> +GH_TAGNAME = v1.12.2
>
> CATEGORIES = x11
>
> @@ -52,5 +51,8 @@ pre-configure:
>
> post-install:
> mv ${WRKINST}/${SYSCONFDIR}/dunst ${PREFIX}/share
> + mv ${PREFIX}/share/examples/bash-completion ${PREFIX}/share/bash-completion
> + mv ${PREFIX}/share/examples/fish ${PREFIX}/share/fish
> + mv ${PREFIX}/share/examples/zsh ${PREFIX}/share/zsh
>
> .include <bsd.port.mk>
> diff --git distinfo distinfo
> index 093bbc84211..131aa727da1 100644
> --- distinfo
> +++ distinfo
> @@ -1,2 +1,2 @@
> -SHA256 (dunst-1.9.2.tar.gz) = +mzkXAr0uuGIsNZ5pbqS+Pm35DAbsbtHboLfJrzp5xo=
> -SIZE (dunst-1.9.2.tar.gz) = 4614703
> +SHA256 (dunst-1.12.2.tar.gz) = j3ZkvU5gg+lgTighRf5bje52VfoLCZpbaColSeHzPTI=
> +SIZE (dunst-1.12.2.tar.gz) = 4657260
> diff --git patches/patch-config_mk patches/patch-config_mk
> index 958ea426173..c010fc861b3 100644
> --- patches/patch-config_mk
> +++ patches/patch-config_mk
> @@ -1,13 +1,13 @@
> Index: config.mk
> --- config.mk.orig
> +++ config.mk
> -@@ -42,8 +42,8 @@ endif
> +@@ -52,8 +52,8 @@ endif
>
> # flags
> - DEFAULT_CPPFLAGS = -Wno-gnu-zero-variadic-macro-arguments -D_DEFAULT_SOURCE -DVERSION=\"${VERSION}\" -DSYSCONFDIR=\"${SYSCONFDIR}\"
> --DEFAULT_CFLAGS = -g -std=gnu99 -pedantic -Wall -Wno-overlength-strings -Os ${ENABLE_WAYLAND} ${EXTRA_CFLAGS}
> + DEFAULT_CPPFLAGS = -Wno-gnu-zero-variadic-macro-arguments -D_DEFAULT_SOURCE -DVERSION=\"${VERSION}\" -DSYSCONFDIR=\"${SYSCONFDIR}\" ${ENABLE_WAYLAND} ${ENABLE_X11}
> +-DEFAULT_CFLAGS = -g -std=gnu11 -pedantic -Wall -Wno-overlength-strings -Os ${EXTRA_CFLAGS}
> -DEFAULT_LDFLAGS = -lm -lrt
> -+DEFAULT_CFLAGS = -g -std=gnu99 -pedantic -Wall -Wno-overlength-strings ${ENABLE_WAYLAND} ${EXTRA_CFLAGS}
> ++DEFAULT_CFLAGS = -g -std=gnu11 -pedantic -Wall -Wno-overlength-strings ${EXTRA_CFLAGS}
> +DEFAULT_LDFLAGS = -lm
>
> CPPFLAGS_DEBUG := -DDEBUG_BUILD
> diff --git patches/patch-docs_dunst_1_pod patches/patch-docs_dunst_1_pod
> index e6662fa63ed..f78cc4119e3 100644
> --- patches/patch-docs_dunst_1_pod
> +++ patches/patch-docs_dunst_1_pod
> @@ -1,7 +1,7 @@
> Index: docs/dunst.1.pod
> --- docs/dunst.1.pod.orig
> +++ docs/dunst.1.pod
> -@@ -61,7 +61,7 @@ Display a notification on startup.
> +@@ -94,7 +94,7 @@ Set notification timeout time.
>
> A default configuration file is included (usually ##SYSCONFDIR##/dunst/dunstrc)
> and serves as the least important configuration file. Note: this was previously
> diff --git patches/patch-docs_dunst_5_pod patches/patch-docs_dunst_5_pod
> index 76eb300d292..df6ae772052 100644
> --- patches/patch-docs_dunst_5_pod
> +++ patches/patch-docs_dunst_5_pod
> @@ -1,7 +1,7 @@
> Index: docs/dunst.5.pod
> --- docs/dunst.5.pod.orig
> +++ docs/dunst.5.pod
> -@@ -409,7 +409,7 @@ Hide the count of stacked duplicate notifications.
> +@@ -463,7 +463,7 @@ Hide the count of stacked duplicate notifications.
> Show an indicator if a notification contains actions and/or open-able URLs. See
> ACTIONS below for further details.
>
> @@ -10,16 +10,16 @@ Index: docs/dunst.5.pod
>
> Can be set to a colon-separated list of paths to search for icons to use with
> notifications.
> -@@ -423,7 +423,7 @@ replace this and will need new settings.
> - Comma-separated of names of the the themes to use for looking up icons. This has
> - to be the name of the directory in which the theme is located, not the
> +@@ -477,7 +477,7 @@ replace icon_path search.
> + Comma-separated list of names of the themes to use for looking up icons. This
> + has to be the name of the directory in which the theme is located, not the
> human-friendly name of the theme. So for example, the theme B<Breeze Dark> is
> -located in F</usr/share/icons/breeze-dark>. In this case you have to set the
> +located in F<${PREFIX}/share/icons/breeze-dark>. In this case you have to set the
> theme to B<breeze-dark>.
>
> The first theme in the list is the most important. Only if the icon cannot be
> -@@ -461,12 +461,12 @@ Maximum number of notifications that will be kept in h
> +@@ -515,12 +515,12 @@ Maximum number of notifications that will be kept in h
> is reached, older notifications will be deleted once a new one arrives. See
> HISTORY.
>
> diff --git patches/patch-dunstify_c patches/patch-dunstify_c
> deleted file mode 100644
> index 16accda91f0..00000000000
> --- patches/patch-dunstify_c
> +++ /dev/null
> @@ -1,75 +0,0 @@
> -From 09501e55cfde0b10f8950adb2a94762198b5fcc9 Mon Sep 17 00:00:00 2001
> -From: bynect <68197565+bynect@users.noreply.github.com>
> -Date: Mon, 24 Feb 2025 17:45:03 +0100
> -Subject: [PATCH 1/2] Use properties instead of private fields in dunstify
> -
> -From 20992966f6c00edb4b0233906971307d8d73148c Mon Sep 17 00:00:00 2001
> -From: bynect <68197565+bynect@users.noreply.github.com>
> -Date: Tue, 25 Feb 2025 19:38:02 +0100
> -Subject: [PATCH 2/2] Fix warning with cast
> -
> -Index: dunstify.c
> ---- dunstify.c.orig
> -+++ dunstify.c
> -@@ -171,53 +171,20 @@ void parse_commandline(int argc, char *argv[])
> - }
> - }
> -
> --typedef struct _NotifyNotificationPrivate
> --{
> -- guint32 id;
> -- char *app_name;
> -- char *summary;
> -- char *body;
> --
> -- /* NULL to use icon data. Anything else to have server lookup icon */
> -- char *icon_name;
> --
> -- /*
> -- * -1 = use server default
> -- * 0 = never timeout
> -- * > 0 = Number of milliseconds before we timeout
> -- */
> -- gint timeout;
> --
> -- GSList *actions;
> -- GHashTable *action_map;
> -- GHashTable *hints;
> --
> -- gboolean has_nondefault_actions;
> -- gboolean updates_pending;
> --
> -- gulong proxy_signal_handler;
> --
> -- gint closed_reason;
> --} knickers;
> --
> - int get_id(NotifyNotification *n)
> - {
> -- knickers *kn = n->priv;
> --
> -- /* I'm sorry for taking a peek */
> -- return kn->id;
> -+ GValue value = G_VALUE_INIT;
> -+ g_value_init(&value, G_TYPE_UINT);
> -+ g_object_get_property(G_OBJECT(n), "id", &value);
> -+ return g_value_get_int(&value);
> - }
> -
> - void put_id(NotifyNotification *n, guint32 id)
> - {
> -- knickers *kn = n->priv;
> --
> -- /* And know I'm putting stuff into
> -- * your knickers. I'm sorry.
> -- * I'm so sorry.
> -- * */
> --
> -- kn->id = id;
> -+ GValue value = G_VALUE_INIT;
> -+ g_value_init(&value, G_TYPE_UINT);
> -+ g_value_set_uint(&value, id);
> -+ g_object_set_property(G_OBJECT(n), "id", &value);
> - }
> -
> - void actioned(NotifyNotification *n, char *a, gpointer foo)
> diff --git patches/patch-dunstrc patches/patch-dunstrc
> index fed4e50ecba..74ca9bd6eed 100644
> --- patches/patch-dunstrc
> +++ patches/patch-dunstrc
> @@ -1,16 +1,16 @@
> Index: dunstrc
> --- dunstrc.orig
> +++ dunstrc
> -@@ -217,7 +217,7 @@
> +@@ -238,7 +238,7 @@
> max_icon_size = 128
>
> - # Paths to default icons (only neccesary when not using recursive icon lookup)
> + # Paths to default icons (only necessary when not using recursive icon lookup)
> - icon_path = /usr/share/icons/gnome/16x16/status/:/usr/share/icons/gnome/16x16/devices/
> + icon_path = ${LOCALBASE}/share/icons/gnome/16x16/status/:${LOCALBASE}/share/icons/gnome/16x16/devices/
>
> ### History ###
>
> -@@ -231,10 +231,10 @@
> +@@ -252,10 +252,10 @@
> ### Misc/Advanced ###
>
> # dmenu path.
> diff --git patches/patch-src_dunst_c patches/patch-src_dunst_c
> deleted file mode 100644
> index cba0c845fee..00000000000
> --- patches/patch-src_dunst_c
> +++ /dev/null
> @@ -1,12 +0,0 @@
> -Index: src/dunst.c
> ---- src/dunst.c.orig
> -+++ src/dunst.c
> -@@ -100,7 +100,7 @@ static gboolean run(void *data)
> - gint64 sleep = timeout_at - now;
> - sleep = MAX(sleep, 1000); // Sleep at least 1ms
> -
> -- LOG_D("Sleeping for %li ms", sleep/1000);
> -+ LOG_D("Sleeping for %lli ms", (long long)sleep/1000);
> -
> - if (sleep >= 0) {
> - if (reason == 0 || next_timeout < now || timeout_at < next_timeout) {
> diff --git patches/patch-src_input_c patches/patch-src_input_c
> deleted file mode 100644
> index f49421b7619..00000000000
> --- patches/patch-src_input_c
> +++ /dev/null
> @@ -1,13 +0,0 @@
> -Index: src/input.c
> ---- src/input.c.orig
> -+++ src/input.c
> -@@ -4,7 +4,9 @@
> - #include "settings.h"
> - #include "queues.h"
> - #include <stddef.h>
> -+#ifdef __linux__
> - #include <linux/input-event-codes.h>
> -+

No comments:

Post a Comment