Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JDK 21 (WIP) - Close #3 #4

Merged
merged 8 commits into from
Jan 14, 2025
Merged

JDK 21 (WIP) - Close #3 #4

merged 8 commits into from
Jan 14, 2025

Conversation

manoelcampos
Copy link
Collaborator

@manoelcampos manoelcampos commented Jan 14, 2025

Close #3

Changes so far

Fix gradle build and some deprecations.
To generate the fat jar you have to run ./gradlew shadowJar

It will create a xml-doclet-all.jar inside build/libs.
The doclet is not running yet.
You can start it by typing:

javadoc -doclet 'com.github.markusbernhardt.xmldoclet.XmlDoclet' -docletpath './build/libs/xml-doclet-all.jar' -J-ea -verbose

@manticore-projects, I don't know if you are making some changes too, because that may cause merge conflicts.

Finally, since this is an independent fork, what do you think of detaching it from the original project?
This way, the following options on the repo home page will desapear and it will be easier to open PRs from feature branches (currently, a new PR targets the original repo).

Screenshot 2025-01-13 at 21 50 32

To detach the project, you just need to access https://support.github.com/tickets/personal/0 and click "New Ticket". Then, send a message to the support, informing that now you have an independent fork that you want to detach from the original project.

I'll try to figure out why the doclet is not running. The major challenge is that there is no logs, just a generic error message "error: Could not instantiate class com.github.markusbernhardt.xmldoclet.XmlDoclet"

Signed-off-by: Manoel Campos <[email protected]>
…vy/CustomVersionTask.groovy.

Removes the function implementation from build.gradle

Signed-off-by: Manoel Campos <[email protected]>
Properties such as fromRepo are now a Property object
that has a set method, which must be called
instead of assigning a value to the property directly.

Previously properties such as fromRepo were String, now they are Property<String>,
requiring the set() call.

Signed-off-by: Manoel Campos <[email protected]>
@manoelcampos manoelcampos self-assigned this Jan 14, 2025
@manoelcampos manoelcampos marked this pull request as draft January 14, 2025 00:54
@manoelcampos
Copy link
Collaborator Author

Now instantiation erros are being catch and logged.

@manoelcampos
Copy link
Collaborator Author

jdeps -summary build/libs/xml-doclet-all.jar may help configuring modules

…on name,

that was causing the doclet instantiation to fail,
indicating invalid option name.

Signed-off-by: Manoel Campos <[email protected]>
@manoelcampos
Copy link
Collaborator Author

Doclet is now being instantiated, but we still have this module hell.

@manticore-projects
Copy link
Owner

Greetings!

So the initiation error was on parameters/option only?

but we still have this module hell.

We can remove module-info.java, its not really needed.
I only thought it may contribute to the initiation error.

@manoelcampos
Copy link
Collaborator Author

Now it starts without instantiation errors, but we get lots of runtime errors such as:

build/generated/sources/java/main/com/github/markusbernhardt/xmldoclet/xjc/EnumConstant.java:14: error: package jakarta.xml.bind.annotation does not exist
import jakarta.xml.bind.annotation.XmlAttribute;

Since I'm using the uber jar, the dependencies are expected to be inside it.
All the erros are now only related to jakarta.xml (but the dependency is there on the build.gradle).
I'll keep looking.

@manticore-projects
Copy link
Owner

To detach the project, you just need to access https://support.github.com/tickets/personal/0 and click "New Ticket". Then, send a message to the support, informing that now you have an independent fork that you want to detach from the original project.

Just commit everything straight into the Repo (origin:master). We don't need PRs at that point since this is a very volatile development stage.

@manticore-projects
Copy link
Owner

I am pulling this in already so I can continue working on it.

@manticore-projects manticore-projects marked this pull request as ready for review January 14, 2025 06:18
@manticore-projects manticore-projects merged commit dba47d2 into master Jan 14, 2025
1 check failed
@manticore-projects manticore-projects deleted the issue3 branch January 14, 2025 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

JDK21 wip
2 participants