Skip to content

Conversation

@wiresketch
Copy link
Contributor

#1184

Add Scrollable.setScrollbarsMode(int) method to provide the possibility to disable/enable overlay scrollbars.

Fixes: #1184

@github-actions
Copy link
Contributor

github-actions bot commented Apr 19, 2024

Test Results

   298 files  ±0    298 suites  ±0   1h 8m 3s ⏱️ - 1m 46s
 4 111 tests ±0    171 ✅ ±0   6 💤 ±0  0 ❌ ±0  3 934 🔥 ±0 
15 963 runs  ±0  8 230 ✅ ±0  61 💤 ±0  0 ❌ ±0  7 672 🔥 ±0 

For more details on these errors, see this check.

Results for commit b6b97bd. ± Comparison against base commit 3d87b22.

♻️ This comment has been updated with latest results.

Copy link
Member

@iloveeclipse iloveeclipse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will update PR in a moment with fixed "since" & rebase on master

Add Scrollable.setScrollbarsMode(int) API to provide the possibility
to disable/enable overlay scrollbars.

Fixes #1184
@iloveeclipse
Copy link
Member

@HannesWell : could you please check what is wrong with the jenkins build? I assume something related to the natives rebuild isn't working as expected.

@HannesWell
Copy link
Member

@HannesWell : could you please check what is wrong with the jenkins build? I assume something related to the natives rebuild isn't working as expected.

The native build works fine (it's visible in the pipeline graph overview), but the subsequent Maven build fails due to out of memory issues:
To my knowledge lines like the following indicate that:
[2024-04-24T16:13:32.040Z] /home/jenkins/agent/workspace/eclipse.platform.swt_PR-1188/eclipse.platform.swt@tmp/durable-96f210d6/script.sh: line 2: 944 Killed

I restarted the build. Sometimes this helps, sometimes one has to wait for times with less overall load in the releng Jenkins instance.

@HannesWell
Copy link
Member

I restarted the build. Sometimes this helps, sometimes one has to wait for times with less overall load in the releng Jenkins instance.

The latest Jenkins build succeeded.

@iloveeclipse
Copy link
Member

The latest Jenkins build succeeded

What about mac build on github?

@HannesWell
Copy link
Member

The latest Jenkins build succeeded

What about mac build on github?

I'm unable to access the logs on my phone, but I just restarted the build and it is at least running for longer than the minute it ran before.

@iloveeclipse
Copy link
Member

Mac github build seem to fail also in other PRs, like in #1064.
I don't see why this PR should affect Mac, especially Windows change is same and Linux one that really changes natives is fine. Merging.

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.

Provide Scrollable.setScrollbarsMode to add the possibility to change the scrollbars mode

3 participants