Paths

Table of Contentst

Differential D44871

adduser: A few more improvements.
ClosedPublic
Actions

Authored by des on Apr 19 2024, 4:04 PM.

Tags

None

Referenced Files

	F166620820: D44871.id137396.diff
	Fri, Aug 14, 10:24 PM

	F166620683: D44871.diff
	Fri, Aug 14, 10:22 PM

	Unknown Object (File)
	Wed, Aug 12, 8:09 PM

	Unknown Object (File)
	Tue, Aug 11, 4:34 PM

	Unknown Object (File)
	Mon, Aug 10, 10:21 AM

	Unknown Object (File)
	Mon, Aug 10, 9:39 AM

	Unknown Object (File)
	Mon, Aug 10, 9:18 AM

	Unknown Object (File)
	Sun, Aug 9, 3:29 PM

Subscribers

Details

Reviewers

Commits

rGedff88aee215: adduser: Really fix a syntax error
rG9e8618276fad: adduser: Really fix a syntax error
rG5cafc38f1129: adduser: Fix a syntax error
rGb37333899bd2: adduser: A few more improvements.

Summary

MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 57222
Build 54110: arc lint + arc unit

Event Timeline

des created this revision.Apr 19 2024, 4:04 PM

Herald added a subscriber: imp. · View Herald TranscriptApr 19 2024, 4:04 PM

des requested review of this revision.Apr 19 2024, 4:04 PM

Harbormaster completed remote builds in B57222: Diff 137396.Apr 19 2024, 4:04 PM

Looks good to me.

In D44871#1023012, @jrm wrote:

Looks good to me.

Then accept it, please.

jrm accepted this revision.Apr 20 2024, 4:47 PM

This revision is now accepted and ready to land.Apr 20 2024, 4:47 PM

Closed by commit rGb37333899bd2: adduser: A few more improvements. (authored by des). · Explain WhyApr 20 2024, 7:36 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGb37333899bd2: adduser: A few more improvements..

jlduran added a subscriber: jlduran.Apr 22 2024, 6:50 PM

jlduran added inline comments.

usr.sbin/adduser/adduser.sh
480	Sorry, for the late review, as this is already committed, but I do not understand this change. Wouldn't it throw an error?

jrm added inline comments.Apr 22 2024, 6:55 PM

usr.sbin/adduser/adduser.sh
480–481	It looks good to me. It's now saying if the zfs kernel module isn't loaded or the user opted not to have the zfs home dataset created.

jrm added inline comments.Apr 22 2024, 7:01 PM

usr.sbin/adduser/adduser.sh
480	Oh, I see what you're saying. The syntax is wrong. The `; then` is missing.

jlduran added inline comments.Apr 22 2024, 7:02 PM

usr.sbin/adduser/adduser.sh
480	Precisely! You beat me to it. Thank you!
480–481	I guess, what I meant was, isn't it missing a `; then` and indenting with 4 spaces instead of a tab the next line?

jrm added a commit: rG5cafc38f1129: adduser: Fix a syntax error.Apr 22 2024, 7:09 PM

des mentioned this in rG4f4905b2ef1c: adduser: Overhaul..Apr 29 2024, 10:15 AM

des added a commit: rG9e8618276fad: adduser: Really fix a syntax error.Apr 29 2024, 10:30 AM

des added a commit: rGedff88aee215: adduser: Really fix a syntax error.

des mentioned this in rGb7a56907af0c: adduser: Overhaul..Apr 29 2024, 10:34 AM

Revision Contents
Changeset List

Path

Size

usr.sbin/

adduser/

24 lines

Diff 137396

usr.sbin/adduser/adduser.sh

Loading...