Page MenuHomeFreeBSD

Move all devfs related files to sbin/devfs/
ClosedPublic

Authored by brd on Aug 18 2018, 10:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 12, 12:11 AM
Unknown Object (File)
Tue, Mar 12, 12:11 AM
Unknown Object (File)
Tue, Mar 12, 12:11 AM
Unknown Object (File)
Tue, Mar 12, 12:02 AM
Unknown Object (File)
Tue, Mar 12, 12:02 AM
Unknown Object (File)
Tue, Mar 12, 12:02 AM
Unknown Object (File)
Fri, Mar 8, 4:05 AM
Unknown Object (File)
Jan 6 2024, 8:45 AM
Subscribers

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Sorta weird to set an empty FILES=, perhaps bluetooth.device.conf should be moved first, so this file can be moved along with deleting the etc/defaults directory altogether.

In D16785#357233, @will wrote:

Sorta weird to set an empty FILES=, perhaps bluetooth.device.conf should be moved first, so this file can be moved along with deleting the etc/defaults directory altogether.

We do it in many places already, is it worth worrying about for a few days until the other move happens? The bluetooth move is a relatively bigger move so I was waiting a bit for that.

In D16785#357262, @brd wrote:
In D16785#357233, @will wrote:

Sorta weird to set an empty FILES=, perhaps bluetooth.device.conf should be moved first, so this file can be moved along with deleting the etc/defaults directory altogether.

We do it in many places already, is it worth worrying about for a few days until the other move happens? The bluetooth move is a relatively bigger move so I was waiting a bit for that.

Not sure about the "many places" bit, but it's just unnecessary churn. You're going to want to do the bluetooth move very soon anyway. After that, it is a trivial update to this diff.

In D16785#357332, @will wrote:
In D16785#357262, @brd wrote:
In D16785#357233, @will wrote:

Sorta weird to set an empty FILES=, perhaps bluetooth.device.conf should be moved first, so this file can be moved along with deleting the etc/defaults directory altogether.

We do it in many places already, is it worth worrying about for a few days until the other move happens? The bluetooth move is a relatively bigger move so I was waiting a bit for that.

Not sure about the "many places" bit, but it's just unnecessary churn. You're going to want to do the bluetooth move very soon anyway. After that, it is a trivial update to this diff.

Ok, opened D16833 for the bluetooth move.

Update after bluetooth move

Update the mode for devfs.rules

It is not clear from the diff, does this change include deleting etc/defaults/* as well?

Missed svn rm etc/defaults

In D16785#358386, @will wrote:

It is not clear from the diff, does this change include deleting etc/defaults/* as well?

Good catch, I missed that.

This revision is now accepted and ready to land.Aug 21 2018, 9:19 PM
This revision was automatically updated to reflect the committed changes.