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

refactor(libreoffice-24.8): update to mesa with glvnd #33890

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

wojciechka
Copy link
Member

No description provided.

Copy link
Contributor

octo-sts bot commented Nov 12, 2024

Gen AI suggestions to solve the build error:

Based on the build error, here are some suggestions to fix the issue:

  1. The main error appears to be a compiler error in static_types.cxx:
error: '(((const rtl::OUString*)(& s_aTypeNames)) != 0)' is not a constant expression

This suggests that the compiler (GCC) is having issues with constexpr evaluation. Here are potential fixes:

  1. Check if we need to update the GCC version in the build environment. Add to environment packages:
- gcc-13
- gcc-13-dev
  1. Add compiler flags to handle C++17/C++20 features properly. Add to the autogen.input:
--with-cpp-version=c++17
  1. The error could be related to missing build dependencies. Add these packages to the environment:
- libstdc++-dev
- libboost-dev
  1. Try adding these compilation flags to autogen.input:
--enable-debug
--disable-optimization

The most likely fix is updating GCC and ensuring C++17 support, as this seems to be a compiler constexpr evaluation issue with string literals.

To debug this locally:

  1. Use melange build --interactive
  2. When the shell opens, navigate to the cppu directory
  3. Run make V=1 to see verbose output
  4. Look at the exact compiler command that fails
  5. Try manually compiling static_types.cxx with different compiler flags

The warning about memory overflow is likely a secondary issue that will be resolved by fixing the main compiler error.

@@ -1,6 +1,6 @@
package:
name: libreoffice-24.8
version: 24.8.3.1
version: 24.8.3.2
Copy link
Member

Choose a reason for hiding this comment

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

this needs to remain the same, when making changes to the melange yaml we need to bump the epoch below.

Copy link
Member Author

Choose a reason for hiding this comment

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

That's needed because the version changed. Otherwise CI won't build it. I needed to do it because automation to update libreoffice did not work, as it's failing in CI.

It's unrelated to my change, but I need to bump it as automation hasn't and otherwise our CI fails.

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.

2 participants