Re: git commit: Fix legal and code style issues

2014-02-22 Thread j...@nanthrax.net
No worries. Double check is always better. Regards JB -- Jean-Baptiste Onofré jbono...@apache.org http://blog.nanthrax.net Talend - http://wwx.talend.com - Reply message - From: "Guillaume Nodet" To: "dev" Subject: git commit: Fix legal and code style issues Date:

Re: git commit: Fix legal and code style issues

2014-02-22 Thread Guillaume Nodet
My bad. I though the commit was on the branch. Sorry about the noise. 2014-02-22 14:54 GMT+01:00 Jean-Baptiste Onofré : > Yes, command exporter is only on master (not on the 3.0.x branch). > > I fixed multiple issue (some files didn't contain license header), and as > the packaging of command-

Re: git commit: Fix legal and code style issues

2014-02-22 Thread Jean-Baptiste Onofré
No worries. It's exactly the purpose of the karaf-3.0.x branch: avoid to include refactoring/new features/etc in a bug fix/minor update release. Regards JB On 02/22/2014 04:27 PM, Christian Schneider wrote: Then it is fine. I did not check the code. I just wanted to make sure we do not accid

Re: git commit: Fix legal and code style issues

2014-02-22 Thread Christian Schneider
Then it is fine. I did not check the code. I just wanted to make sure we do not accidentally include it. Christian On 22.02.2014 16:19, Jean-Baptiste Onofré wrote: It's the case. I think you didn't see the content of the commit: - it's on master (so 3.1.0-SNAPSHOT) - I didn't merge on karaf-3.

Re: git commit: Fix legal and code style issues

2014-02-22 Thread Jean-Baptiste Onofré
It's the case. I think you didn't see the content of the commit: - it's on master (so 3.1.0-SNAPSHOT) - I didn't merge on karaf-3.0.x branch I fixed the NOTICE, headers, etc on all branches. Regards JB On 02/22/2014 04:01 PM, Christian Schneider wrote: I agree. Lets not introduce it in 3.0.1

Re: git commit: Fix legal and code style issues

2014-02-22 Thread Christian Schneider
I agree. Lets not introduce it in 3.0.1 as I will most probably remove it. Christian Am 22.02.2014 14:47 schrieb "Guillaume Nodet" : > I don't think the command exporter is supposed to be released. > My understanding is that cschneider was going to remote it from master, but > it should not be in

Re: git commit: Fix legal and code style issues

2014-02-22 Thread Jean-Baptiste Onofré
Yes, command exporter is only on master (not on the 3.0.x branch). I fixed multiple issue (some files didn't contain license header), and as the packaging of command-exporter is a bundle, I put a NOTICE file. Regards JB On 02/22/2014 02:46 PM, Guillaume Nodet wrote: I don't think the command

Re: git commit: Fix legal and code style issues

2014-02-22 Thread Guillaume Nodet
I don't think the command exporter is supposed to be released. My understanding is that cschneider was going to remote it from master, but it should not be in 3.0.1. 2014-02-22 10:55 GMT+01:00 : > Repository: karaf > Updated Branches: > refs/heads/master ed167f00d -> ab704c6bc > > > Fix legal