RFR: 8290473: update Eclipse .classpath in apps, buildSrc [v6]

[Andy Goryachev]

On Tue, 9 Aug 2022 15:14:15 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Andy Goryachev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains eight additional commits since the last revision:
>> 
>>  - 8290473: junit 5
>>  - Merge remote-tracking branch 'origin/master' into 8290473.apps
>>  - 8290473: removed eclipse project in buildSrc
>>  - 8290473: whitespace
>>  - 8290473: added initDirs task
>>  - Merge remote-tracking branch 'origin/master' into 8290473.apps
>>  - 8290473: added ColorCube project
>>  - 8290473: eclipse config for apps
>
> build.gradle line 4176:
> 
>> 4174: 
>> 4175: task sdk() {
>> 4176:     dependsOn(initDirs)
> 
> I think this should be done in the graphics project, closer to the build logic for the shaders, maybe something like this:
> 
> 
> --- a/build.gradle
> +++ b/build.gradle
> @@ -2501,6 +2501,15 @@ project(":graphics") {
>          }
>      }
> 
> +    task initShaderDirs() {
> +        doLast {
> +            // Create empty hlsl dirs on all platforms for IDE support
> +            file("$project.buildDir/hlsl/Decora").mkdirs()
> +            file("$project.buildDir/hlsl/Prism").mkdirs()
> +        }
> +    }
> +    project.processShaders.dependsOn(initShaderDirs)
> +
>      nativePrism.dependsOn compilePrismHLSLShaders;
> 
>      project.nativeAllTask.dependsOn nativeDecora

the reason I put them at the top level is to avoid a situation when gradle thinks that the graphics task is up-to-date (ex.: after a branch change) but the directories are not there.
but you are right, this is a better place to add.  thank you.

-------------

PR: https://git.openjdk.org/jfx/pull/858