-
Notifications
You must be signed in to change notification settings - Fork 137
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
Remove obsolete API filter for GCData and Display on MacOS #1478
Remove obsolete API filter for GCData and Display on MacOS #1478
Conversation
039111e
to
3527c09
Compare
Test Results 483 files ±0 483 suites ±0 7m 56s ⏱️ - 1m 0s For more details on these failures, see this check. Results for commit b27a408. ± Comparison against base commit e03196c. ♻️ This comment has been updated with latest results. |
3527c09
to
d081891
Compare
@laeubi can you say whether this is somehow expected? I am not sure which PDE change introduced (or rather fixed) this. But since the recent Tycho update (eclipse-platform/eclipse.platform.releng.aggregator#2360), SWT builds fail because of this, e.g., https://github.com/eclipse-platform/eclipse.platform.swt/actions/runs/10941734968/job/30377080455?pr=1378
|
@HeikoKlare I can't tell you exactly why but it seems more that after a release some filters become no longer required (maybe because there is a new API baseline now), but I never have digged to much into this and just removed items that are declared no longer required. |
Alright, then it's probably fine to remove them. Even though in this case it's not a consequence of a release and thus a change in the API baseline, but a consequence of using Tycho 4.0.9 instead of 4.0.8. Anyway, there currently is another build issue introduced by the Tycho version bump (eclipse-platform/eclipse.platform.releng.aggregator#2360 (comment)), which needs to be fixed before merging this PR makes sense. |
API Tools is always a mystery ;-) I can only tell that 4.0.8 by default ships with the API-Tools 2024-03 and 4.0.9 uses API-Tools 2024-09 as a default, maybe something has changed there, I just noticed similar messages in the past. |
d081891
to
5dc86aa
Compare
Since recent PDE and Tycho changes, the API filters for GCData and Display on MacOS are not necessary anymore and produce warnings due to being obsolete. This removes the according filters.
5dc86aa
to
b27a408
Compare
Since recent PDE and Tycho changes, the API filters for GCData and Display on MacOS are not necessary anymore and produce warnings due to being obsolete. This removes the according filters.