Differential D37496

flua: add a chown(2) binding
ClosedPublic
Actions

Authored by bapt on Nov 24 2022, 8:16 AM.

Tags

None

Referenced Files

	F132456538: D37496.id113493.diff
	Fri, Oct 17, 2:24 AM

	F132446436: D37496.diff
	Fri, Oct 17, 12:37 AM

	Unknown Object (File)
	Tue, Oct 14, 3:47 PM

	Unknown Object (File)
	Fri, Oct 10, 9:06 PM

	Unknown Object (File)
	Sun, Oct 5, 1:00 AM

	Unknown Object (File)
	Sat, Sep 27, 3:22 PM

	Unknown Object (File)
	Sep 17 2025, 8:13 AM

	Unknown Object (File)
	Sep 13 2025, 4:21 PM

View All 80 Files

Subscribers

Details

Reviewers

kevans
bapt

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 48495
Build 45381: arc lint + arc unit

Event Timeline

bapt created this revision.Nov 24 2022, 8:16 AM

Herald added a subscriber: imp. · View Herald TranscriptNov 24 2022, 8:16 AM

bapt requested review of this revision.Nov 24 2022, 8:16 AM

Harbormaster completed remote builds in B48487: Diff 113479.Nov 24 2022, 8:16 AM

bapt added a reviewer: kevans.Nov 24 2022, 8:16 AM

kevans added inline comments.Nov 24 2022, 2:02 PM

libexec/flua/modules/lposix.c
77	The official luaposix interface actually allows 2 or 3 args, nil for either user/group resulting in -1 -- IMO I think we should do the same. It technically also resolves strings if they're passed, though I don't feel as strongly about that.
81	s/2/3/?

almost follow luaposix, just be stricter and report if a user or group
is valid or not.

Harbormaster completed remote builds in B48495: Diff 113493.Nov 24 2022, 2:40 PM

bapt marked 2 inline comments as done.Nov 24 2022, 2:41 PM

bapt accepted this revision.Nov 25 2022, 7:59 AM

This revision is now accepted and ready to land.Nov 25 2022, 7:59 AM

bapt closed this revision.Nov 25 2022, 7:59 AM

Revision Contents
Changeset List

Path

Size

libexec/

flua/

modules/

59 lines

Diff 113493

libexec/flua/modules/lposix.c

Loading...