per-module module-source-path for javac

2016-12-12 Thread Nikolay Chashnikov
Hello, we're actively working on support for Jigsaw in IntelliJ IDEA. Some time ago we asked (see this post ) about restrictions in --module-source-path parameter of javac command line. It appears that currently javac has

Re: per-module module-source-path for javac

2016-12-16 Thread Nikolay Chashnikov
Any comments? On Mon, Dec 12, 2016 at 8:29 PM, Nikolay Chashnikov < nikolay.chashni...@jetbrains.com> wrote: > Hello, > > we're actively working on support for Jigsaw in IntelliJ IDEA. Some time > ago we asked (see this post >

Re: per-module module-source-path for javac

2016-12-16 Thread Jonathan Gibbons
Nikolay, We are looking at this. One aspect of the problem we have to take into account is that currently, the JavaFileManager API does not special case the module source path (yes, I know the *implementation* special cases it, even at the handleOption level) and so we need to come up with a

Re: per-module module-source-path for javac

2016-12-16 Thread Nikolay Chashnikov
Hi Jonathan, thank you, glad to hear it. Regarding API design: we already have JavaFileManager#getLocationForModule which takes Location and moduleName, so it looks logical to add setLocationForModule which takes location, moduleName and list of paths, as you suggested. It would be useful not onl

Re: per-module module-source-path for javac

2016-12-16 Thread Jonathan Gibbons
Thank you for noting the additional use case of CLASS_OUTPUT. -- Jon On 12/16/16 8:57 AM, Nikolay Chashnikov wrote: Hi Jonathan, thank you, glad to hear it. Regarding API design: we already have JavaFileManager#getLocationForModule which takes Location and moduleName, so it looks logical t