On Wed, Nov 29, 2023 at 5:07 PM PJ Fanning <fannin...@gmail.com> wrote:
>
> Thanks Paul. I'm not sure if the build works on Windows. The CI is
> Linux based. Most developers on the Pekko project use MacOS or Linux
> to build. I'll log an issue in the hope that someone with a Windows
> computer could look at it.

Perhaps in the "building from source" doco, just add a note that the
build has minimal testing on Windows.

> The AddThis license comes from:
> https://github.com/addthis/stream-lib/blob/master/NOTICE.txt

Okay, fair enough. I guess an entity can have any name. :-)

> This was pulled in due to this line:
> https://github.com/apache/incubator-pekko/blob/8fefca1c81e13d089ed8db85f375eb3538089c43/remote/src/main/java/org/apache/pekko/remote/artery/compress/CountMinSketch.java#L27
>
> Which has links to
> https://github.com/addthis/stream-lib/blob/master/src/main/java/com/clearspring/analytics/stream/frequency/CountMinSketch.java
>
>
> On Wed, 29 Nov 2023 at 00:59, Paul King <pa...@asert.com.au> wrote:
> >
> > +1 (binding)
> >
> > [X] Download links are valid.
> > [X] Checksums and signatures.
> > [X] LICENSE/NOTICE files exist
> >     I did notice "Copyright 2016 AddThis" in the LICENSE file wrt
> > MurmurHash3, something to fix for the next release?
> >     Given the accompanying explanation in the file, I don't see this
> > as a showstopper for this release.
> > [X] No unexpected binary files
> >     I didn't do an exhaustive check
> > [X] Source files have ASF headers
> > [-] Can compile from source
> >     I ran "sbt compile" and "sbt package" but both had errors. A sample:
> >     [error] bnd: Invalid value for Bundle-Version, HEAD+20231129-0950
> > does not match [0-9]{1,9}(\.[0-9]{1,9}(\.[0-9]{1,9}(\.[0-9A-Za-z_-]+)?)?)?
> >     [error] java.util.zip.ZipException: duplicate entry: 
> > META-INF/MANIFEST.MF
> >     ...
> >     [error] 
> > D:\tmp\pekko102\apache-pekko-1.0.2-incubating-src-20231115\actor\src\main\java\org\apache\pekko\util\Unsafe.java:25:23:
> > not found: value sun
> >     [error]   public static final sun.misc.Unsafe instance;
> >     [error]                       ^
> >     I rarely run sbt, so this could easily be my local setup. I didn't
> > try my linux box. Maybe it would have worked there.
> >
> > I also tested the artifacts in the staging repo with a simple typed
> > actors example in Groovy:
> > https://github.com/paulk-asert/groovy-pekko-gpars/tree/main/src/main/groovy/pekko
> > I used Groovy 4.0.15, the 1.0.2-RC1 artifacts and the Scala 2.13 runtime.
> > It ran without problems.
> >
> > Cheers, Paul.
> >
> > On Sat, Nov 18, 2023 at 11:56 PM PJ Fanning <fannin...@apache.org> wrote:
> > >
> > > Hello Incubator Community,
> > >
> > > This is a call for a vote to release Apache Pekko(incubating)
> > > Core version 1.0.2-RC1.
> > >
> > > The discussion thread:
> > > https://lists.apache.org/thread/zy84yk5gxp54bkn81ntlj5ps5o2z9w4j
> > >
> > > Pekko Vote Thread
> > > https://lists.apache.org/thread/0ojxypkm650bhk6pf53cp7h8036jgyol
> > >
> > > Pekko Result:
> > > https://lists.apache.org/thread/mh398gtjm751fq4fhss5c0xfwxnvx2xy
> > >
> > > The release candidate:
> > >
> > > https://dist.apache.org/repos/dist/dev/incubator/pekko/1.0.2-RC1/
> > >
> > > This release has been signed with a PGP key, available here:
> > >
> > > https://dist.apache.org/repos/dist/release/incubator/pekko/KEYS
> > >
> > > Git branch for the release:
> > >
> > > https://github.com/apache/incubator-pekko/tree/v1.0.2-RC1
> > > Git commit ID: 58fa510455190bd62d04f92a83c9506a7588d29c
> > >
> > > Release Notes:
> > >
> > > https://nightlies.apache.org/pekko/docs/pekko/1.0/docs/release-notes/index.html#1-0-2
> > >
> > > Please download, verify, and test.
> > >
> > > We have also staged jars in the Apache Nexus Repository. These were
> > > built with the same code
> > > as appears in this Source Release Candidate. We would appreciate if
> > > users could test with these too.
> > > If anyone finds any serious problems with these jars, please also
> > > notify us on this thread.
> > >
> > > https://repository.apache.org/content/groups/staging/org/apache/pekko/
> > >
> > > In sbt, you can add this resolver.
> > >
> > > resolvers += "Apache Pekko Staging" at
> > > "https://repository.apache.org/content/groups/staging";
> > >
> > >
> > > The vote will be left open for at least 72 hours.
> > >
> > > [ ] +1 approve
> > > [ ] +0 no opinion
> > > [ ] -1 disapprove with the reason
> > >
> > > To learn more about Apache Pekko, please see https://pekko.apache.org/
> > >
> > > Checklist for reference:
> > >
> > > [ ] Download links are valid.
> > > [ ] Checksums and signatures.
> > > [ ] LICENSE/NOTICE files exist
> > > [ ] No unexpected binary files
> > > [ ] Source files have ASF headers
> > > [ ] Can compile from source
> > >
> > > To compile from the source, please refer to:
> > >
> > > https://github.com/apache/incubator-pekko/blob/main/README.md#building-from-source
> > >
> > > Some notes about verifying downloads can be found at:
> > >
> > > https://pekko.apache.org/download.html#verifying-downloads
> > >
> > >
> > > Here is my +1 (binding).
> > >
> > > Thanks,
> > >
> > > PJ Fanning
> > >
> > > ---------------------------------------------------------------------
> > > 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
>

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

Reply via email to