From 33918080e1eb10fd4c32c41e7dd9e966fd947db0 Mon Sep 17 00:00:00 2001 From: Milan Crha Date: Thu, 16 Jan 2025 17:03:04 +0100 Subject: [PATCH] utils: Remove leftover g_chmod() The file has set the attributes after commit c6903ab9db7dc702fdff1e4964fab49a3d0de91c, thus this is not needed. If it was meant to set the executable flag on the parent directory, then it is not needed either, at least according to my tests. The `path_dest` points to the installed file here, not to the parent directory. Closes https://github.com/ximion/appstream/issues/681 --- src/as-utils.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/as-utils.c b/src/as-utils.c index d67c4f1d..acb49377 100644 --- a/src/as-utils.c +++ b/src/as-utils.c @@ -2496,7 +2496,6 @@ as_utils_install_metadata_file_internal (const gchar *filename, return FALSE; } - g_chmod (path_dest, 0755); return TRUE; }