Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename Gradle project name when conflict #2190

Merged
merged 1 commit into from
Aug 24, 2022

Conversation

CsCherrYY
Copy link
Contributor

@CsCherrYY CsCherrYY commented Aug 18, 2022

Signed-off-by: Shi Chen [email protected]

related to microsoft/vscode-java-pack#1040

When we open a Gradle project having the same name in the rootfolder and subfolder, the language server will report the following error:

image

The root cause is that eclipse resource system doesn't allow the duplicate names, so the proposal here is that we can rename the project name with ${rootProject.name}-${project.path} when conflicts are detected. The new project name looks like this:

image

TODOs:

@CsCherrYY CsCherrYY marked this pull request as ready for review August 22, 2022 08:18
@snjeza
Copy link
Contributor

snjeza commented Aug 22, 2022

This PR uses a similar Gradle init mechanism with #2189 so will rebase after it gets merged.]

#2189 has been merged,

@CsCherrYY
Copy link
Contributor Author

@snjeza Yeah, I have rebased this PR, feel free to review :)

@CsCherrYY CsCherrYY merged commit c29b0f5 into eclipse-jdtls:master Aug 24, 2022
@CsCherrYY CsCherrYY deleted the cs-duplicate-project-name branch August 24, 2022 02:10
@testforstephen testforstephen added this to the End August 2022 milestone Aug 24, 2022
gayanper pushed a commit to gayanper/eclipse.jdt.ls that referenced this pull request Sep 8, 2022
rename project name when duplicate regardless case

Signed-off-by: Shi Chen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants