Bug 2258853

Summary: syslog-ng: FTBFS in F40/Rawhide on i686
Product: [Fedora] Fedora Reporter: Ben Beasley <code>
Component: syslog-ngAssignee: Peter Czanik <peter>
Status: CLOSED COMPLETED QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: rawhideCC: fweimer, mrunge, peter
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2024-02-05 17:02:27 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 2137512, 2142177    

Description Ben Beasley 2024-01-17 17:13:42 UTC
The syslog-ng package FTBFS on i686. Koschei hasn’t detected this because it doesn’t run scratch builds on that architecture.

Reproducible: Always

Actual Results:  
modules/secure-logging/slog.c: In function 'writeKey':
modules/secure-logging/slog.c:937:63: error: passing argument 4 of 'g_io_channel_write_chars' from incompatible pointer type [-Wincompatible-pointer-types]
  937 |   status = g_io_channel_write_chars(keyfile, key, KEY_LENGTH, &outlen, &error);
      |                                                               ^~~~~~~
      |                                                               |
      |                                                               guint64 * {aka long long unsigned int *}

Expected Results:  
Successful build

Comment 2 Peter Czanik 2024-02-04 19:55:44 UTC
Thanks! Reported to my colleagues upstream last week. There should be a fix in the coming days.

Comment 3 Peter Czanik 2024-02-05 17:02:27 UTC
For now I worked it around with:

ExcludeArch: %{ix86}

I'll remove this line from syslog-ng.spec once a fix is available upstream.

Comment 4 Peter Czanik 2024-02-06 07:58:48 UTC
OK. I got a fix from my colleagues, so ix86 is enabled again, and package submitted to rawhide.

Comment 5 Ben Beasley 2024-02-06 12:53:42 UTC
Nice work! Thanks for following up on these.