-
Notifications
You must be signed in to change notification settings - Fork 139
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
Build GTK 4 binaries #1422
Build GTK 4 binaries #1422
Conversation
Test Results 483 files ±0 483 suites ±0 7m 46s ⏱️ -51s For more details on these failures, see this check. Results for commit 9da5b34. ± Comparison against base commit ca1bb19. ♻️ This comment has been updated with latest results. |
This comment was marked as outdated.
This comment was marked as outdated.
Now Github succeed and Jenkins fails as expected:
|
e447b1c
to
397da2e
Compare
Given that I wanted to have a look at some of those deprecation errors in the near future, what version of GTK4 do you plan on compiling against? With #1421, SWT would require at least require 4.10 but then the build would have to continue despite the deprecations. With older versions, the build would succeed but then the PR no longer builds as the new methods don't exist yet. |
As GTK4 is WIP it should be fine to require as new version as needed . By the time this is ready for prime time that version would have made it to mainstream distros. |
Main goal is to build it with "what is available right now", if we have this working one can go step further to support a specific version. But still then one need to take care as using "the latest" might simply make it impossible for some linux distributions (e.g LTE) that not always have latest and greatest of libraries, but "latest stable" version. |
Even though GTK4 was added here: and container build seem to have completed https://ci.eclipse.org/releng/view/Miscellaneous/job/Build-Docker-images/732/ the build still complains:
@fredg02 anything else that needs to be done so the new container is visible? Can we do anything to verify the change is actually used? I didn't find anything in the log that indicates the actual used container has/version/date/.... |
We should have target Linux versions specified in the project plan, similar to Java versions.
|
I would agree with such version targeting when we discuss making GTK 4 default one. Unfortunately we are very far from it so by the time this discussion comes we will most likely have RHEL 10(most likely 11) and Ubuntu 26.04. |
One such impossible feature on RHEL 9 would be Browser support as even though there is gtk4 there is no webkitgtk4. |
That's why I said lets first get something compiled, then its easier to iterate, I tried to add some commands to find out actual installed versions for the github runners it is this versions:
|
ec1b98b
to
60dc09c
Compare
Jenkins CentOS currently using
so it seems docker file change has had no effect... |
https://ci.eclipse.org/releng/view/Miscellaneous/job/Build-Docker-images builds docker images from https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/tree/master/cje-production/dockerfiles. I don't know if there is a Jenkins job that actually builds https://github.com/eclipse-platform/eclipse.platform.swt/tree/master/container. I'd recommend to talk to project members that know more about the platform/swt docker builds. This should be more effective than you or me trying to decipher the inner workings of platform releng builds. I would also recommend to not use a matrix build while trying to get this running. It makes debugging harder than it should be. |
When you have found the right job to build the right docker image, I'd verify that it includes everything that is needed by running it locally. Once that is done, you can specify the sha256 of the image in your Jenkinsfile to make sure that the exact image is pulled. |
@fredg02 So according to https://ci.eclipse.org/releng/job/Build-Docker-images/732/artifact/eclipse.platform.releng.aggregator/cje-production/dockerfiles/swt_9_build.log and https://ci.eclipse.org/releng/job/Build-Docker-images/732/artifact/eclipse.platform.releng.aggregator/cje-production/dockerfiles/push-swt_9_build.log image should be built and pushed but looking at https://hub.docker.com/layers/eclipse/platformreleng-centos-swt-build/9/images/sha256-31294c0be3473791b82c8f6cbb9e9e1bf5ff42701553eed6a5e17f5f427c59e4?context=explore image was last updated 2 months ago. I have no idea how to help more here. |
As noted above, https://ci.eclipse.org/releng/job/Build-Docker-images/ is not building the Dockerfile that @laeubi modified. |
I changed the file here: And Jenkins config says it runs
do I miss something obvious? |
@sravanlakkimsetti can you maybe shade some light here, the Jenkins file config and collection of shellscripts seems not easy to understand :-\ I even wonder if it would not be easier to have one Jeankinsfile (in git) that includes the shell calls directly instead of many different shell scripts with only a few line in it. |
Ok. That was not obvious to me, based on the comments here. |
Finally the docker images was updated and now we have GTK4 at the build node, this is the used GTK version and it shows a lot of deprecation warnings:
See https://ci.eclipse.org/releng/job/eclipse.platform.swt/job/PR-1422/14/pipeline-console/?selected-node=195 |
@fredg02 can I somehow run my docker image on the native (e.g.
|
If the native build agent's have docker installed, we could maybe launch another docker node from that agent? And we could maybe even build a docker-image for SWT on the fly using: If that works as desired/hoped, we could even maintain the docker-image in this repo without the need to push it to docker-hub. |
One drawback is that building a docker image can take some time and requires download of a lot of stuff, so I think using a prebuild image from a registry can still be good. On the other hand on a "real" machine one can probably benefit from caching (in both cases). Anyways as ther is no easy "out of the box" way I first will head on to make the lib available on the machines: |
0be5151
to
6c6e672
Compare
This is now ready to be merged once master is open for the next stream. |
If we can audit that this change doesn't modify any of the already built artifacts (only adds new ones) and that the behavior seems backward compatible; then could we consider a merge ASAP, before next stream starts even? |
This will at least raise the GLIBC version requirements and therefore might not be good in current release phase. |
@mickaelistria This has been already tested and building on newer distros leads to using newer glibc symbols (versioned dlsym/dlopen IIRC) that render old systems no longer loading. So it's best to delay till start of next cycle. |
21fa88d
to
3573d46
Compare
Now the native build succeeds but for some reason the text execution failed in the initialize shell. |
I think the problem is that SWT still uses centos-latest pod: eclipse.platform.swt/Jenkinsfile Line 87 in 6ab4ccd
|
From the log:
Please move to some newer image. Maybe in separate PR and rebase this one on top of it so we deliver that part now independent of gtk4. |
I have created |
Currently the GTK4 binaries are not build as part of the build, this has several drawbacks: 1) If anything is adjusted for GTK4 part it might break without notice 2) We have no GTK4 binaries by default This enables *compilation* of the gtk4 parts to see at least everything can be compiled.
Currently the GTK4 binaries are not build as part of the build, this has several drawbacks:
This enables compilation of the gtk4 parts to see at least everything can be compiled.
The following has to be checked: