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

Update API workaround for tycho bundle names containing dashes #1192

Merged
merged 1 commit into from
Mar 11, 2024

Conversation

iloveeclipse
Copy link
Member

Tycho/maven renames original bundles and replaces underscore with dash, so let our "expected" 4.31 bundle file patches have everything with dashes. That should work in IDE / on tycho build.

See eclipse-platform/eclipse.platform.swt#1011

See #1187

Tycho/maven renames original bundles and replaces underscore with dash,
so let our "expected" 4.31 bundle file patches have everything with
dashes. That should work in IDE / on tycho build.

See eclipse-platform/eclipse.platform.swt#1011

See eclipse-pde#1187
Copy link

Test Results

   291 files  ±0     291 suites  ±0   55m 45s ⏱️ - 4m 41s
 3 526 tests ±0   3 468 ✅ ±0   58 💤 ±0  0 ❌ ±0 
10 875 runs  ±0  10 698 ✅ ±0  177 💤 ±0  0 ❌ ±0 

Results for commit 3851c3d. ± Comparison against base commit 2fd4268.

@iloveeclipse iloveeclipse merged commit a6824f7 into eclipse-pde:master Mar 11, 2024
17 checks passed
@iloveeclipse iloveeclipse deleted the issue_1187 branch March 11, 2024 17:35
HannesWell added a commit to HannesWell/eclipse.pde that referenced this pull request Sep 8, 2024
This reverts commit 2fd4268
and commit a6824f7,
introduced with eclipse-pde#1191
and eclipse-pde#1192

These workarounds for missing API description files are not necessary
anymore because they are now correctly included again for some time.
HannesWell added a commit to HannesWell/eclipse.pde that referenced this pull request Sep 9, 2024
This reverts commit 2fd4268
and commit a6824f7,
introduced with eclipse-pde#1191
and eclipse-pde#1192

These workarounds for missing API description files are not necessary
anymore because they are now correctly included again for some time.
HannesWell added a commit to HannesWell/eclipse.pde that referenced this pull request Sep 19, 2024
This reverts commit 2fd4268
and commit a6824f7,
introduced with eclipse-pde#1191
and eclipse-pde#1192

These workarounds for missing API description files are not necessary
anymore because they are now correctly included again for some time.
HannesWell added a commit to HannesWell/eclipse.pde that referenced this pull request Sep 24, 2024
This reverts commit 2fd4268
and commit a6824f7,
introduced with eclipse-pde#1191
and eclipse-pde#1192

These workarounds for missing API description files are not necessary
anymore because they are now correctly included again for some time.
HannesWell added a commit to HannesWell/eclipse.pde that referenced this pull request Sep 24, 2024
This reverts commit 2fd4268
and commit a6824f7,
introduced with eclipse-pde#1191
and eclipse-pde#1192

These workarounds for missing API description files are not necessary
anymore because they are now correctly included again for some time.
HannesWell added a commit to HannesWell/eclipse.pde that referenced this pull request Sep 25, 2024
This reverts commit 2fd4268
and commit a6824f7,
introduced with eclipse-pde#1191
and eclipse-pde#1192

These workarounds for missing API description files are not necessary
anymore because they are now correctly included again for some time.
HannesWell added a commit to HannesWell/eclipse.pde that referenced this pull request Sep 25, 2024
This reverts commit 2fd4268
and commit a6824f7,
introduced with eclipse-pde#1191
and eclipse-pde#1192

These workarounds for missing API description files are not necessary
anymore because they are now correctly included again for some time.
HannesWell added a commit that referenced this pull request Sep 25, 2024
This reverts commit 2fd4268
and commit a6824f7,
introduced with #1191
and #1192

These workarounds for missing API description files are not necessary
anymore because they are now correctly included again for some time.
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.

1 participant