RFR: 8169588: [AOT] jaotc --classpath option is confusing

[Rickard Bäckman]

On 01/05, Vladimir Kozlov wrote:

Vladimir,

I removed the lines.

We just check the different possibilities, the first thing that matches
is used. The current order is:

module
class name
jar file
directory

We could always change it so that it prints an error if multiple things
matches (example: java.base is a module and a directory in the users
current directory).

http://cr.openjdk.java.net/~rbackman/8169588.2/

/R

> Hi Rickard,
> 
> You can remove commented lines Main.java which use ClassCollector.
> 
> I am not sure how you separate module name from directory name without using --module flag. You removed --module from test.
> 
> Thanks,
> Vladimir
> 
> On 1/5/17 6:12 AM, Rickard Bäckman wrote:
> >Noticed that I forgot to add changes to test cases in the webrev.
> >So updated.
> >
> >http://cr.openjdk.java.net/~rbackman/8169588.1/
> >
> >/R
> >
> >On 01/05, Rickard Bäckman wrote:
> >>Hi,
> >>
> >>can I please have this change reviewed?
> >>
> >>It changes how jaotc handles command line options.
> >>With this change it handles:
> >>
> >>jaotc /tmp/some/directory /tmp/some/directory2/
> >>jaotc /tmp/some/file.jar /tmp/foo/bar.jar
> >>jaotc java.base
> >>
> >>You can also use a new flag --search-path
> >>
> >>jaotc --search-path /tmp/jarfiles:/tmp/jarfiles2 file1.jar file2.jar
> >>
> >>or mix the kind of things you are trying to compile
> >>
> >>jaotc /tmp/foo/dir /tmp/bar/foobar.jar java.base
> >>
> >>https://bugs.openjdk.java.net/browse/JDK-8169588
> >>http://cr.openjdk.java.net/~rbackman/8169588/
> >>
> >>Thanks
> >>/R