Skip to content

rust: exclude DETECT_BYTEMATH_ENDIAN_DEFAULT from bindings - #15591

Closed
inashivb wants to merge 1 commit into
OISF:mainfrom
inashivb:cbindgen-byteendian/v1
Closed

rust: exclude DETECT_BYTEMATH_ENDIAN_DEFAULT from bindings#15591
inashivb wants to merge 1 commit into
OISF:mainfrom
inashivb:cbindgen-byteendian/v1

Conversation

@inashivb

Copy link
Copy Markdown
Member

No description provided.

to deal with the failure due to cbindgen updates and mismatches in
generated bindings.

detect-bytemath.c:61: error: "DETECT_BYTEMATH_ENDIAN_DEFAULT" redefined [-Werror]
   61 | #define DETECT_BYTEMATH_ENDIAN_DEFAULT (uint8_t) BigEndian
      |
In file included from rust.h:34,
                 from detect-bytemath.c:32:
./../rust/gen/rust-bindings.h:5071: note: this is the location of the previous definition
 5071 | #define DETECT_BYTEMATH_ENDIAN_DEFAULT BigEndian
      |
@suricata-qa

Copy link
Copy Markdown

Information: QA skipped due to no C or rust code changed detected. Set to force a run.

Pipeline = code

@inashivb
inashivb marked this pull request as ready for review June 10, 2026 06:51
@inashivb
inashivb requested a review from jasonish as a code owner June 10, 2026 06:51
@victorjulien victorjulien added this to the 9.0 milestone Jun 10, 2026
@victorjulien

Copy link
Copy Markdown
Member

Merged in #15601, thanks!

@inashivb
inashivb deleted the cbindgen-byteendian/v1 branch June 12, 2026 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants