Re: [VOTE] Release Apache Commons JXPath 1.4.0 based on RC2

2025-04-17 Thread Rob Tompkins
+1 all looks good.

-Rob

> On Apr 13, 2025, at 7:04 PM, Gary Gregory  wrote:
> 
> We have fixed a few bugs and added enhancements since Apache Commons
> JXPath 1.3 was released, so I would like to release Apache Commons
> JXPath 1.4.0.
> 
> Apache Commons JXPath 1.4.0 RC2 is available for review here:
>https://dist.apache.org/repos/dist/dev/commons/jxpath/1.4.0-RC2
> (svn revision 76234)
> 
> The Git tag commons-jxpath-1.4.0-RC2 commit for this RC is
> 146f2534e885fd7085fba4bf3fb658d434416504 which you can browse here:
>
> https://gitbox.apache.org/repos/asf?p=commons-jxpath.git;a=commit;h=146f2534e885fd7085fba4bf3fb658d434416504
> You may checkout this tag using:
>git clone https://gitbox.apache.org/repos/asf/commons-jxpath.git
> --branch commons-jxpath-1.4.0-RC2 commons-jxpath-1.4.0-RC2
> 
> Maven artifacts are here:
>
> https://repository.apache.org/content/repositories/orgapachecommons-1824/commons-jxpath/commons-jxpath/1.4.0/
> 
> These are the artifacts and their hashes:
> 
> #Release SHA-512s
> #Sun Apr 13 22:56:27 UTC 2025
> commons-jxpath-1.4.0-bom.json=91035b52e7305884f1f283c54f6d3f1255cac8bdd18b62820fac07bc3e872d1b1baf9d76a14c62d94ba08fe9ca9e19222c6983cdac3b67d6eaab7b2a2056eb16
> commons-jxpath-1.4.0-bom.xml=6f9735d831e24c0811d9aa041913e8dcb4b8492984307134582198155d503c37e0775d51eab8c9cbdfdd30638ac1522651dbe65aadd365d0074391082ce24782
> commons-jxpath-1.4.0-javadoc.jar=1b97abd849e96d7a45ccdd69e4726b3c9c8cc34ee10dd3bea1fc1fffa8f91b586a6ec253f78dde5dcb8af7587963708dc4a377c3fe77d97cb1061d05b11e8eff
> commons-jxpath-1.4.0-sources.jar=918fd91e9a74c71ab1b0531e0013738eb78d5591debaa75323ac8d731f5845ad45657322666bc73b9640a977947321eda3c737545c0fa0c43a1f95c20bfeee95
> commons-jxpath-1.4.0-src.tar.gz=fd18abb89fa37f6d78e906a4e65af54a4d8424949f01df1166e6ce1ef7a1938f07290bba15e14185fcdd108a4836243f94dcff77f8bbd98ae4b2c7c546607416
> commons-jxpath-1.4.0-src.zip=3be62ff300cda63463f21015dacdd96d7574c4e391b10aeff1817050190056d98f67027e27c0fbea027dcb6b550bd11c53dc1cbf0727214ecefa3e0b40a8c05d
> commons-jxpath-1.4.0-test-sources.jar=385552bdbcd65da1c54f5150537ec123008fb9a56e1fb070f105d81b679613ce5ef4b97c2042d2f29a9d1931257b1a2d418e195774efa14edbcbb5ab3851d7c5
> commons-jxpath-1.4.0-tests.jar=0590d15d5c120a70ad6dc8d6529bce855cbc31fd716c7df6fdb9abf25dce1052defd7365bac24820d2680f1194f1dce850ac939d05052821c717bbd7d6ba6f3a
> commons-jxpath_commons-jxpath-1.4.0.spdx.json=f569e60e85882d4bfb1134c4b0c00384daa42a272ac2d6077b9fee1d233932a9ebc6466f520298eda27c4d22167423a5fce488dd61ef9a69a09493b8473ba58b
> 
> 
> I have tested this with 'mvn' and 'mvn -e -V -P release -P test-deploy
> -P jacoco -P japicmp clean package site deploy' using:
> 
> openjdk version "21.0.6" 2025-01-21
> OpenJDK Runtime Environment Homebrew (build 21.0.6)
> OpenJDK 64-Bit Server VM Homebrew (build 21.0.6, mixed mode, sharing)
> 
> Apache Maven 3.9.9 (8e8579a9e76f7d015ee5ec7bfcdc97d260186937)
> Maven home: /opt/homebrew/Cellar/maven/3.9.9/libexec
> Java version: 21.0.6, vendor: Homebrew, runtime:
> /opt/homebrew/Cellar/openjdk@21/21.0.6/libexec/openjdk.jdk/Contents/Home
> Default locale: en_US, platform encoding: UTF-8
> OS name: "mac os x", version: "15.4", arch: "aarch64", family: "mac"
> 
> Darwin .local 24.4.0 Darwin Kernel Version 24.4.0: Wed Mar 19
> 21:17:35 PDT 2025; root:xnu-11417.101.15~1/RELEASE_ARM64_T6041 arm64
> 
> Docker version 28.0.4, build b8034c0
> 
> 
> Details of changes since 1.3 are in the release notes:
>
> https://dist.apache.org/repos/dist/dev/commons/jxpath/1.4.0-RC2/RELEASE-NOTES.txt
>
> https://dist.apache.org/repos/dist/dev/commons/jxpath/1.4.0-RC2/site/changes.html
> 
> Site:
>
> https://dist.apache.org/repos/dist/dev/commons/jxpath/1.4.0-RC2/site/index.html
>(note some *relative* links are broken and the 1.4.0 directories
> are not yet created - these will be OK once the site is deployed.)
> 
> JApiCmp Report (compared to 1.3):
>
> https://dist.apache.org/repos/dist/dev/commons/jxpath/1.4.0-RC2/site/japicmp.html
> 
>Note that the above report notes several errors.
>These are considered OK for the reasons stated below.
> 
>Errors reported:
>- False positive:
> org.apache.commons.jxpath.JXPathContextFactoryConfigurationError#getMessage()
> is supported through the superclass.
>- False positive:
> org.apache.commons.jxpath.JXPathException#getCause() is supported
> through the superclass.
>- False positive:
> org.apache.commons.jxpath.JXPathException#getMessage() is supported
> through the superclass.
> 
> RAT Report:
>
> https://dist.apache.org/repos/dist/dev/commons/jxpath/1.4.0-RC2/site/rat-report.html
> 
> KEYS:
>  https://downloads.apache.org/commons/KEYS
> 
> Please review the release candidate and vote.
> This vote will close no sooner than 72 hours from now.
> 
>  [ ] +1 Release these artifacts
>  [ ] +0 OK, but...
>  [ ] -0 OK, but really should fix...
>  [ ] -1 I oppose this release because...
> 
> Thank you,
> 
> Gary Gregory,
> Release Manager (us

[RDF] ModelGraphImpl returns 'parallelStream()' instead of expected 'stream()'

2025-04-17 Thread Fred Hauschel

Hi there, in my opinion this is a bug:

 * 
https://github.com/apache/commons-rdf/blob/006fbc3e47003c2d0fe8eb055f60f7cb267e1321/commons-rdf-rdf4j/src/main/java/org/apache/commons/rdf/rdf4j/impl/ModelGraphImpl.java#L142
 * 
https://github.com/apache/commons-rdf/blob/006fbc3e47003c2d0fe8eb055f60f7cb267e1321/commons-rdf-rdf4j/src/main/java/org/apache/commons/rdf/rdf4j/impl/ModelGraphImpl.java#L150

Because nobody expects a parallel stream to be created and returned when 
stream() is called.
There is also a great danger that this parallel stream will be used to 
perform write operations with RDF4J. However, RDF4J is not thread-safe 
and this can lead to non-deterministic results.


What is your opinion on this?

Thanks Fredy


[RDF] PR - GraphLike with addAll

2025-04-17 Thread Fred Hauschel

Hi there,

reworked my request from last year 



Looking foreward to your feedback.

Thanks Fredy



[RDF] PR - upgrade rdf4j to 5.1.3

2025-04-17 Thread Fred Hauschel

Hi there,

i upgraded the rdf4j version to 5.1.3 (Tue, Apr 15, 2025)

Looking forward to your feedback.

PR 324 

Thanks Fredy


Re: [RDF] PR - GraphLike with addAll

2025-04-17 Thread Gary Gregory
Fredy,

Would you mind investigating how to fix the build on Java 21 and above
before we make any changes?

TY!

Gary

On Thu, Apr 17, 2025, 13:17 Fred Hauschel  wrote:

> Hi there,
>
> reworked my request from last year
> 
>
> Looking foreward to your feedback.
>
> Thanks Fredy
>
>


RE: Re: [RDF] Problems building master

2025-04-17 Thread Fred Hauschel

Hi Gary,
i thought that. The info in the Readme is wrong:


However, if i build it with

$ java -version
openjdk version "11.0.12" 2021-07-20
OpenJDK Runtime Environment 18.9 (build 11.0.12+7)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.12+7, mixed mode)

I stuck with:

$ mvn clean install
...
...
[INFO] --- install:3.1.2:install(default-install)@ 
commons-rdf-integration-tests---
[INFO] 


[INFO] Reactor Summary for Commons RDF 0.6.0-SNAPSHOT:
[INFO]
[INFO] Commons RDF  SUCCESS[ 
 8.523 s]
[INFO] Commons RDF API  SUCCESS[ 
 9.577 s]
[INFO] Commons RDF impl: Simple ... SUCCESS[ 
 7.771 s]
[INFO] Commons RDF impl: RDF4j  SUCCESS[ 
11.064 s]
[INFO] Commons RDF impl: Jena . SUCCESS[ 
 7.915 s]
[INFO] Commons RDF impl: JSON-LD Java . SUCCESS[ 
 7.146 s]
[INFO] Commons RDF Integration tests .. FAILURE[ 
 3.169 s]
[INFO] 


[INFO] BUILD FAILURE
[INFO] 


[INFO] Total time:  55.423 s
[INFO] Finished at: 2025-04-17T10:55:57+02:00
[INFO] 

[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-install-plugin:3.1.2:install(default-install)on 
project commons-rdf-integration-tests: The packaging plugin for project 
commons-rdf-integration-tests did not assign a main
file to the project but it has attachments. Change packaging to 'pom'.-> 
[Help 1]


Best regards
Fredy

On 2025/04/16 19:07:27 Gary Gregory wrote:
> Hi Fredy,
>
> GitHub shows builds are green on Java 11 and 17:
> https://github.com/apache/commons-rdf/actions
>
> Some unit tests are failing on Java 21 and up.
>
> Java 8 is no longer supported in git master (see the POM compiler 
settings).

>
> HTH,
> Gary
>
> On Wed, Apr 16, 2025 at 1:50 PM Fred Hauschel  wrote:
> >
> > Hi there,
> >
> > haven't heard from me for a long time. I'm trying to think my way back
> > into commons-rdf. I made a copy years ago that I'm building on, it's
> > time to change that and switch to the original version.
> > I just built the tag 0.5.0 successfully. but when I build master, I get
> > an error.
> >
> > Used Java Version:
> >
> > $ java -version
> > openjdk version "1.8.0_302"
> > OpenJDK Runtime Environment (build 1.8.0_302-b08)
> > OpenJDK 64-Bit Server VM (build 25.302-b08, mixed mode)
> >
> >
> > $ mvn clean install
> > ...
> > ...
> > [INFO]
> > 


> > [INFO] Reactor Summary for Commons RDF 0.6.0-SNAPSHOT:
> > [INFO]
> > [INFO] Commons RDF  FAILURE[
> > 3.334 s]
> > [INFO] Commons RDF API  SKIPPED
> > [INFO] Commons RDF impl: Simple ... SKIPPED
> > [INFO] Commons RDF impl: RDF4j  SKIPPED
> > [INFO] Commons RDF impl: Jena . SKIPPED
> > [INFO] Commons RDF impl: JSON-LD Java . SKIPPED
> > [INFO] Commons RDF Integration tests .. SKIPPED
> > [INFO]
> > 


> > [INFO] BUILD FAILURE
> > [INFO]
> > 


> > [INFO] Total time: 3.577 s
> > [INFO] Finished at: 2025-04-16T19:48:05+02:00
> > [INFO]
> > 


> > [ERROR] Failed to execute goal
> > 
org.codehaus.mojo:animal-sniffer-maven-plugin:1.24:check(checkAPIcompatibility)on

> > project commons-rdf-parent: Failed to obtain signature:
> > org.codehaus.mojo.signature:java110:1.0: The following artifacts
> > could not be resolved: 
org.codehaus.mojo.signature:java110:signature:1.0

> > (absent): Could not find artifact
> > org.codehaus.mojo.signature:java110:signature:1.0 in central
> > (https://repo.maven.apache.org/maven2) -> [Help 1]
> >
> >
> > Any Idea ??
> >
> > Thanks Fredy
> >
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>

RE: RE: Re: [RDF] Problems building master

2025-04-17 Thread Fred Hauschel
Aha, i never used maven without specifying a goal, but commons-rdf is 
building when just running 'mvn'.


Magic happens, but the build is green.

br Fredy


On 2025/04/17 08:59:45 Fred Hauschel wrote:
> Hi Gary,
> i thought that. The info in the Readme is wrong:
>
>
> However, if i build it with
>
> $ java -version
> openjdk version "11.0.12" 2021-07-20
> OpenJDK Runtime Environment 18.9 (build 11.0.12+7)
> OpenJDK 64-Bit Server VM 18.9 (build 11.0.12+7, mixed mode)
>
> I stuck with:
>
> $ mvn clean install
> ...
> ...
> [INFO] --- install:3.1.2:install(default-install)@
> commons-rdf-integration-tests---
> [INFO]
> 
> [INFO] Reactor Summary for Commons RDF 0.6.0-SNAPSHOT:
> [INFO]
> [INFO] Commons RDF  SUCCESS[
>  8.523 s]
> [INFO] Commons RDF API  SUCCESS[
>  9.577 s]
> [INFO] Commons RDF impl: Simple ... SUCCESS[
>  7.771 s]
> [INFO] Commons RDF impl: RDF4j  SUCCESS[
> 11.064 s]
> [INFO] Commons RDF impl: Jena . SUCCESS[
>  7.915 s]
> [INFO] Commons RDF impl: JSON-LD Java . SUCCESS[
>  7.146 s]
> [INFO] Commons RDF Integration tests .. FAILURE[
>  3.169 s]
> [INFO]
> 
> [INFO] BUILD FAILURE
> [INFO]
> 
> [INFO] Total time:  55.423 s
> [INFO] Finished at: 2025-04-17T10:55:57+02:00
> [INFO]
> 
> [ERROR] Failed to execute goal
> 
org.apache.maven.plugins:maven-install-plugin:3.1.2:install(default-install)on 


> project commons-rdf-integration-tests: The packaging plugin for project
> commons-rdf-integration-tests did not assign a main
> file to the project but it has attachments. Change packaging to 'pom'.->
> [Help 1]
>
> Best regards
> Fredy
>
> On 2025/04/16 19:07:27 Gary Gregory wrote:
> > Hi Fredy,
> >
> > GitHub shows builds are green on Java 11 and 17:
> > https://github.com/apache/commons-rdf/actions
> >
> > Some unit tests are failing on Java 21 and up.
> >
> > Java 8 is no longer supported in git master (see the POM compiler
> settings).
> >
> > HTH,
> > Gary
> >
> > On Wed, Apr 16, 2025 at 1:50 PM Fred Hauschel  
wrote:

> > >
> > > Hi there,
> > >
> > > haven't heard from me for a long time. I'm trying to think my way 
back

> > > into commons-rdf. I made a copy years ago that I'm building on, it's
> > > time to change that and switch to the original version.
> > > I just built the tag 0.5.0 successfully. but when I build master, 
I get

> > > an error.
> > >
> > > Used Java Version:
> > >
> > > $ java -version
> > > openjdk version "1.8.0_302"
> > > OpenJDK Runtime Environment (build 1.8.0_302-b08)
> > > OpenJDK 64-Bit Server VM (build 25.302-b08, mixed mode)
> > >
> > >
> > > $ mvn clean install
> > > ...
> > > ...
> > > [INFO]
> > >
> 
> > > [INFO] Reactor Summary for Commons RDF 0.6.0-SNAPSHOT:
> > > [INFO]
> > > [INFO] Commons RDF  FAILURE[
> > > 3.334 s]
> > > [INFO] Commons RDF API  SKIPPED
> > > [INFO] Commons RDF impl: Simple ... SKIPPED
> > > [INFO] Commons RDF impl: RDF4j  SKIPPED
> > > [INFO] Commons RDF impl: Jena . SKIPPED
> > > [INFO] Commons RDF impl: JSON-LD Java . SKIPPED
> > > [INFO] Commons RDF Integration tests .. SKIPPED
> > > [INFO]
> > >
> 
> > > [INFO] BUILD FAILURE
> > > [INFO]
> > >
> 
> > > [INFO] Total time: 3.577 s
> > > [INFO] Finished at: 2025-04-16T19:48:05+02:00
> > > [INFO]
> > >
> 
> > > [ERROR] Failed to execute goal
> > >
> 
org.codehaus.mojo:animal-sniffer-maven-plugin:1.24:check(checkAPIcompatibility)on

> > > project commons-rdf-parent: Failed to obtain signature:
> > > org.codehaus.mojo.signature:java110:1.0: The following artifacts
> > > could not be resolved:
> org.codehaus.mojo.signature:java110:signature:1.0
> > > (absent): Could not find artifact
> > > org.codehaus.mojo.signature:java110:signature:1.0 in central
> > > (https://repo.maven.apache.org/maven2) -> [Help 1]
> > >
> > >
> > > Any Idea ??
> > >
> > > Thanks Fredy
> > >
> >
> > -
> > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> > For additional commands, e-mail: dev-h...@commons.apache.org
> >
> >

---