Skip to content

Conversation

@bryancall
Copy link
Contributor

@bryancall bryancall commented Nov 19, 2025

Description

Fix build failures on Ubuntu 20.04 in header_rewrite.

Changes

  1. header_rewrite: Fix format specifier for ip_source bitfield
    • Changed from %d to %u with explicit cast to unsigned int
    • Ensures cross-platform compatibility (Ubuntu and Fedora have different bitfield promotion rules)

Testing

@bryancall bryancall self-assigned this Nov 19, 2025
@bryancall bryancall added Build work related to build configuration or environment CI labels Nov 19, 2025
@bryancall bryancall added this to the 10.2.0 milestone Nov 19, 2025
@bryancall
Copy link
Contributor Author

[approve ci]

@bryancall bryancall force-pushed the fix-ubuntu-build branch 3 times, most recently from c11bd76 to 4bd8093 Compare November 20, 2025 19:33
Use PRIu64 with explicit cast to uint64_t to match the bitfield's underlying
type and ensure cross-platform compatibility. This fixes build failures on
Ubuntu 20.04.
@maskit
Copy link
Member

maskit commented Nov 20, 2025

It looks like either the description is outdated or you forgot to git add the other change (tcpinfo).

@bryancall
Copy link
Contributor Author

bryancall commented Nov 20, 2025

@maskit I removed the tcpinfo change in this commit, I am fixing only one issue at a time and will see what CI does next. I updated the description of the PR.

@bryancall bryancall merged commit 775a324 into apache:master Nov 21, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build work related to build configuration or environment CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants