[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] notifications: Don't expand-file-name app-icon.
From: |
Eli Zaretskii |
Subject: |
Re: [PATCH] notifications: Don't expand-file-name app-icon. |
Date: |
Wed, 26 Jul 2023 14:16:16 +0300 |
> From: Michael Albinus <michael.albinus@gmx.de>
> Cc: dmitry@gutov.dev, andrew@trop.in, emacs-devel@gnu.org
> Date: Wed, 26 Jul 2023 09:05:04 +0200
>
> Eli Zaretskii <eliz@gnu.org> writes:
>
> >> As always with relative file names: the caller is responsible to use a
> >> proper default-directory. What else?
> >
> > Wait a minute. The icon's file is not processed by Emacs, right? We
> > pass it to an external program/library/service, right?
>
> We pass it to an external service via a D-Bus call.
In that case, we _definitely_ must call expand-file-name, and using
relative file names without that is asking for trouble.
- Re: [PATCH] notifications: Don't expand-file-name app-icon., (continued)
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Eli Zaretskii, 2023/07/25
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Dmitry Gutov, 2023/07/25
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Eli Zaretskii, 2023/07/25
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Andrew Tropin, 2023/07/25
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Eli Zaretskii, 2023/07/25
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Dmitry Gutov, 2023/07/25
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Eli Zaretskii, 2023/07/25
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Michael Albinus, 2023/07/25
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Eli Zaretskii, 2023/07/25
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Michael Albinus, 2023/07/26
- Re: [PATCH] notifications: Don't expand-file-name app-icon.,
Eli Zaretskii <=
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Matthias Meulien, 2023/07/26
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Michael Albinus, 2023/07/26
- Re: [PATCH] notifications: Don't expand-file-name app-icon., Matthias Meulien, 2023/07/26
[PATCH v2] notifications: Allow to use Icon Naming Specification for app-icon, Andrew Tropin, 2023/07/26
[PATCH v3] notifications: Allow to use Icon Naming Specification for app-icon, Andrew Tropin, 2023/07/26
[PATCH v4] notifications: Allow to use Icon Naming Specification for app-icon, Andrew Tropin, 2023/07/27