Hi JB,

Thanks for your check and give suggestions, I checked the NOTICE file,
and find there have notice of scala, eclipse jetty.
For fastjson, we didn't mention this in NOTICE, since fastjson doesn't
have a NOTICE file in their repo.

Thanks,
Wenjun Ruan

On Sat, Apr 23, 2022 at 1:22 PM Jean-Baptiste Onofré <j...@nanthrax.net> wrote:
>
> +1 (binding)
>
> I checked:
> - incubating is in the name
> - hash and signature are good
> - DISCLAIMER is there
> - LICENSE is there
> - NOTICE is there but it seems incomplete, I would suggest to mention
> dependencies license (Apache projects are mentioned in the NOTICE file
> but not third parties like scala, fastjson, eclipse jetty, etc)
> - ASF headers are present
> - No binaries in the source distribution
> - build ok using mvn
>
> Thanks,
> Regards
> JB
>
> On Fri, Apr 22, 2022 at 9:25 AM wenjun <wen...@apache.org> wrote:
> >
> > Hi IPMC,
> >
> > This is an official vote for the Apache SeaTunnel(Incubating) 2.1.1
> > that we have been working toward.
> >
> > To learn more about Apache SeaTunnel(Incubating), please see
> > https://seatunnel.apache.org/
> >
> > Apache SeaTunnel (incubating) community has voted and approved the release.
> >
> > Vote thread:
> > https://lists.apache.org/thread/xkrr8po0frm6176x6wkvzngy1t44qlfo
> > Result thread:
> > https://lists.apache.org/thread/twtvx0zgh2t0xopf1nf30s64cm8wozq8
> >
> > Release changes:
> > https://github.com/apache/incubator-seatunnel/blob/2.1.1/release-note.md
> >
> > The release candidates:
> > https://dist.apache.org/repos/dist/dev/incubator/seatunnel/2.1.1
> >
> > Maven 2 staging repository:
> > https://repository.apache.org/content/repositories/orgapacheseatunnel-1006/org/apache/seatunnel/
> >
> > Git tag for the release:
> > https://github.com/apache/incubator-seatunnel/tree/2.1.1
> >
> > Release Commit ID:
> > https://github.com/apache/incubator-seatunnel/commit/81c523b1d04864db3d936e3186fe43771adde224
> >
> > Keys to verify the Release Candidate:
> > https://downloads.apache.org/incubator/seatunnel/KEYS
> >
> > The vote will be open for at least 72 hours or until necessary number
> > of votes are reached.
> >
> > Please vote accordingly:
> >
> > [ ] +1 approve
> >
> > [ ] +0 no opinion
> >
> > [ ] -1 disapprove with the reason
> >
> > Checklist for reference:
> >
> > [ ] Download links are valid.
> >
> > [ ] Checksums and PGP signatures are valid.
> >
> > [ ] DISCLAIMER is included.
> >
> > [ ] Source code artifacts have correct names matching the current release.
> >
> > [ ] LICENSE and NOTICE files are correct for each SeaTunnel repo.
> >
> > [ ] All files have license headers if necessary.
> >
> > [ ] No compiled archives bundled in source archive.
> >
> > More detail checklist please refer:
> > https://cwiki.apache.org/confluence/display/INCUBATOR/Incubator+Release+Checklist
> >
> >
> > The following votes are carried over from the SeaTunnel dev mailing list:
> > +1(binding)
> > Calvin Kirs,
> > Zhenxu Ke
> >
> > Best,
> > Wenjun Ruan
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> > For additional commands, e-mail: general-h...@incubator.apache.org
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to