Page MenuHomeFreeBSD

Allow rc.d/linux to create mount points for Linux-specific filestsystems
AbandonedPublic

Authored by otis on Aug 30 2020, 7:13 PM.
Tags
Referenced Files
Unknown Object (File)
Mon, Nov 11, 9:47 PM
Unknown Object (File)
Sat, Nov 9, 6:23 PM
Unknown Object (File)
Sun, Nov 3, 10:27 AM
Unknown Object (File)
Wed, Oct 30, 9:07 PM
Unknown Object (File)
Sep 22 2024, 8:39 PM
Unknown Object (File)
Sep 22 2024, 8:35 PM
Unknown Object (File)
Sep 22 2024, 8:32 PM
Unknown Object (File)
Sep 22 2024, 6:26 PM

Details

Reviewers
trasz
Group Reviewers
manpages
Summary
  • Allow rc.d/linux to create mount points for Linux-specific filestsystems
  • Document it in linux(4) manpage
  • Clean mandoc error in that manpage

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

otis requested review of this revision.Aug 30 2020, 7:13 PM

Grammo

share/man/man4/linux.4
92

s/does/do/

otis marked an inline comment as done.Aug 31 2020, 6:10 AM
share/man/man4/linux.4
95

I wonder if we should perhaps default to YES? There's an existing problem, previously pointed out by tijl@, in that linux(4) tells the user to install linux_base-c7 package first, and then set linux_enable="YES"; however, install scripts for some of the linux packages might need those directories. I think we should flip the order (first enable, then "service linux start", then install linux_base-c7. Defaulting to YES would simplify the proces by removing one of the steps.

In any case, you also need to add this variable to libexec/rc/rc.conf.

share/man/man4/linux.4
95

OTOH, it might be better to do that (flipping to YES and updating the installation instructions) in a subsequent commit.

otis marked 2 inline comments as done.

Is there something I need to do with it, please?

Hello, and sorry for not getting to this earlier; I’m slowly going through my list of things that need to happen before 13.0.

One problem I have is this causes errors to be printed when the rc script is restarted; this is arguably a bug in install(1). On the other hand, the current script fails in that case anyway...

Is there something I need to do with it, please?

Sigh, sorry, it's my fault; every few months I got reminded about this, then find a problem, try to fix it, and then I get distracted and the cycle repeats.

There is the problem with install(1) described above, and also I'm not sure if we really need additional rc variable here. Given the fact that the patch doesn't apply anymore, can we perhaps just add it to https://reviews.freebsd.org/D32463, or add as a followup?