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 for release 4.35 #1614

Merged
merged 2 commits into from
Nov 25, 2024
Merged

Update for release 4.35 #1614

merged 2 commits into from
Nov 25, 2024

Conversation

github-actions[bot]
Copy link
Contributor

A new release milstone was created, please review the changes and merge if appropriate.

Copy link
Contributor Author

github-actions bot commented Nov 24, 2024

Test Results

80 files   -    403  80 suites   - 403   3s ⏱️ - 12m 31s
34 tests  -  4 061  34 ✅  -  4 051  0 💤  -  7  0 ❌  - 3 
34 runs   - 16 139  34 ✅  - 16 046  0 💤  - 90  0 ❌  - 3 

Results for commit 73a3edd. ± Comparison against base commit e6588c2.

This pull request removes 4061 tests.
org.eclipse.swt.graphics.ImageWin32Tests ‑ testImageShouldHaveDimesionAsPerZoomLevel
org.eclipse.swt.tests.junit.DPIUtilTests ‑ scaleDownFloat
org.eclipse.swt.tests.junit.DPIUtilTests ‑ scaleDownFloatArray
org.eclipse.swt.tests.junit.DPIUtilTests ‑ scaleDownInteger
org.eclipse.swt.tests.junit.DPIUtilTests ‑ scaleDownPoint
org.eclipse.swt.tests.junit.DPIUtilTests ‑ scaleDownRectangle
org.eclipse.swt.tests.junit.DPIUtilTests ‑ scaleUpFloat
org.eclipse.swt.tests.junit.DPIUtilTests ‑ scaleUpIntArray
org.eclipse.swt.tests.junit.DPIUtilTests ‑ scaleUpInteger
org.eclipse.swt.tests.junit.DPIUtilTests ‑ scaleUpPoint
…

♻️ This comment has been updated with latest results.

@akurtakov
Copy link
Member

#1615 deserves a minor version bump and the prospects to land it in are good so I'll change the version bump to be a minor one.

@akurtakov akurtakov force-pushed the update_R4.35 branch 3 times, most recently from 3490b02 to 98630d0 Compare November 24, 2024 15:57
@akurtakov
Copy link
Member

Now it fails with:

[ERROR] Failed to execute goal org.eclipse.tycho:tycho-apitools-plugin:4.0.10:verify (verify) on project org.eclipse.swt.gtk.linux.ppc64le: There are API errors:
18:01:42  [ERROR] META-INF/MANIFEST.MF:1 The minor version should be the same for version 3.129.0, since no new APIs have been added since version 3.128.0
18:01:42  [ERROR] -> [Help 1]
18:01:42  org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.eclipse.tycho:tycho-apitools-plugin:4.0.10:verify (verify) on project org.eclipse.swt.gtk.linux.ppc64le: There are API errors:
18:01:42  META-INF/MANIFEST.MF:1 The minor version should be the same for version 3.129.0, since no new APIs have been added since version 3.128.0

@akurtakov
Copy link
Member

I believe we should stop runnint apitools for swt.

@akurtakov
Copy link
Member

What should we do here ?

@merks
Copy link
Contributor

merks commented Nov 25, 2024

Suppress the error...

We really do need to know in the future if some change breaks API. I can't reproduce that error such that I could quick fix it to create the suppression of it. 😞

image

Which version of API tools are being used?

@HannesWell How best to make progress here?

@HannesWell
Copy link
Member

Suppress the error...

Agree.

We really do need to know in the future if some change breaks API. I can't reproduce that error such that I could quick fix it to create the suppression of it. 😞

You can change the environment in the target for the workspace TP and the API baseline explicitly to linux.gtk.riscv64 for example.
And then I get one on Windows for the checked-out linux.gtk.riscv64 fragment the corresponding error:
grafik

which can be suppressed with

    <resource path="META-INF/MANIFEST.MF">
        <filter id="926941240">
            <message_arguments>
                <message_argument value="3.129.0"/>
                <message_argument value="3.128.0"/>
            </message_arguments>
        </filter>
    </resource>

@HannesWell HannesWell force-pushed the update_R4.35 branch 3 times, most recently from 57e3195 to b2bd5ce Compare November 25, 2024 08:56
@akurtakov
Copy link
Member

akurtakov commented Nov 25, 2024

So "only" failing tests. I'm pushing this one. Thanks @HannesWell !

@akurtakov akurtakov merged commit 266cc26 into master Nov 25, 2024
8 of 13 checks passed
@akurtakov akurtakov deleted the update_R4.35 branch November 25, 2024 09:15
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