Skip to content

libodfgen: fix build with gcc15#446154

Merged
Aleksanaa merged 1 commit into
NixOS:masterfrom
ghpzin:pr-libodfgen-gcc15
Sep 28, 2025
Merged

libodfgen: fix build with gcc15#446154
Aleksanaa merged 1 commit into
NixOS:masterfrom
ghpzin:pr-libodfgen-gcc15

Conversation

@ghpzin

@ghpzin ghpzin commented Sep 25, 2025

Copy link
Copy Markdown
Contributor

Fixes build failure with gcc15:

OdfGenerator.cxx:53:9: error: 'uint8_t' was not declared in this scope
   53 |         uint8_t first;
      |         ^~~~~~~
OdfGenerator.cxx:45:1: note: 'uint8_t' is defined in header '<cstdint>';
this is probably fixable by adding '#include <cstdint>'
   44 | #include "GraphicFunctions.hxx"
  +++ |+#include <cstdint>
   45 | #include "InternalHandler.hxx"
OdfGenerator.cxx:57:17: error: 'first' was not declared in this scope
   57 |                 first = 0;
      |                 ^~~~~

Tested build with:

nix-build --expr 'with import ./. {}; libodfgen.override { stdenv = gcc15Stdenv; }'

Part of fixes for gcc15 update:
#440456

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

- add patch that adds `#include <cstdint>` to `OdfGenerator.cxx`,
based on upstream commit that is not included in tagged versions:
https://sourceforge.net/p/libwpd/libodfgen/ci/4da0b148def5b40ee60d4cd79762c0f158d64cc7/

Fixes build failure with gcc15:
```
OdfGenerator.cxx:53:9: error: 'uint8_t' was not declared in this scope
   53 |         uint8_t first;
      |         ^~~~~~~
OdfGenerator.cxx:45:1: note: 'uint8_t' is defined in header '<cstdint>';
this is probably fixable by adding '#include <cstdint>'
   44 | #include "GraphicFunctions.hxx"
  +++ |+#include <cstdint>
   45 | #include "InternalHandler.hxx"
OdfGenerator.cxx:57:17: error: 'first' was not declared in this scope
   57 |                 first = 0;
      |                 ^~~~~
```
@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. labels Sep 25, 2025
@ghpzin

ghpzin commented Sep 26, 2025

Copy link
Copy Markdown
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 446154
Commit: a336da9b6cac774a6e9aabb31d5e5339013e79eb


x86_64-linux

✅ 29 packages built:
  • collabora-online
  • gotenberg
  • gotenberg.hyphen
  • kdePackages.calligra
  • kdePackages.calligra.debug
  • kdePackages.calligra.dev
  • kdePackages.calligra.devtools
  • libodfgen
  • libreoffice (libreoffice-still)
  • libreoffice-collabora
  • libreoffice-fresh
  • libreoffice-fresh-unwrapped
  • libreoffice-qt (libreoffice-qt-still)
  • libreoffice-qt-fresh
  • libreoffice-qt-fresh-unwrapped
  • libreoffice-qt-unwrapped (libreoffice-qt-still-unwrapped)
  • libreoffice-unwrapped (libreoffice-still-unwrapped)
  • lomiri.lomiri-docviewer-app
  • paperwork
  • paperwork.dist
  • python312Packages.paperwork-backend
  • python312Packages.paperwork-backend.dist
  • python312Packages.paperwork-shell
  • python312Packages.paperwork-shell.dist
  • python313Packages.paperwork-backend
  • python313Packages.paperwork-backend.dist
  • python313Packages.paperwork-shell
  • python313Packages.paperwork-shell.dist
  • unoconv

@ghpzin ghpzin marked this pull request as ready for review September 26, 2025 04:08
@nix-owners nix-owners Bot requested a review from 7c6f434c September 26, 2025 04:09
@Aleksanaa Aleksanaa enabled auto-merge September 28, 2025 10:39
@Aleksanaa Aleksanaa added this pull request to the merge queue Sep 28, 2025
Merged via the queue into NixOS:master with commit d0d9e67 Sep 28, 2025
34 of 36 checks passed
@ghpzin ghpzin deleted the pr-libodfgen-gcc15 branch September 28, 2025 10:47
@fabianhjr fabianhjr mentioned this pull request Dec 1, 2025
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants