-
Notifications
You must be signed in to change notification settings - Fork 31
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
Use XMLBeans' BindingConfigImpl in Axis2BindingConfig instead of XSDConfig, +doc. #589
base: master
Are you sure you want to change the base?
Conversation
Can you rebase onto the current HEAD? |
7868dee
to
1268640
Compare
Sure, but it didn't seem to improve the situation. Some errors I got:
Are there some obvious build instructions I'm maybe missing? |
As mentioned in the |
Not reading the README, typical. 😑 Sorry about that. Next set of errors:
Not sure what's going here, the CI build doesn't seem to have this problem. I commented out this test and then got:
This looks like I may be running a too old Java version (11). That turned out to be the case, trying again with Java 17 it works. Perhaps the maven-enforcer-plugin configuration in the main POM should be updated?
|
1268640
to
b96efd4
Compare
About the documentation... I noticed (and rebased onto) the commit removing the Eclipse repository. That's better, but:
|
b96efd4
to
a8358f4
Compare
So I actually tested my code now, and I found a silly mistake. Could have known it, it was in the only code that's different from my 1.7.9 patch... Fixed by new commit. How hard would it be to write a test case? If it's not too much effort I'd like to make one, but I have no idea in which module and how. |
Can someone give me a hint about |
a8358f4
to
d002117
Compare
I managed to generate generate-sources-mojo.html, seems OK. Trouble with site generation is reported at https://issues.apache.org/jira/browse/AXIS2-6066. So I think it's ready for review. |
As discussed on the -user mailing list, here is my proposal to use XMLBeans' own implementation of the .xsdconfig file. Changes are mostly to Axis2BindingConfig, which now keeps a BindingConfigImpl instance created from the given .xsdconfig file.
I have not been able to test this though! While I could patch version 1.7.9 with essentially the same changes, I can't get the code in master to compile. When I disabled one module with a compilation error, another missing resource or test error showed up, and I gave up.