RFR: JDK-8065912 Better handling of classpath in build-infra

[Erik Joelsson]

Thanks, looks good.

/Erik

On 2015-09-09 09:35, Magnus Ihse Bursie wrote:
> On 2015-09-09 09:25, Erik Joelsson wrote:
>> In JavaCompilation.gmk, could you add the new CLASSPATH parameter to 
>> the documentation comment?
> Of course.
>
> diff --git a/make/common/JavaCompilation.gmk 
> b/make/common/JavaCompilation.gmk
> --- a/make/common/JavaCompilation.gmk
> +++ b/make/common/JavaCompilation.gmk
> @@ -403,6 +403,7 @@
>  #   SRC:=one or more directories to search for sources. The order of 
> the source roots
>  #        is significant. The first found file of a certain name has 
> priority.
>  #   BIN:=store classes here
> +#   CLASSPATH:=a list of additional entries to set as classpath to javac
>  #   INCLUDES:=myapp.foo means will only compile java files in 
> myapp.foo or any of its sub-packages.
>  #   EXCLUDES:=myapp.foo means will do not compile java files in 
> myapp.foo or any of its sub-packages.
>  #   COPY:=.prp means copy all prp files to the corresponding package 
> in BIN.
>
>> Otherwise, very nice!
> Thanks!
>
> /Magnus