bug#15835: [PATCH] chmod: change in the inode's st_ctime

[Guilherme de Almeida Suckevicz]

Hello,

I saw this issue on TODO file:

"Modify chmod so that it does not change an inode's st_ctime
  when the selected operation would have no other effect.
  First suggested by Hans Ecke <http://hans.ecke.ws> in
    http://thread.gmane.org/gmane.comp.gnu.coreutils.bugs/2920
  Discussed more recently on <http://bugs.debian.org/497514>."

I made a patch just to chmod, for now, to this issue and I would like
to ask if it need something else or if you can make some comments about
this patch, I really appreciate.


--- src/chmod.c    2013-11-08 10:19:48.608214043 -0200
+++ src/chmod-orig.c    2013-11-08 10:16:30.378505318 -0200
@@ -266,7 +266,7 @@
       new_mode = mode_adjust (old_mode, S_ISDIR (old_mode) != 0,
umask_value,
                               change, NULL);

-      if (mode_changed (file, old_mode, new_mode) && ! S_ISLNK (old_mode))
+      if (! S_ISLNK (old_mode))
         {
           if (chmodat (fts->fts_cwd_fd, file, new_mode) == 0)
             chmod_succeeded = true;


I really thank you,
Guilherme Almeida Suckevicz.