-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add libcurl, openssl and libbreakpad to the base image #13
Comments
I’ll take a look this evening. |
Thanks! It's likely some missed files, or an environment variable that we haven't populated. |
Some more information on what I'm building: https://docs.sentry.io/platforms/native/guides/qt/ |
Can you try adding |
Sure, although that will break building with the normal toolchain. Ideally, the toolchain should just work for cmake. I'll report back when it's done. |
That gets us further now. I'll sort out how to handle building in both toolchains in my package recipie. Next up is that the toolchain is missing curl: https://github.com/Eeems-Org/oxide/runs/4746173317 |
That’s a good point. I see that in recent versions (⩾3.21), CMake reads the CMAKE_TOOLCHAIN_FILE variable from environment variables. We can add this to the image. |
Since this requires upgrading CMake, I’m rebuilding the base image to use the latest Debian release. This will take a few hours, so I’ll work on adding libcurl tomorrow. |
Thank you! |
Regarding setting a default value in the environment for As you’ve mentioned, having such a behavior differ from the oecore toolchain means both toolchains need to be handled differently. This may not be desirable. But the oecore behavior is really not ideal for projects having a dual-compiler layout. [1] I have encountered such cases before. This can be used for example to generate some code that is used for the second half of the build. |
As a compromise, we may add some |
Fair enough. Even if we don't go with the script, we should properly document this to lower the barrier to entry for people wanting to package things with our toolchain. |
Definitely. Referencing #12 as a reminder. |
Unfortunately the latest GCC 11 release breaks the build of glibc 2.31 in several ways, including a build-time segfault that I didn’t manage to resolve yet. This may take more time than initially planned. |
@matteodelabre have you had a chance to look at this again? |
Not yet. It’s still on my list but I have limited time available currently. |
Finally managed to make the build work again on the latest Debian… by pinning GCC to version 10. This should allow moving forward for now. Ideally reMarkable would update their glibc to the latest version soon so that we can move to more recent versions of GCC. |
Is there a new image tag I can use? |
Should be available on |
Just added libcurl and OpenSSL to |
@matteodelabre https://github.com/Eeems-Org/oxide/runs/5324827668
Which is odd since it should be building it as part of the compile. |
Looks like the |
That might be why. I guess I'll have to add building it into the steps |
Since it’s available on the rM toolchain I could also add it to ours. |
Just published another |
@matteodelabre Running it again |
Looks like that did it! @matteodelabre |
Excellent! I’ll prepare a new release then. |
Now available under the 2.3 tag. |
Closing this, will track adding docs for using CMake in #12. |
See https://github.com/Eeems/oxide/runs/4736055110
It's provided in the reMarkable toolchain.
The text was updated successfully, but these errors were encountered: