Differential D34934

sbin/devfs: Correct usage
ClosedPublic
Actions

Authored by zlei on Apr 16 2022, 4:58 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, May 7, 8:02 AM

	Unknown Object (File)
	Thu, May 7, 1:51 AM

	Unknown Object (File)
	Wed, May 6, 3:04 AM

	Unknown Object (File)
	Mon, May 4, 8:50 AM

	Unknown Object (File)
	Sun, May 3, 6:45 AM

	Unknown Object (File)
	Tue, Apr 14, 9:33 PM

	Unknown Object (File)
	Apr 13 2026, 2:28 PM

	Unknown Object (File)
	Mar 25 2026, 6:05 PM

Subscribers

naito.yuichiro_gmail.com

Details

Reviewers

ngie
hrs

Commits

rGac3223a57e11: sbin/devfs: Correct usage
rG66b882078973: sbin/devfs: Correct usage
rGf9f42a709b2b: sbin/devfs: Correct usage

Summary

The -s applies to rule keyword only and it follows the rule keyword.

PR: 263289
Submitted by: Yuichiro NAITO <naito.yuichiro@gmail.com>
Fixes: c3e412c08333 sbin/devfs: clarify usage

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

zlei created this revision.Apr 16 2022, 4:58 AM

Herald added a subscriber: imp. · View Herald TranscriptApr 16 2022, 4:58 AM

zlei requested review of this revision.Apr 16 2022, 4:58 AM

zlei edited the summary of this revision. (Show Details)Apr 16 2022, 5:13 AM

naito.yuichiro_gmail.com added a subscriber: naito.yuichiro_gmail.com.Apr 19 2022, 9:04 AM

zlei added a reviewer: hrs.Apr 24 2022, 1:38 AM

ngie accepted this revision.Apr 30 2022, 5:48 PM

This revision is now accepted and ready to land.Apr 30 2022, 5:48 PM

Closed by commit rGf9f42a709b2b: sbin/devfs: Correct usage (authored by ngie). · Explain WhyApr 30 2022, 6:00 PM

This revision was automatically updated to reflect the committed changes.

ngie added a commit: rGf9f42a709b2b: sbin/devfs: Correct usage.

In retrospect, I should have removed/edited the metadata in the commit message. It looks nice on Phabricator, but it looks less nice in raw git/email form.

In D34934#795609, @ngie wrote:

In retrospect, I should have removed/edited the metadata in the commit message. It looks nice on Phabricator, but it looks less nice in raw git/email form.

I'm OK with this, except that the commit message contains full URL of PR. I thinks a single PR id is enough.

Hi @ngie ,
will this be MFCed to stable branches ? stable/13 and stable/12 also have this issue.

Ping .

zlei added a commit: rG66b882078973: sbin/devfs: Correct usage.Feb 3 2023, 3:58 PM

zlei added a commit: rGac3223a57e11: sbin/devfs: Correct usage.Feb 3 2023, 4:01 PM

Revision Contents
Changeset List

Path

Size

sbin/

devfs/

2 lines

Diff 105612

sbin/devfs/devfs.c

Loading...