linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Anton Altaparmakov <aia21@cam.ac.uk>
To: Michael Thompson <michael.craig.thompson@gmail.com>
Cc: Greg KH <greg@kroah.com>,
	Phillip Hellewell <phillip@hellewell.homeip.net>,
	linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org,
	mike@halcrow.us, mhalcrow@us.ibm.com, mcthomps@us.ibm.com,
	yoder1@us.ibm.com
Subject: Re: [PATCH 1/12: eCryptfs] Makefile and Kconfig
Date: Fri, 4 Nov 2005 16:22:58 +0000 (GMT)	[thread overview]
Message-ID: <Pine.LNX.4.64.0511041621550.14940@hermes-1.csi.cam.ac.uk> (raw)
In-Reply-To: <afcef88a0511040809p4e9cf962me25c037cbfb9e967@mail.gmail.com>

On Fri, 4 Nov 2005, Michael Thompson wrote:
> On 11/3/05, Greg KH <greg@kroah.com> wrote:
> > On Thu, Nov 03, 2005 at 09:21:16AM -0600, Michael Thompson wrote:
> > > On 11/2/05, Phillip Hellewell <phillip@hellewell.homeip.net> wrote:
> > > > These patches modify fs/Makefile and fs/Kconfig to provide build
> > > > support for eCryptfs.
> > > >
> > > > Signed off by: Phillip Hellewell <phillip@hellewell.homeip.net>
> > > > Signed off by: Michael Halcrow <mhalcrow@us.ibm.com>
> > > > Signed off by: Michael Thompson <mmcthomps@us.ibm.com>
> > >
> > > That should read:
> > > Signed off by: Michael Thompson <mcthomps@us.ibm.com>
> >
> > No, that's not how it is documented on how to do this.  Please try
> > again.
> 
> I've just rummaged around in linux/Documentation and I have not been
> able to find, either in a specific file, or by looking at "The Perfect
> Patch", or other related links, on how to fix an incorrect
> "Signed-off-by" line.
> 
> Like I've already said, I have no problems with a RTFM response, but
> point me to the right M.

I have no idea what Greg was talking about either but one thing that is 
obviously wrong is that as you say it should be "Signed-off-by:" and not 
"Signed off by:", i.e. you have to have dashes between the words, not 
spaces.

Best regards,

	Anton
-- 
Anton Altaparmakov <aia21 at cam.ac.uk> (replace at with @)
Unix Support, Computing Service, University of Cambridge, CB2 3QH, UK
Linux NTFS maintainer / IRC: #ntfs on irc.freenode.net
WWW: http://linux-ntfs.sf.net/ & http://www-stu.christs.cam.ac.uk/~aia21/

  reply	other threads:[~2005-11-04 16:23 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-03  3:32 [PATCH 0/12: eCryptfs] eCryptfs version 0.1 Phillip Hellewell
2005-11-03  3:42 ` [PATCH 1/12: eCryptfs] Makefile and Kconfig Phillip Hellewell
2005-11-03 15:21   ` Michael Thompson
2005-11-03 23:05     ` Greg KH
2005-11-04 16:09       ` Michael Thompson
2005-11-04 16:22         ` Anton Altaparmakov [this message]
2005-11-04 16:28           ` Michael Thompson
2005-11-04 16:35         ` Greg KH
2005-11-04 17:07           ` Michael Thompson
2005-11-03  3:43 ` [PATCH 2/12: eCryptfs] Documentation Phillip Hellewell
2005-11-03  3:43 ` [PATCH 3/12: eCryptfs] Makefile Phillip Hellewell
2005-11-03  3:49 ` [PATCH 4/12: eCryptfs] Main module functions Phillip Hellewell
2005-11-03  6:02   ` Greg KH
2005-11-03 15:09     ` Michael Thompson
2005-11-03 15:47       ` Alexey Dobriyan
2005-11-03 15:40         ` Michael Thompson
2005-11-03 21:34     ` Michael Thompson
2005-11-04 23:28     ` [PATCH: eCryptfs] Remove debug wrappers Michael Halcrow
2005-11-03  3:50 ` [PATCH 5/12: eCryptfs] Header declarations Phillip Hellewell
2005-11-03 23:37   ` James Morris
2005-11-03  3:51 ` [PATCH 6/12: eCryptfs] Superblock operations Phillip Hellewell
2005-11-03 23:47   ` James Morris
2005-11-03  3:52 ` [PATCH 7/12: eCryptfs] File operations Phillip Hellewell
2005-11-03 23:49   ` James Morris
2005-11-03  3:53 ` [PATCH 8/12: eCryptfs] Dentry operations Phillip Hellewell
2005-11-03  3:54 ` [PATCH 9/12: eCryptfs] Inode operations Phillip Hellewell
2005-11-03 23:51   ` James Morris
2005-11-04  0:24     ` Andreas Schwab
2005-11-04  6:29       ` James Morris
2005-11-03  3:55 ` [PATCH 10/12: eCryptfs] Mmap operations Phillip Hellewell
2005-11-03  5:32   ` Michael Halcrow
2005-11-07 20:39     ` [PATCH: eCryptfs] Encrypt on writepage() Michael Halcrow
2005-11-03  3:56 ` [PATCH 11/12: eCryptfs] Keystore Phillip Hellewell
2005-11-03 23:59   ` James Morris
2005-11-04  0:08     ` Michael Halcrow
2005-11-04 17:52   ` Timothy R. Chavez
2005-11-03  3:56 ` [PATCH 12/12: eCryptfs] Crypto functions Phillip Hellewell
2005-11-03 22:06   ` Dave Kleikamp
2005-11-03 22:25     ` Michael Thompson
2005-11-03 22:30       ` Dave Kleikamp
2005-11-04  0:08   ` James Morris
2005-11-04  0:14     ` Michael Halcrow
  -- strict thread matches above, loose matches on Subject: below --
2005-11-19  4:11 [PATCH 0/12: eCryptfs] eCryptfs version 0.1 Phillip Hellewell
2005-11-19  4:14 ` [PATCH 1/12: eCryptfs] Makefile and Kconfig Phillip Hellewell

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=Pine.LNX.4.64.0511041621550.14940@hermes-1.csi.cam.ac.uk \
    --to=aia21@cam.ac.uk \
    --cc=greg@kroah.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mcthomps@us.ibm.com \
    --cc=mhalcrow@us.ibm.com \
    --cc=michael.craig.thompson@gmail.com \
    --cc=mike@halcrow.us \
    --cc=phillip@hellewell.homeip.net \
    --cc=yoder1@us.ibm.com \
    /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).