Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 Search Hadoop and all its subprojects:

Switch to Threaded View
Kafka >> mail # dev >> git workflow


Copy link to this message
-
Re: git workflow
that did not work either

I can't even get the patch to apply from the latest trunk because of this
message of patch without email

so the patch is here
https://issues.apache.org/jira/secure/attachment/12563266/KAFKA-133.patch

I go through the steps on the git workflow

git clone https://git-wip-us.apache.org/repos/asf/kafka.git kafka
cd kafka
git fetch
git checkout trunk
//already on trunk
git apply --stat ../KAFKA-133.patch
//project/build.properties         |    2 +-
//project/build/KafkaProject.scala |   44
+++++++++++++++++++++-----------------
//2 files changed, 25 insertions(+), 21 deletions(-)
git apply --check ../KAFKA-133.patch
git am --signoff < ../KAFKA-133.patch
//Patch does not have a valid e-mail address.

my git --version = 1.8.0.3

now what is interesting is when I grab the patch using wget
https://issues.apache.org/jira/secure/attachment/12563266/KAFKA-133.patchinstead
of downloading it it through a browser I get "Patch format
detection failed." instead of the error saying "Patch does not have a valid
e-mail address"

I am guessing it is something I am doing wrong and could be doing different
but am interested to see where exactly the problem is.

any thoughts?  I gotta work on some code for work right will bang on this
later tonight again but if anyone can reproduce this same thing or not or
has an idea that would be great.

could just be the patch, but would prefer to fix the patch and review the
code change for what it is and communicate moving forward how to make
patches differently (if that is in fact the problem)

On Sat, Jan 5, 2013 at 12:38 PM, David Arthur <[EMAIL PROTECTED]> wrote:

> You can amend the previous commit (as long as you havent pushed) with an
> author like "git --amend --author='...'"
>
> On Saturday, January 5, 2013, Joe Stein wrote:
>
> > I am getting the no email after doing
> >
> > git am --signoff < xyz.patch
> >
> > so nothing gets in to commit to set the author :(
> >
> > On Sat, Jan 5, 2013 at 12:30 AM, Jay Kreps <[EMAIL PROTECTED]
> <javascript:;>>
> > wrote:
> >
> > > I have but I don't really know why. This format worked for me:
> > >   git commit --author='Bertrand Russell <[EMAIL PROTECTED]
> <javascript:;>
> > >'
> > >
> > >
> > > On Fri, Jan 4, 2013 at 8:35 PM, Joe Stein <[EMAIL PROTECTED]
> <javascript:;>>
> > wrote:
> > >
> > > > I started following this so far really helpful thanks!!
> > > >
> > > > Running into some issues reviewing someone's patch getting "Patch
> does
> > > not
> > > > have a valid e-mail address." googling to figure out what is wrong
> > > figure I
> > > > mention here if anyone bumped into this yet
> > > >
> > > > thnx
> > > >
> > > > On Thu, Jan 3, 2013 at 11:17 AM, Jun Rao <[EMAIL PROTECTED]
> <javascript:;>>
> > wrote:
> > > >
> > > > > Thanks for documenting this. Could you also add how to resolve
> > > conflicts
> > > > > during rebase?
> > > > >
> > > > > Jun
> > > > >
> > > > > On Wed, Jan 2, 2013 at 1:45 PM, Jay Kreps <[EMAIL PROTECTED]
> <javascript:;>>
> > wrote:
> > > > >
> > > > > > I don't know about other people but I find git kind of
> confusing. I
> > > > > thought
> > > > > > it would be useful to try to document the normal workflow for
> > > different
> > > > > use
> > > > > > cases:
> > > > > > 1. Contributing a patch
> > > > > > 2. Reviewing and integrating a patch that is contributed
> > > > > > 3. Doing development as a committer
> > > > > > 4. Keeping a copy of your local work on github (since it doesn't
> > seem
> > > > > > Apache has a place to keep local backups of work in progress).
> > > > > >
> > > > > > https://cwiki.apache.org/confluence/display/KAFKA/Git+Workflow
> > > > > >
> > > > > > I would like to link this up from the contributor page to help
> > people
> > > > > > (including my future self). Objections?
> > > > > >
> > > > > > I am not a git expert, so any feedback to improve these recipes
> or
> > > bug
> > > > > > fixes (since I haven't tried everything) would be very much
> > > > appreciated.
> > > > > If
/*
Joe Stein
http://www.linkedin.com/in/charmalloc
Twitter: @allthingshadoop <http://www.twitter.com/allthingshadoop>
*/

 
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB