[GitHub] [cayenne] aarrsseni commented on issue #382: Cayenne 4.2.M1 testing

[GitBox <gi...@apache.org>]

aarrsseni commented on issue #382: Cayenne 4.2.M1 testing
URL: https://github.com/apache/cayenne/pull/382#issuecomment-489304213
 
 
   Hi, John
   - Since 4.2 you can provide your own properties to cgen. This mechanic isn't described in docs. 
   - Ant's cgen task is really broken. But the problem is not in adding cgenModule to Ant task.
   ServiceLoader doesn't work in Ant tasks. If you try to load services from code everything will be ok, but from task nothing will be found. Because of it autoloaded cgen module can't be added to Ant. Maybe you have some tips how to make CgenToolsModuleProvider visible to Ant task(is everything ok with ServiceLoader from Ant task?).
   If we fix loading cgen module to Ant you can create your own autoloaded module with custom properties and provide them to cgen. You can look to cgen module and PropertyDescriptorCreator to see how it works. 
   - Another option is to create this properties in joda-module. According to this you can be able to add dependency of joda-module and this properties can be used.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services