Differential D37496

flua: add a chown(2) binding
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Apr 9, 2:30 PM

	Unknown Object (File)
	Mar 21 2024, 1:19 PM

	Unknown Object (File)
	Dec 30 2023, 3:23 AM

	Unknown Object (File)
	Dec 23 2023, 1:55 AM

	Unknown Object (File)
	Dec 10 2023, 10:11 AM

	Unknown Object (File)
	Nov 17 2023, 3:03 PM

	Unknown Object (File)
	Nov 17 2023, 2:10 PM

	Unknown Object (File)
	Nov 17 2023, 12:46 PM

View All 19 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...