Re: JavaFX fails to build with multiple different issues

2018-12-04 Thread Michael Ennen
Thanks for opening the PR! That's the beauty of the public Github
repository.

The changeset looks good. In order to get it accepted you will need to sign
the OCA (Oracle Contributor Agreement) but Kevin or another Oracle dev
should
help you through that process by commenting on the PR. You took the first
step, so thank you.

As for the gradle task issue the only half-baked suggestion I can offer is
to try
"gradlew all test" instead of "gradle build/clean".

On Tue, Dec 4, 2018 at 3:34 AM Ty Young  wrote:

>
> On 12/3/18 6:52 PM, Michael Ennen wrote:
> > The first one is definitely a bug and I would consider opening a PR to
> > remove
> > the final modifier on https://github.com/javafxports/openjdk-jfx.
>
>
> Done.  This is the
> first time I've ever done a PR request so if there is anything wrong
> besides not signing away my left kidney with my own blood then let me
> know(/joke). I don't think it has any negative impact seeing as how the
> code requires the instance to be mutable anyway unless this really is a
> different code path from before(why would it be?) but I can't exactly
> check either way until I can compile and it's wrong either way.
>
>
>
> >
> > The second one is a gradle error - what gradle command-line invocation
> > are you using?
>
>
> "gradle build" though just doing "gradle" or even "gradle clean" results
> in the same fail. All tasks are borked would be my guess.
>
>
> >
> > On Mon, Dec 3, 2018 at 4:58 PM Ty Young  > > wrote:
> >
> > So here is an odd one: JavaFX is now longer building on Arch Linux
> > even
> > on a previous source directory that did compile just fine. I'm
> > guessing
> > that something was updated and isn't compatible with JavaFX's build
> > scripts but I'm not entirely sure what exactly it is.
> >
> >
> > (Note: just to make sure the files didn't magically become corrupt or
> > something I redownloaded from the Github mirror. Same issue.)
> >
> >
> > First error is this:
> >
> >
> >
>  
> ~/openjdk-jfx-develop/buildSrc/src/main/groovy/com/sun/javafx/gradle/NativeCompileTask.groovy:
> >
> > 118: The variable [files] is declared final but is reassigned
> > . At [118:21]  @ line 118, column 21.
> > files += file;
> >
> >
> > which is easy enough to fix by simply removing final from line
> > 105. Even
> > though this was clearly an issue(why use final on local variables?
> > I've
> > never understood why people do it...), I'm not entirely sure why it's
> > throwing an error now. Is it because Groovy is a scripting
> > language and
> > the build scripts never used to follow that logic path?
> >
> >
> > Next issue is this:
> >
> >
> >  > Could not get unknown property 'moduleEmptyPublicationJarLinux'
> > for
> > task set of type org.gradle.api.internal.tasks.DefaultTaskContainer
> >
> >
> > which I'm not sure on how to resolve. Thinking that it may be
> > related to
> > my custom built JDK I tried swapping it out for an older one that
> > I know
> > worked before and yet it fails all the same.
> >
> >
> >
> > --
> > Michael Ennen
>


-- 
Michael Ennen


Re: JavaFX fails to build with multiple different issues

2018-12-03 Thread Ty Young



On 12/3/18 6:52 PM, Michael Ennen wrote:
The first one is definitely a bug and I would consider opening a PR to 
remove

the final modifier on https://github.com/javafxports/openjdk-jfx.



Done.  This is the 
first time I've ever done a PR request so if there is anything wrong 
besides not signing away my left kidney with my own blood then let me 
know(/joke). I don't think it has any negative impact seeing as how the 
code requires the instance to be mutable anyway unless this really is a 
different code path from before(why would it be?) but I can't exactly 
check either way until I can compile and it's wrong either way.






The second one is a gradle error - what gradle command-line invocation
are you using?



"gradle build" though just doing "gradle" or even "gradle clean" results 
in the same fail. All tasks are borked would be my guess.





On Mon, Dec 3, 2018 at 4:58 PM Ty Young > wrote:


So here is an odd one: JavaFX is now longer building on Arch Linux
even
on a previous source directory that did compile just fine. I'm
guessing
that something was updated and isn't compatible with JavaFX's build
scripts but I'm not entirely sure what exactly it is.


(Note: just to make sure the files didn't magically become corrupt or
something I redownloaded from the Github mirror. Same issue.)


First error is this:



~/openjdk-jfx-develop/buildSrc/src/main/groovy/com/sun/javafx/gradle/NativeCompileTask.groovy:

118: The variable [files] is declared final but is reassigned
. At [118:21]  @ line 118, column 21.
    files += file;


which is easy enough to fix by simply removing final from line
105. Even
though this was clearly an issue(why use final on local variables?
I've
never understood why people do it...), I'm not entirely sure why it's
throwing an error now. Is it because Groovy is a scripting
language and
the build scripts never used to follow that logic path?


Next issue is this:


 > Could not get unknown property 'moduleEmptyPublicationJarLinux'
for
task set of type org.gradle.api.internal.tasks.DefaultTaskContainer


which I'm not sure on how to resolve. Thinking that it may be
related to
my custom built JDK I tried swapping it out for an older one that
I know
worked before and yet it fails all the same.



--
Michael Ennen


Re: JavaFX fails to build with multiple different issues

2018-12-03 Thread Michael Ennen
The first one is definitely a bug and I would consider opening a PR to
remove
the final modifier on https://github.com/javafxports/openjdk-jfx.

The second one is a gradle error - what gradle command-line invocation
are you using?

On Mon, Dec 3, 2018 at 4:58 PM Ty Young  wrote:

> So here is an odd one: JavaFX is now longer building on Arch Linux even
> on a previous source directory that did compile just fine. I'm guessing
> that something was updated and isn't compatible with JavaFX's build
> scripts but I'm not entirely sure what exactly it is.
>
>
> (Note: just to make sure the files didn't magically become corrupt or
> something I redownloaded from the Github mirror. Same issue.)
>
>
> First error is this:
>
>
> ~/openjdk-jfx-develop/buildSrc/src/main/groovy/com/sun/javafx/gradle/NativeCompileTask.groovy:
>
> 118: The variable [files] is declared final but is reassigned
> . At [118:21]  @ line 118, column 21.
> files += file;
>
>
> which is easy enough to fix by simply removing final from line 105. Even
> though this was clearly an issue(why use final on local variables? I've
> never understood why people do it...), I'm not entirely sure why it's
> throwing an error now. Is it because Groovy is a scripting language and
> the build scripts never used to follow that logic path?
>
>
> Next issue is this:
>
>
>  > Could not get unknown property 'moduleEmptyPublicationJarLinux' for
> task set of type org.gradle.api.internal.tasks.DefaultTaskContainer
>
>
> which I'm not sure on how to resolve. Thinking that it may be related to
> my custom built JDK I tried swapping it out for an older one that I know
> worked before and yet it fails all the same.
>
>

-- 
Michael Ennen


JavaFX fails to build with multiple different issues

2018-12-03 Thread Ty Young
So here is an odd one: JavaFX is now longer building on Arch Linux even 
on a previous source directory that did compile just fine. I'm guessing 
that something was updated and isn't compatible with JavaFX's build 
scripts but I'm not entirely sure what exactly it is.



(Note: just to make sure the files didn't magically become corrupt or 
something I redownloaded from the Github mirror. Same issue.)



First error is this:


~/openjdk-jfx-develop/buildSrc/src/main/groovy/com/sun/javafx/gradle/NativeCompileTask.groovy: 
118: The variable [files] is declared final but is reassigned

. At [118:21]  @ line 118, column 21.
   files += file;


which is easy enough to fix by simply removing final from line 105. Even 
though this was clearly an issue(why use final on local variables? I've 
never understood why people do it...), I'm not entirely sure why it's 
throwing an error now. Is it because Groovy is a scripting language and 
the build scripts never used to follow that logic path?



Next issue is this:


> Could not get unknown property 'moduleEmptyPublicationJarLinux' for 
task set of type org.gradle.api.internal.tasks.DefaultTaskContainer



which I'm not sure on how to resolve. Thinking that it may be related to 
my custom built JDK I tried swapping it out for an older one that I know 
worked before and yet it fails all the same.