Page MenuHomeFreeBSD
Differential D42711

lockf: hide unavailable error with -n -s
ClosedPublic
Actions

Authored by kevans on Nov 22 2023, 5:11 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Sep 18, 11:35 PM
Unknown Object (File)
Wed, Sep 18, 2:16 AM
Unknown Object (File)
Tue, Sep 17, 1:48 PM
Unknown Object (File)
Tue, Sep 17, 1:06 AM
Unknown Object (File)
Mon, Sep 16, 10:16 PM
Unknown Object (File)
Sun, Sep 15, 12:42 PM
Unknown Object (File)
Fri, Sep 13, 3:08 AM
Unknown Object (File)
Tue, Sep 10, 5:18 PM
View All 49 Files
Subscribers
0mp
allanjude
des
imp

Details

Reviewers
0mp
allanjude
Group Reviewers
Klara
Commits
rG3041e6950d07: lockf: hide unavailable error with -n -s
Summary

The error message is expected, allow -s to suppress just that one since
it would loosely fall under the definition of "failure to acquire the
lock" described in the manpage for the -s option.

Sponsored by: Klara, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 54559
Build 51448: arc lint + arc unit

Event Timeline

kevans created this revision.Nov 22 2023, 5:11 AM
Herald added a subscriber: imp. · View Herald TranscriptNov 22 2023, 5:11 AM
kevans requested review of this revision.Nov 22 2023, 5:11 AM
Harbormaster completed remote builds in B54559: Diff 130415.Nov 22 2023, 5:11 AM
kevans added a child revision: D42712: lockf: correct omission in the usage statement.Nov 22 2023, 5:12 AM
0mp accepted this revision.Nov 22 2023, 9:23 AM
0mp added a subscriber: 0mp.

LGTM.

This revision is now accepted and ready to land.Nov 22 2023, 9:23 AM
allanjude accepted this revision.Nov 22 2023, 1:39 PM
allanjude added a subscriber: allanjude.

reviewed-by: allanjude

des added a subscriber: des.Nov 22 2023, 2:34 PM
des added inline comments.
usr.bin/lockf/lockf.c
182

This is a bit repetitive, I would prefer if (!silent) warn(); exit();.

kevans marked an inline comment as done.Nov 22 2023, 4:35 PM
kevans added inline comments.
usr.bin/lockf/lockf.c
182

Ah, yeah, I like that more. Fixed locally, thanks.

Closed by commit rG3041e6950d07: lockf: hide unavailable error with -n -s (authored by kevans). · Explain WhyNov 26 2023, 4:11 AM
This revision was automatically updated to reflect the committed changes.
kevans marked an inline comment as done.
kevans added a commit: rG3041e6950d07: lockf: hide unavailable error with -n -s.

Revision Contents
Changeset List

PathSize
usr.bin/
lockf/
16 lines

Diff 130415

View Options

usr.bin/lockf/lockf.c

Loading...