PERMISSIONS: Reword error messages to avoid "stat" jargon.
authorBen Pfaff <blp@cs.stanford.edu>
Fri, 12 May 2023 05:28:31 +0000 (22:28 -0700)
committerBen Pfaff <blp@cs.stanford.edu>
Fri, 12 May 2023 05:28:31 +0000 (22:28 -0700)
"Cannot stat" doesn't make sense to most people.

Reported by pspp.vertaler@gmail.com at
https://lists.gnu.org/archive/html/pspp-dev/2020-09/msg00058.html

src/language/commands/permissions.c

index 1854cff9f49fb5cf4d4d7cb9b0b938d5cff5987a..3047c19d2bb16fb8359bf79c8499d19b1e0d19ae 100644 (file)
@@ -109,10 +109,11 @@ change_permissions (const char *file_name, enum PER per)
   mode_t mode;
 
   locale_file_name = utf8_to_filename (file_name);
-  if (-1 == stat(locale_file_name, &buf))
+  if (-1 == stat (locale_file_name, &buf))
     {
       const int errnum = errno;
-      msg (SE, _("Cannot stat %s: %s"), file_name, strerror(errnum));
+      msg (SE, _("Cannot read permissions for %s: %s"),
+           file_name, strerror (errnum));
       free (locale_file_name);
       return 0;
     }
@@ -122,11 +123,11 @@ change_permissions (const char *file_name, enum PER per)
   else
     mode = buf.st_mode & ~0222;
 
-  if (-1 == chmod(locale_file_name, mode))
-
+  if (-1 == chmod (locale_file_name, mode))
     {
       const int errnum = errno;
-      msg (SE, _("Cannot change mode of %s: %s"), file_name, strerror(errnum));
+      msg (SE, _("Cannot change permissions for %s: %s"),
+           file_name, strerror (errnum));
       free (locale_file_name);
       return 0;
     }