Kafka, mail # dev - Re: git workflow - 2013-01-05, 18:43
Solr & Elasticsearch trainings in New York & San Francisco [more info][hide]
 Search Hadoop and all its subprojects:

Switch to Plain View
Jay Kreps 2013-01-02, 21:45
David Arthur 2013-01-02, 22:17
Jay Kreps 2013-01-03, 00:03
David Arthur 2013-01-03, 00:26
Jun Rao 2013-01-03, 16:18
Joe Stein 2013-01-05, 04:35
Jay Kreps 2013-01-05, 05:30
Joe Stein 2013-01-05, 05:41
David Arthur 2013-01-05, 17:38
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

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 =

now what is interesting is when I grab the patch using wget
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:

Joe Stein
Twitter: @allthingshadoop <http://www.twitter.com/allthingshadoop>

Joe Stein 2013-01-05, 19:02
Joe Stein 2013-01-06, 05:36
Derek Chen-Becker 2013-01-07, 05:52
David Arthur 2013-01-07, 14:45
Derek Chen-Becker 2013-01-07, 15:06
Jay Kreps 2013-01-07, 16:07
Derek Chen-Becker 2013-01-07, 16:33
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