incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ming Wen <wenm...@apache.org>
Subject Re: [VOTE] Release Apache TubeMQ (Incubating) 0.5.0-incubating RC0
Date Sun, 02 Aug 2020 12:52:29 GMT
+1 non-binding

I checked:
- incubating in name.
- DISCLAIMER exists (WIP).
- signature and hashes correct.
- LICENSE is not good which contains a lot of dependencies that shouldn't
be there.
- All files have ASF headers, but some of files's ASF header contains html
<p> tag [1].

> - LICENSE incorrectly list dependancies rather than what is bundled in
the release.
agreed, you only need to add license to LICENSE if it's code bundled in the
release, Instead of all dependencies.
You can refer to how TLP handles that, for example sykwalking[2].

[1]
https://github.com/apache/incubator-tubemq/blob/0.5.0-incubating-RC0/tubemq-server/src/main/java/org/apache/tubemq/server/Server.java#L8
[2] https://github.com/apache/skywalking/blob/master/LICENSE#L204

Thanks,
Ming Wen, Apache APISIX & Apache SkyWalking
Twitter: _WenMing


Justin Mclean <justin@classsoftware.com> 于2020年8月2日周日 下午6:15写道:

> Hi,
>
> +1 (binding)
>
> However if it wasn’t for the Incubating WIP disclaimer I would be -1.
>
> The LICENSE issues where brought up last time [1] It is best if some work
> is done to improve each release so it better than the last. Please ask your
> mentors to help with this.
>
> I checked:
> - incubating in names.
> - signature and hashes correct.
> - LICENSE incorrectly list dependancies rather than what is bundled in the
> release.
> - LICENSE don’t list point to full license text so is unlikely yo comply
> with 3rd party license terms.
> - NOTICE is very likely missing information from other Apache license
> NOTICE files, but with all dependancies included it’s hard to know for sure.
> - All files have ASF headers
> - Failed to compile from source
>
> I didn’t check the binary artefacts.
>
> It would be interesting to know where these come from [2] and how they
> were licensed.
>
> More concerning is why teh build is failing wth this:
> [ERROR] Failed to execute goal on project tubemq-all: Could not resolve
> dependencies for project org.apache.tubemq:tubemq-all:pom:0.5.0-incubating:
> Could not find artifact
> org.apache.tubemq:tubemq-server:tar.gz:bin:0.5.0-incubating in
> berkeleydb-je (https://download.dcache.org/nexus/repository/berkeleydb-je/)
> -> [Help 1]
>
> Why does the source release depend on the binary one?
>
> Thanks,
> Justin
>
> 1.
> https://lists.apache.org/thread.html/r875e3e1abe41369611e7c0c6610ac29af35c552cddc3088e4df3f99c%40%3Cgeneral.incubator.apache.org%3E
> 2. ./resources/assets/public/css/fonts/icon.*
>
>
>
> ---------------------------------------------------------------------
> 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