-
Notifications
You must be signed in to change notification settings - Fork 116
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
Strange linked-resource error when importing Maven Project #1912
Comments
OK, by much analysis I found out what was triggering this error. Two levels deep into poms we have a This is valid in Maven, so seems to be an m2e bug. Seems like m2e, if it wants to make a linked resource, should just reference the top level file — diff --git a/tools/cldr-code/pom.xml b/tools/cldr-code/pom.xml
index 9dc23550d8..4a479cf3b5 100644
--- a/tools/cldr-code/pom.xml
+++ b/tools/cldr-code/pom.xml
@@ -69,23 +69,6 @@
<resource>
<directory>src/main/resources</directory>
</resource>
- <resource>
- <directory>${project.basedir}/../..</directory>
- <!-- This is put here so that it can be read by CldrUtility, even outside of a jar -->
- <targetPath>${project.build.directory}/classes/org/unicode/cldr/util/data</targetPath>
- <includes>
- <include>LICENSE</include>
- </includes>
- </resource>
- <resource>
- <directory>${project.basedir}/../..</directory>
- <!-- This is put here so that it shows up in the final .jar in an expected spot -->
- <!-- Note: icu4j keeps its license file at the root, /LICENSE. We won't conflict. -->
- <targetPath>META-INF</targetPath>
- <includes>
- <include>LICENSE</include>
- </includes>
- </resource>
</resources>
<plugins>
<plugin> |
Any comments on this one? it seems like a bug. |
Indeed, it's a bug. A tricky one. |
thanks… m2e-core/org.eclipse.m2e.jdt/src/org/eclipse/m2e/jdt/internal/AbstractJavaProjectConfigurator.java Line 907 in 69aafa0
perhaps the link creation could be skipped if the directory in question actually exists? In this case the directory in question is the root of the maven import. I know that Any sense if this bug needs to be filed elsewhere? |
Summary Error
When importing a Maven project tree, an obscure message. The user wasn't trying to create a linked resource, but just to open a project.
The message seems to be coming from tools/cldr-code/pom.xml - project name
cldr-code
which is 2 levels deep from thecldr2
directory mentioned, which is the root project.Side Request
I don't know if this next message is addressed at the correct level of a very complex stack, but: Please consider improving the message reporting structure here with much more context. It's not an actionable error message.
Background
Hello. Another user at least has seen this as well, and it's been reproduced across both the Eclipse IDE and VSCode. I'm including VSCode details because it has stack traces.
Tracking this downstream at https://unicode-org.atlassian.net/browse/CLDR-18205
To reproduce:
Details:
The text was updated successfully, but these errors were encountered: