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

Resolve classpath entries against project base directory #3818 #3819

Conversation

HeikoKlare
Copy link
Contributor

When using a linked .classpath file, the path entries within that file are resolved against the directory containing the .classpath file instead of the project base directory into which the .classpath file is linked.

With this change, the paths within a .classpath file are resolved against a passed EclipseProject. An existing regression test for linked .classpath files is enhanced to not contain all linked resources in the same folder (thus hiding the bug of using the folder containing the linked .classpath file for resolving classpath entry paths).

Fixes #3818

…o#3818

When using a linked .classpath file, the path entries within that file
are resolved against the directory containing the .classpath file
instead of the project base directory into which the .classpath file is
linked.

With this change, the paths within a .classpath file are resolved
against a passed EclipseProject. An existing regression test for linked
.classpath files is enhanced to not contain all linked resources in the
same folder (thus hiding the bug of using the folder containing the
linked .classpath file for resolving classpath entry paths).

Fixes eclipse-tycho#3818
Copy link

github-actions bot commented May 5, 2024

Test Results

  594 files  ±0    594 suites  ±0   3h 38m 58s ⏱️ - 5m 41s
  419 tests ±0    412 ✅ ±0   7 💤 ±0  0 ❌ ±0 
1 257 runs  ±0  1 235 ✅ ±0  22 💤 ±0  0 ❌ ±0 

Results for commit 8f702f9. ± Comparison against base commit 8ef98a0.

@HeikoKlare HeikoKlare marked this pull request as ready for review May 5, 2024 17:25
@laeubi laeubi added the backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change label May 5, 2024
@laeubi laeubi merged commit 7490426 into eclipse-tycho:main May 5, 2024
12 checks passed
@eclipse-tycho-bot
Copy link

💚 All backports created successfully

Status Branch Result
tycho-4.0.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Linked .classpath files: linked classpath entries paths not resolve against project base directory
3 participants