RFR: 8326964: Remove Eclipse Shared Workspaces [v3]

Julian Waters jwaters at openjdk.org
Fri Mar 15 13:17:26 UTC 2024


On Fri, 15 Mar 2024 12:42:17 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> Julian Waters 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 five additional commits since the last revision:
>> 
>>  - Merge branch 'openjdk:master' into patch-15
>>  - Refresh documentation
>>  - Remove Shared Workspaces entirely
>>  - Remove shared targets
>>  - 8326964
>
> make/ide/eclipse/CreateWorkspace.gmk line 192:
> 
>> 190:     # make dist-clean omitted since it would delete the Workspace as well
>> 191:     # $1_PLAIN_MAKE_TARGETS += reconfigure
>> 192:     # $1_PLAIN_MAKE_TARGETS += dist-clean
> 
> Why leave these additions commented out and not just remove them? If you think the comment is important, you can just put it above the PLAIN_MAKE_TARGETS definition.

Fair point, I'll address this now

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18046#discussion_r1526271085


More information about the build-dev mailing list