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
F82005356: D28097.id82339.diff
Wed, Apr 24, 10:17 AM
F81950722: D28097.id82088.diff
Tue, Apr 23, 4:07 PM
Unknown Object (File)
Mar 7 2024, 4:20 PM
Unknown Object (File)
Feb 23 2024, 12:48 PM
Unknown Object (File)
Feb 3 2024, 7:22 AM
Unknown Object (File)
Feb 3 2024, 5:53 AM
Unknown Object (File)
Jan 9 2024, 10:56 AM
Unknown Object (File)
Dec 1 2023, 7:35 AM
View All 31 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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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

Diff 82339

View Options

libexec/rc/rc.d/tmp

Loading...