RFR: 8289255: update Eclipse .classpath and other configuration files [v3]

Nir Lisker nlisker at openjdk.org
Tue Jul 5 21:49:45 UTC 2022


On Tue, 5 Jul 2022 18:49:40 GMT, Andy Goryachev <duke at openjdk.org> wrote:

>> 1. The way the apps are structured is that each app is its own project, which means that there are many new projects to create and configure. It can be a bit of work, though I have done it in my branch already. If you want to leave it to round 2 of fixing the Eclipse files it's fine.
>> 
>> 2+3. I worked directly from master and from your branch, so no files have been changed. The folders under `build` are OS-specific and are generated as needed by the build. Since you are on Mac, it would explain why the hlsl folders are empty, but they are not empty on Windows. This is why they are marked as "ignore_optional_problems" - to not produce errors even on the wrong system. Since they don't contain Java classes that are referenced by the build (as far as I could tell), it should be safe to remove the `build/...` source folders in general. I don't know if on Mac you get any issues, but I assume you don't.
>
> 1. I would rather leave the apps/ for day 2, if I may.
> 2+3.  Currently, on Mac, I don't see any issues related to build/... folders.  I think, generally speaking, it's a bad idea to refer to derived folders as source - it complicates the build process in Eclipse.  Does my branch fail on windows in Eclipse, Nir?  I mean, do I need to make changes to address this comment?

I don't get any errors on the graphics module, I'm suggesting that you remove the entries:

	<classpathentry kind="src" path="build/gensrc/jsl-prism"/>
	<classpathentry kind="src" path="build/gensrc/jsl-decora"/>

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

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


More information about the openjfx-dev mailing list