Page MenuHomeFreeBSD
Differential D28097

include tmpfs mount into FILESYSTEMS
ClosedPublic
Actions

Authored by glebius on Jan 11 2021, 7:17 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 27, 11:28 AM
Unknown Object (File)
Sun, Oct 27, 11:28 AM
Unknown Object (File)
Sun, Oct 27, 11:28 AM
Unknown Object (File)
Sun, Oct 27, 11:27 AM
Unknown Object (File)
Sun, Oct 27, 11:10 AM
Unknown Object (File)
Sep 29 2024, 9:03 AM
Unknown Object (File)
Sep 29 2024, 3:40 AM
Unknown Object (File)
Sep 29 2024, 3:38 AM
View All 70 Files
Subscribers
lwhsu

Details

Reviewers
des
imp
bz
ngie
obrien
brooks
Commits
rG4f48fd7c5677: Add 'tmp' to the list of FILESYSTEMS dependencies. Some scripts that
rG7e4eca7136ea: Revert 97ec6eba653a07. There shouldn't be a dependency of 'tmp' on
Summary

Add 'tmp' to the list of FILESYSTEMS dependencies. Some scripts that depend on FILESYSTEMS run mktemp(1). For systems that have read-only root this is broken until memory disk based /tmp is instantiated. At least 'os-release' and 'motd' are subject to this.

The reviewers list is based on the history of the file. More reviewers are welcome.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 36116
Build 33005: arc lint + arc unit

Event Timeline

glebius requested review of this revision.Jan 11 2021, 7:17 PM
glebius created this revision.
Harbormaster completed remote builds in B36111: Diff 82088.Jan 11 2021, 7:17 PM
glebius retitled this revision from Add 'tmp' to the list of FILESYSTEMS dependencies. Some scripts that depend on FILESYSTEMS run mktemp(1). For systems that have read-only root this is broken until memory disk based /tmp is instantiated. At least 'os-release' and 'motd' are... to include tmpfs mount into FILESYSTEMS.Jan 11 2021, 7:18 PM
glebius edited the summary of this revision. (Show Details)
glebius edited the summary of this revision. (Show Details)
glebius updated this revision to Diff 82095.Jan 11 2021, 8:16 PM
  • Revert 97ec6eba653a07. There shouldn't be a dependency of 'tmp' on
  • Add 'tmp' to the list of FILESYSTEMS dependencies. Some scripts that
Harbormaster completed remote builds in B36116: Diff 82095.Jan 11 2021, 8:16 PM
ngie accepted this revision.Jan 11 2021, 9:02 PM
ngie added a subscriber: lwhsu.

Could you please update the commit message to reflect the changes being made?

@lwhsu : it's unfortunate that 97ec6eba653a07 doesn't automatically get linked to this commit.

This revision is now accepted and ready to land.Jan 11 2021, 9:02 PM
ngie added a comment.EditedJan 11 2021, 9:03 PM

@glebius: please also run rcorder libexec/rc/rc.d/* to confirm that the changes work.

glebius added a reviewer: brooks.Jan 11 2021, 9:04 PM
Closed by commit rG7e4eca7136ea: Revert 97ec6eba653a07. There shouldn't be a dependency of 'tmp' on (authored by glebius). · Explain WhyJan 15 2021, 8:45 PM
This revision was automatically updated to reflect the committed changes.
glebius added a commit: rG7e4eca7136ea: Revert 97ec6eba653a07. There shouldn't be a dependency of 'tmp' on.
glebius added a commit: rG4f48fd7c5677: Add 'tmp' to the list of FILESYSTEMS dependencies. Some scripts that.

Revision Contents
Changeset List

PathSize
libexec/
rc/
rc.d/
2 lines
2 lines

Diff 82095

View Options

libexec/rc/rc.d/FILESYSTEMS

Loading...
View Options

libexec/rc/rc.d/tmp

Loading...