incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John D. Ament" <johndam...@apache.org>
Subject Re: [VOTE] Release Apache Zipkin Brave Karaf (Incubating) version 0.1.2
Date Thu, 14 Feb 2019 11:25:35 GMT
Hi all,

I just want to point out that the issue is that MavenWrapperDownloader has
incorrect license headers in its source repository, see [1].  We should not
fault Apache projects for leveraging this, since the upstream is not
properly setup.  I can follow up with the developer to fix their headers
via a github issue (if no one else gets to it before me).  Likewise, we
should leave the source in as that is the ideal way to bootstrap the
plugin.  The developer even added it since many Apache projects were
struggling to create releases when his toolkit was in use.  Likewise, since
this project includes no NOTICE file there's nothing to add to a project's
notice file, per the Apache License.

Including prior statements about NOTICE is completely optional, but in the
case of no SGA would be ideal.  I don't think that's a blocker for release
since we have ICLAs on file for the main contributors [2] to this
codebase.  I see Adrian has already corrected this in the repo, so I would
be +1 to releasing with that modification.

John

[1]:
https://github.com/takari/maven-wrapper/blob/master/.mvn/wrapper/MavenWrapperDownloader.java
[2]:
https://github.com/apache/incubator-zipkin-brave-karaf/graphs/contributors



On Thu, Feb 14, 2019 at 1:02 AM Dave Fisher <dave2wave@comcast.net> wrote:

> Hi Adrian/Justin,
>
> We are discussing the wrong points. The clear question is what is
> MavenWrapperDownloader.java and why does it need to be in source release?
> It doesn’t.
>
> The Zipkin build instruction should include the installation instructions
> for MavenWrapper. [3]
>
> Adrian will that work for Zipkin?
>
> Regards,
> Dave
>
> [3] https://www.baeldung.com/maven-wrapper
>
> Sent from my iPhone
>
> > On Feb 13, 2019, at 6:07 PM, Justin Mclean <justin@classsoftware.com>
> wrote:
> >
> > Hi,
> >
> > Sorry but it’s -1 (binding) as it contains compiled code (a .jar) [1],
> an ASF release must consist of source code only. The other issue are minor
> IMO and can be fixed in a future release.
> >
> > I checked:
> > - incubating in name
> > - signature and hashed correct
> > - LICENSE is fine
> > - NOTICE has some minor issue (see below)
> > - All source files have have headers
> > - An unexpected binary file in source release [1]
> > - Can compile from source
> >
> > Re NOTICE did you project come to the ASF in 2016? Who did it come from,
> and if the headers were changed to ASF ones then they should be mentioned
> in NOTICE.
> >
> > Should this file have an ASF header? [2] Where did it originally come
> from? Answer to this question may mean that some changes to LICENSE and
> HEADER are required.
> >
> > Re the maven wrapper jar several project have run into this issue and
> have managed to resolve it without including the jar. You should be able to
> find them with a search of this list. BTW A warning that rat didn’t pick
> this up and it seem it doesn’t follow directories with a dot in front of
> them.
> >
> > While i can compile it looks like the build assumes the code is checked
> out of GitHub and you’re not compiling the source release as I get a lot of
> these:
> > failure occured while calling class
> com.mycila.maven.plugin.license.git.CopyrightRangeProvider
> > java.lang.RuntimeException: Could not compute the year of the last git
> commit for file xxxxx
> >
> > Thanks,
> > Justin
> >
> > 1. ./.mvn/wrapper/maven-wrapper.jar
> > 2. .mvn/wrapper/MavenWrapperDownloader.java
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> > For additional commands, e-mail: general-help@incubator.apache.org
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message