linux-xfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Deepa Dinamani <deepa.kernel@gmail.com>
To: Eryu Guan <eguan@redhat.com>
Cc: linux-xfs@vger.kernel.org, Arnd Bergmann <arnd@arndb.de>,
	y2038 Mailman List <y2038@lists.linaro.org>
Subject: Re: [PATCH v2] xfs_io: implement 'utimes' command
Date: Thu, 29 Dec 2016 15:54:12 -0800	[thread overview]
Message-ID: <CABeXuvq4EZS=eZsyp2y8B7zMPw0FEhuNziY9-QA8k16py++urg@mail.gmail.com> (raw)
In-Reply-To: <20161228083420.GZ1859@eguan.usersys.redhat.com>

>> Add the utimes command to provide a way to utilize
>> the futimens C library call. This is the
>> interface to the utimensat system call, which updates
>> the mtime and atime of a file.
>>
>> Signed-off-by: Deepa Dinamani <deepa.kernel@gmail.com>
>> ---
>> Changes since v1:
>> * changed error return values
>> * removed redundant roff formatting directive
>> * removed unneeded argument count check
>>
>>  include/input.h   |  1 +
>>  io/Makefile       |  2 +-
>>  io/init.c         |  1 +
>>  io/io.h           |  1 +
>>  io/utimes.c       | 81 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>  libxcmd/input.c   | 22 +++++++++++++++
>>  man/man8/xfs_io.8 | 11 ++++++++
>>  7 files changed, 118 insertions(+), 1 deletion(-)
>>  create mode 100755 io/utimes.c
>>  mode change 100644 => 100755 libxcmd/input.c
>
> Just one nit, I think c files should not have "x" in file modes.

Yes, my bad.
Will post an update with this fixed.

Thanks,
Deepa

      reply	other threads:[~2016-12-29 23:54 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-20  2:51 [PATCH v2] xfs_io: implement 'utimes' command Deepa Dinamani
2016-12-20 20:23 ` Eric Sandeen
2016-12-28  8:34 ` Eryu Guan
2016-12-29 23:54   ` Deepa Dinamani [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CABeXuvq4EZS=eZsyp2y8B7zMPw0FEhuNziY9-QA8k16py++urg@mail.gmail.com' \
    --to=deepa.kernel@gmail.com \
    --cc=arnd@arndb.de \
    --cc=eguan@redhat.com \
    --cc=linux-xfs@vger.kernel.org \
    --cc=y2038@lists.linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).