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