[SCM] Samba Shared Repository - branch master updated

[Jeremy Allison]

On Thu, May 17, 2012 at 09:04:15AM -0400, simo wrote:
> On Wed, 2012-05-16 at 22:00 -0700, Jeremy Allison wrote: 
> > On Thu, May 17, 2012 at 12:55:59AM -0400, simo wrote:
> > > On Thu, 2012-05-17 at 05:09 +0200, Andrew Bartlett wrote:
> > > > commit 367a644c4d91531faf8b2ce9a167fc196da12422
> > > > Author: Andrew Bartlett <abartlet at samba.org>
> > > > Date:   Mon May 14 17:11:09 2012 -0700
> > > > 
> > > >     We need to split things up into a new helper function
> > > > add_current_ace_to_acl() in order for there to be more posix ACL
> > > > elements than NT ACL elements (so a group SID can own a file, but also
> > > > get the group permissions that will be honoured)
> > > 
> > > Andrew,
> > > it would be really appreciate if you could stick to git standard of
> > > providing a short 'subject' in the first line, then leave one empty, and
> > > then put the commit message wrapped to 72 chars.
> > > 
> > > This is also the standard used in the kernel and makes logs a lot more
> > > readable (also in gitk).
> > 
> > This one might be my fault, as I hand-patched some of
> > Andrews changes, made the edits I wanted and then commited
> > them under his identity with my sign-off (as they were 99.999%
> > his changes).
> > 
> > I may have missed the first line of his original commit
> > message when doing the commit, so this might be my error
> > in the review.
> 
> Should we set up checks in the push hook to reject patches with so badly
> formatted comments ?

So long as it's clear that has happened (with a link on how
to fix it :-) I'm ok with that.

Jeremy.