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