CreateInterface for firewalld Creates Class / Package Name clash about dbus-java HOT 6 CLOSED

Could you provide the introspection xml generated by DBus?

I don't use Fedora/firewalld and I've started a re-write of the CreateInterface tool and some more test data would be helpful.

from dbus-java.

Attached. If it helps I'm using firewalld on Mint for this.

firewalld_dbus_xml.zip

from dbus-java.

I've identified a problem I introduced. Because the Structs can now be in a package, the import for that package needs to be added to the generated interface class. For some interfaces/paths, the Struct definition has not yet been created, so the 'import xxx' statement cannot be written. I think that means that code generation needs to be a 2 pass process, but I am still trying to think of alternatives.

from dbus-java.

Fix offered in #36

from dbus-java.

I have added a new branch with a completely rewritten version of the interface creation tool.
If you could take a look, that would be great.

The branch name is 'interface-tool', the class you have to run is org.freedesktop.dbus.utils.generator.InterfaceCodeGenerator

Please note that this tool is not done yet. It still lacks support for multi-value return and may have some serious bugs.
I only tested it with a few DBus services and with some of your provided introspection XMLs.

If you find bugs and want to provide a fix for that, you are always welcome.

from dbus-java.

The new InterfaceCodeGenerator is now part of the latest release.
I'll close this bug report, please open a new one if you find bugs in the new implementation.

The old tool is still available, but deprecated.

from dbus-java.