Skip to content

Conversation

@gezero
Copy link
Contributor

@gezero gezero commented Dec 2, 2025

Configure the maven-jar-plugin to add an Automatic-Module-Name entry (org.locationtech.proj4j.geoapi) to the JAR manifest.

This provides a stable and explicit JPMS module name for consumers using the Java module system, instead of relying on the automatically derived name based on the JAR file name. It also improves Gradle JPMS support: Gradle only puts dependencies on the module path if they either contain a module-info.class or declare an Automatic-Module-Name. Without this entry, this artifact is always kept on the classpath in modular Gradle builds and cannot be required by name from module-info.java.

@pomadchin
Copy link
Member

pomadchin commented Dec 2, 2025

Hi @gezero that's fantastic; I can cut a release later this week to include the change; thank you for the contribution.

Also, could I ask you to sign an ECA please? https://github.com/locationtech/proj4j/blob/master/CONTRIBUTING.md#sign-off-on-commits-and-filing-a-cla

@gezero
Copy link
Contributor Author

gezero commented Dec 2, 2025

I tried to sign the ECA, I need an account first, but for some reason I cannot log in after I registered. I asked the webmaster for help. I am not sure how long until this gets resolved, sorry.

@pomadchin
Copy link
Member

pomadchin commented Dec 2, 2025

@gezero I see; thank you for trying; mb the pwd reset might help? 👀 so sad unfortunately.

@gezero
Copy link
Contributor Author

gezero commented Dec 2, 2025

Hmm, I tried that, but reset password does not send anything to me (not in spam, not in inbox). I tried to ask the webmaster. They have some no dot in email policy that might be relevant, hard to say before I get a reply from them.

@pomadchin
Copy link
Member

I might grab the diff from this PR and make a separate PR if that becomes a blocker 👍

@gezero
Copy link
Contributor Author

gezero commented Dec 2, 2025

Sounds good to me, thank you!

@gezero
Copy link
Contributor Author

gezero commented Dec 2, 2025

There was a minor outage in their account creation workflow that prevented new accounts from completing. The issue has been resolved and hopefully, I accepted the ECA successfully.

@pomadchin
Copy link
Member

pomadchin commented Dec 2, 2025

@gezero could you git commit --amend && git push origin -f master to re-trigger CI?

Configure the maven-jar-plugin to add an Automatic-Module-Name entry
(org.locationtech.proj4j.geoapi) to the JAR manifest.

This provides a stable and explicit JPMS module name for consumers using
the Java module system, instead of relying on the automatically derived
name based on the JAR file name. It also improves Gradle JPMS support:
Gradle only puts dependencies on the module path if they either contain
a module-info.class or declare an Automatic-Module-Name. Without this
entry, this artifact is always kept on the classpath in modular Gradle
builds and cannot be required by name from module-info.java.

Signed-off-by: Jiri Peinlich <[email protected]>
@pomadchin
Copy link
Member

@gezero interesting, it says that no ECA is signed for your jiri.pe***@gm*il account.

@gezero
Copy link
Contributor Author

gezero commented Dec 2, 2025

They call it, working as expected:
https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/6449

I had to use email address without the dot (.) -> [email protected] to register with EF

@pomadchin
Copy link
Member

pomadchin commented Dec 2, 2025

oooh ic, the bot now wants you to sign the commit with this new email without a dot.

@pomadchin
Copy link
Member

I'm sorry that this bureaucracy is causing troubles 😞

@pomadchin
Copy link
Member

But I can confirm, i see the ECA signed for the email address without a dot!

@gezero
Copy link
Contributor Author

gezero commented Dec 2, 2025

I commented on the issue here: https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/6292
You are welcome to just take the diff.

@pomadchin
Copy link
Member

@gezero no worries, will sort it out. thank you so much and really appreciate you taking everything possible to overcome the issue.

@pomadchin pomadchin merged commit f34c589 into locationtech:master Dec 2, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants