RFR: 8258499: JavaFX: Move src.zip out of the lib directory
    Marius Hanl 
    mhanl at openjdk.java.net
       
    Wed Jun 30 16:13:03 UTC 2021
    
    
  
On Wed, 30 Jun 2021 15:06:50 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
> The JavaFX SDK bundle provides a set of modular jars in the `lib` directory. It provides a `src.zip` file for use by IDEs in that same `lib` directory. If a developer adds the `lib` directory to their application's module path in the IDE, it will try to load `src.zip` as if it were a jar file, and will fail. This is a pain point for developers using the SDK. 
> 
> The proposed solution is to move the `src.zip` file from the lib directory to the top directory of the SDK.
> 
> With this fix, the layout of the SDK will be:
> 
> 
> <sdk>/
>     bin/                      (Windows)
>         <native libraries>    (Windows)
> 
>     legal/
>         ...
> 
>     lib/
>         *.jar
>         <native libraries>    (macOS and Linux)
> 
>     src.zip
> 
> 
> NOTES:
> 1. The JavaFX Eclipse plugin will need a minor change, which is tracked by eclipse-efx/efxclipse-eclipse#85.
> 2. This still needs to be tested on IntelliJ
I just tried it with IntelliJ. 
I did the following steps:
1. I build the PR with **gradle sdk**
2. I copied the content of the sdk folder into an own '**jfx-17-internal**' folder.
3. I added the **lib** folder as library in IntelliJ<details><summary>Screenshot</summary></details>
4. I navigated into a class file (**Stage** in this case) and clicked on '**Choose sources...**' (top right)<details><summary>Screenshot</summary></details>
5. I chose the **src.zip** which is now inside the root folder<details><summary>Screenshot</summary></details>
6. I can now see the sources (in my case: **Stage**)<details><summary>Screenshot</summary>

</details>
Is this sufficient/Do I miss something or is this fine as test?
-------------
PR: https://git.openjdk.java.net/jfx/pull/558
    
    
More information about the openjfx-dev
mailing list