Bug 2151353

Summary: vttest: FIONREAD not detected by configure script
Product: [Fedora] Fedora Reporter: Florian Weimer <fweimer>
Component: vttestAssignee: Florian Weimer <fweimer>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: dickey
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: vttest-2.7.20220827-2.fc38 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2022-12-07 05:52:01 UTC Type: Bug
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: 2141798    
Attachments:
Description Flags
vttest-configure-c99.patch none

Description Florian Weimer 2022-12-06 20:09:22 UTC 
Created attachment 1930563 [details]
vttest-configure-c99.patch

I'm filing this under the C99 project, but it's a bug today, with current GCC. The configure script does not detect FIONREAD support because on GNU/Linux, it's defined in <sys/ioctl.h>:

“
checking for strerror... yes
checking if POSIX VDISABLE symbol should be used... yes
checking if we may use FIONREAD... no
configure: creating ./config.status
config.status: creating makefile
”

<https://kojipkgs.fedoraproject.org//packages/vttest/2.7.20220827/1.fc38/data/logs/x86_64/build.log>

I saw this because the test flags the implicit declaration of ioctl.

I assume this patch is okay, and I'm going to push it to rawhide.

(And I'm treating this as the de-facto upstream bug tracker. 8-)
Comment 1 Thomas E. Dickey 2022-12-06 21:14:33 UTC 
Looks ok (I see that the check has been working for BSDs).
Comment 2 Florian Weimer 2022-12-07 05:52:01 UTC 
(In reply to Thomas E. Dickey from comment #1)
> Looks ok (I see that the check has been working for BSDs).

Thank you!