Bug 136305
Summary: | gdbm prefers flock over fcntl when being built | ||
---|---|---|---|
Product: | Red Hat Enterprise Linux 3 | Reporter: | Ralph Angenendt <ralph+rh-bugzilla> |
Component: | gdbm | Assignee: | Jeff Johnson <jbj> |
Status: | CLOSED WONTFIX | QA Contact: | Jay Turner <jturner> |
Severity: | low | Docs Contact: | |
Priority: | medium | ||
Version: | 3.0 | CC: | redhat, srevivo, tao |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | i386 | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2005-09-29 20:56:47 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: | |||
Attachments: |
Description
Ralph Angenendt
2004-10-19 09:50:08 UTC
Created attachment 105433 [details]
Patch for systems.h which checks for fcntl first
When being rebuild with this patch, gdbm uses fcntl over flock, so that posix
locking is possible
See also bug #123415 Created attachment 108815 [details]
patch to allow gdbm to use fcntl even if flock is present
This patch adds a configure script option to gdbm so that even if flock is
detected during configuration, fcntl will be used to lock files. This will
allow the package to be rebuilt to work over NFS by adding a
--enable-force-fcntl flag on the %configure line of the gdbm spec file.
Is there any reason ever to prefer flock to fcntl if both are available? If not, I would prefer the first approach - it's better to automatically always make the right decision rather requiring the person building the package to override the wrong decision via --enable-force-fcntl. Unless you have some other reasons I haven't thought of for taking this latter approach? Created attachment 108884 [details]
Fixed version of patch for systems.h which checks for fcntl first
The first patch didn't apply to 1.8.0 cleanly for me. Attaching a version
which does; should be identical in functionality.
|