Re: [GIT PULL] osd: Boaz Harrosh - change of email

2014-10-22 Thread Boaz Harrosh
On 10/22/2014 12:04 AM, James Bottomley wrote:
> On Tue, 2014-10-21 at 17:05 +0300, Boaz Harrosh wrote:
>> Hi Sir Linus
>>
>> A small administrative stuff, was on vacation and the old email bounced on 
>> me.
>> I was hoping to still make the 3 weeks merge window, but it was 2 weeks at 
>> the
>> end.
>> Your call if to make this wait for next window. Needless to say that it is
>> ZERO risk, just change of email.
>>
>> Based on commit [bfe01a5b] Linux 3.17
>>
>> 3 patches available in the git repository at:
>>
>>   git://git.open-osd.org/linux-open-osd.git for-linus
>>
>> for you to fetch changes up to 1fa3a002b2546c42c343c77c144871285896ced5:
>>
>>   Boaz Harrosh - fix email in Documentation (2014-10-19 20:36:36 +0300)
>>
>> 
>> Boaz Harrosh (3):
>>   MAINTAINERS: Change Boaz Harrosh's email
>>   Boaz Harrosh - Fix broken email address
>>   Boaz Harrosh - fix email in Documentation
>>
>>  Documentation/scsi/osd.txt  | 3 +--
>>  MAINTAINERS | 2 +-
>>  drivers/scsi/osd/Kbuild | 2 +-
>>  drivers/scsi/osd/Kconfig| 2 +-
>>  drivers/scsi/osd/osd_debug.h| 2 +-
>>  drivers/scsi/osd/osd_initiator.c| 4 ++--
>>  drivers/scsi/osd/osd_uld.c  | 4 ++--
>>  fs/exofs/Kbuild | 2 +-
>>  fs/exofs/common.h   | 2 +-
>>  fs/exofs/dir.c  | 2 +-
>>  fs/exofs/exofs.h| 2 +-
>>  fs/exofs/file.c | 2 +-
>>  fs/exofs/inode.c| 2 +-
>>  fs/exofs/namei.c| 2 +-
>>  fs/exofs/ore.c  | 4 ++--
>>  fs/exofs/ore_raid.c | 2 +-
>>  fs/exofs/ore_raid.h | 2 +-
>>  fs/exofs/super.c| 2 +-
>>  fs/exofs/symlink.c  | 2 +-
>>  fs/exofs/sys.c  | 2 +-
>>  fs/nfs/objlayout/objio_osd.c| 2 +-
>>  fs/nfs/objlayout/objlayout.c| 2 +-
>>  fs/nfs/objlayout/objlayout.h| 2 +-
>>  fs/nfs/objlayout/pnfs_osd_xdr_cli.c | 2 +-
>>  include/linux/pnfs_osd_xdr.h| 2 +-
>>  include/scsi/osd_initiator.h| 2 +-
>>  include/scsi/osd_ore.h  | 2 +-
>>  include/scsi/osd_protocol.h | 4 ++--
>>  include/scsi/osd_sec.h  | 2 +-
>>  include/scsi/osd_sense.h| 2 +-
>>  include/scsi/osd_types.h| 2 +-
>>  31 files changed, 35 insertions(+), 36 deletions(-)
> 
> This is a bit of an unnecessary massive churn.  No one expects the
> author named in the file to stay up to date, especially because the
> @domain.com usually credits the company who paid for the code, so it's
> left in as a kind of mark of respect for them.  I'm not saying it
> applies in your case, just that it creates the common expectation of
> in-file authors needing to be traced through the MAINTAINERS file.
> 
> Could you not just update the MAINTAINERS file only, like everyone else?

OK, I did not know this. The Panasas credit is still there so no
harm done. Anyway Linus pulled it, so lets leave it at that for now.

Next time I'll know.

> James

Thanks
Boaz

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] osd: Boaz Harrosh - change of email

2014-10-21 Thread J. Bruce Fields
On Tue, Oct 21, 2014 at 11:04:12PM +0200, James Bottomley wrote:
> On Tue, 2014-10-21 at 17:05 +0300, Boaz Harrosh wrote:
> > Hi Sir Linus
> > 
> > A small administrative stuff, was on vacation and the old email bounced on 
> > me.
> > I was hoping to still make the 3 weeks merge window, but it was 2 weeks at 
> > the
> > end.
> > Your call if to make this wait for next window. Needless to say that it is
> > ZERO risk, just change of email.
> > 
> > Based on commit [bfe01a5b] Linux 3.17
> > 
> > 3 patches available in the git repository at:
> > 
> >   git://git.open-osd.org/linux-open-osd.git for-linus
> > 
> > for you to fetch changes up to 1fa3a002b2546c42c343c77c144871285896ced5:
> > 
> >   Boaz Harrosh - fix email in Documentation (2014-10-19 20:36:36 +0300)
> > 
> > 
> > Boaz Harrosh (3):
> >   MAINTAINERS: Change Boaz Harrosh's email
> >   Boaz Harrosh - Fix broken email address
> >   Boaz Harrosh - fix email in Documentation
> > 
> >  Documentation/scsi/osd.txt  | 3 +--
> >  MAINTAINERS | 2 +-
> >  drivers/scsi/osd/Kbuild | 2 +-
> >  drivers/scsi/osd/Kconfig| 2 +-
> >  drivers/scsi/osd/osd_debug.h| 2 +-
> >  drivers/scsi/osd/osd_initiator.c| 4 ++--
> >  drivers/scsi/osd/osd_uld.c  | 4 ++--
> >  fs/exofs/Kbuild | 2 +-
> >  fs/exofs/common.h   | 2 +-
> >  fs/exofs/dir.c  | 2 +-
> >  fs/exofs/exofs.h| 2 +-
> >  fs/exofs/file.c | 2 +-
> >  fs/exofs/inode.c| 2 +-
> >  fs/exofs/namei.c| 2 +-
> >  fs/exofs/ore.c  | 4 ++--
> >  fs/exofs/ore_raid.c | 2 +-
> >  fs/exofs/ore_raid.h | 2 +-
> >  fs/exofs/super.c| 2 +-
> >  fs/exofs/symlink.c  | 2 +-
> >  fs/exofs/sys.c  | 2 +-
> >  fs/nfs/objlayout/objio_osd.c| 2 +-
> >  fs/nfs/objlayout/objlayout.c| 2 +-
> >  fs/nfs/objlayout/objlayout.h| 2 +-
> >  fs/nfs/objlayout/pnfs_osd_xdr_cli.c | 2 +-
> >  include/linux/pnfs_osd_xdr.h| 2 +-
> >  include/scsi/osd_initiator.h| 2 +-
> >  include/scsi/osd_ore.h  | 2 +-
> >  include/scsi/osd_protocol.h | 4 ++--
> >  include/scsi/osd_sec.h  | 2 +-
> >  include/scsi/osd_sense.h| 2 +-
> >  include/scsi/osd_types.h| 2 +-
> >  31 files changed, 35 insertions(+), 36 deletions(-)
> 
> This is a bit of an unnecessary massive churn.  No one expects the
> author named in the file to stay up to date, especially because the
> @domain.com usually credits the company who paid for the code, so it's
> left in as a kind of mark of respect for them.  I'm not saying it
> applies in your case, just that it creates the common expectation of
> in-file authors needing to be traced through the MAINTAINERS file.
> 
> Could you not just update the MAINTAINERS file only, like everyone else?

I guess there's also .mailmap, though it doesn't look like that's being
used much.

--b.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] osd: Boaz Harrosh - change of email

2014-10-21 Thread James Bottomley
On Tue, 2014-10-21 at 17:05 +0300, Boaz Harrosh wrote:
> Hi Sir Linus
> 
> A small administrative stuff, was on vacation and the old email bounced on me.
> I was hoping to still make the 3 weeks merge window, but it was 2 weeks at the
> end.
> Your call if to make this wait for next window. Needless to say that it is
> ZERO risk, just change of email.
> 
> Based on commit [bfe01a5b] Linux 3.17
> 
> 3 patches available in the git repository at:
> 
>   git://git.open-osd.org/linux-open-osd.git for-linus
> 
> for you to fetch changes up to 1fa3a002b2546c42c343c77c144871285896ced5:
> 
>   Boaz Harrosh - fix email in Documentation (2014-10-19 20:36:36 +0300)
> 
> 
> Boaz Harrosh (3):
>   MAINTAINERS: Change Boaz Harrosh's email
>   Boaz Harrosh - Fix broken email address
>   Boaz Harrosh - fix email in Documentation
> 
>  Documentation/scsi/osd.txt  | 3 +--
>  MAINTAINERS | 2 +-
>  drivers/scsi/osd/Kbuild | 2 +-
>  drivers/scsi/osd/Kconfig| 2 +-
>  drivers/scsi/osd/osd_debug.h| 2 +-
>  drivers/scsi/osd/osd_initiator.c| 4 ++--
>  drivers/scsi/osd/osd_uld.c  | 4 ++--
>  fs/exofs/Kbuild | 2 +-
>  fs/exofs/common.h   | 2 +-
>  fs/exofs/dir.c  | 2 +-
>  fs/exofs/exofs.h| 2 +-
>  fs/exofs/file.c | 2 +-
>  fs/exofs/inode.c| 2 +-
>  fs/exofs/namei.c| 2 +-
>  fs/exofs/ore.c  | 4 ++--
>  fs/exofs/ore_raid.c | 2 +-
>  fs/exofs/ore_raid.h | 2 +-
>  fs/exofs/super.c| 2 +-
>  fs/exofs/symlink.c  | 2 +-
>  fs/exofs/sys.c  | 2 +-
>  fs/nfs/objlayout/objio_osd.c| 2 +-
>  fs/nfs/objlayout/objlayout.c| 2 +-
>  fs/nfs/objlayout/objlayout.h| 2 +-
>  fs/nfs/objlayout/pnfs_osd_xdr_cli.c | 2 +-
>  include/linux/pnfs_osd_xdr.h| 2 +-
>  include/scsi/osd_initiator.h| 2 +-
>  include/scsi/osd_ore.h  | 2 +-
>  include/scsi/osd_protocol.h | 4 ++--
>  include/scsi/osd_sec.h  | 2 +-
>  include/scsi/osd_sense.h| 2 +-
>  include/scsi/osd_types.h| 2 +-
>  31 files changed, 35 insertions(+), 36 deletions(-)

This is a bit of an unnecessary massive churn.  No one expects the
author named in the file to stay up to date, especially because the
@domain.com usually credits the company who paid for the code, so it's
left in as a kind of mark of respect for them.  I'm not saying it
applies in your case, just that it creates the common expectation of
in-file authors needing to be traced through the MAINTAINERS file.

Could you not just update the MAINTAINERS file only, like everyone else?

James


--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[GIT PULL] osd: Boaz Harrosh - change of email

2014-10-21 Thread Boaz Harrosh
Hi Sir Linus

A small administrative stuff, was on vacation and the old email bounced on me.
I was hoping to still make the 3 weeks merge window, but it was 2 weeks at the
end.
Your call if to make this wait for next window. Needless to say that it is
ZERO risk, just change of email.

Based on commit [bfe01a5b] Linux 3.17

3 patches available in the git repository at:

  git://git.open-osd.org/linux-open-osd.git for-linus

for you to fetch changes up to 1fa3a002b2546c42c343c77c144871285896ced5:

  Boaz Harrosh - fix email in Documentation (2014-10-19 20:36:36 +0300)


Boaz Harrosh (3):
  MAINTAINERS: Change Boaz Harrosh's email
  Boaz Harrosh - Fix broken email address
  Boaz Harrosh - fix email in Documentation

 Documentation/scsi/osd.txt  | 3 +--
 MAINTAINERS | 2 +-
 drivers/scsi/osd/Kbuild | 2 +-
 drivers/scsi/osd/Kconfig| 2 +-
 drivers/scsi/osd/osd_debug.h| 2 +-
 drivers/scsi/osd/osd_initiator.c| 4 ++--
 drivers/scsi/osd/osd_uld.c  | 4 ++--
 fs/exofs/Kbuild | 2 +-
 fs/exofs/common.h   | 2 +-
 fs/exofs/dir.c  | 2 +-
 fs/exofs/exofs.h| 2 +-
 fs/exofs/file.c | 2 +-
 fs/exofs/inode.c| 2 +-
 fs/exofs/namei.c| 2 +-
 fs/exofs/ore.c  | 4 ++--
 fs/exofs/ore_raid.c | 2 +-
 fs/exofs/ore_raid.h | 2 +-
 fs/exofs/super.c| 2 +-
 fs/exofs/symlink.c  | 2 +-
 fs/exofs/sys.c  | 2 +-
 fs/nfs/objlayout/objio_osd.c| 2 +-
 fs/nfs/objlayout/objlayout.c| 2 +-
 fs/nfs/objlayout/objlayout.h| 2 +-
 fs/nfs/objlayout/pnfs_osd_xdr_cli.c | 2 +-
 include/linux/pnfs_osd_xdr.h| 2 +-
 include/scsi/osd_initiator.h| 2 +-
 include/scsi/osd_ore.h  | 2 +-
 include/scsi/osd_protocol.h | 4 ++--
 include/scsi/osd_sec.h  | 2 +-
 include/scsi/osd_sense.h| 2 +-
 include/scsi/osd_types.h| 2 +-
 31 files changed, 35 insertions(+), 36 deletions(-)

Thanks, have a good Kernel Cycle
Boaz
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html