Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#1818 tmpfs on tmp #1958

Closed
wants to merge 7 commits into from
Closed

Conversation

abalfoort
Copy link
Contributor

This fixes #1818

Configure mounting /tmp as tmpfs in fstab can be configured in fstab.conf.
Old behavior (mounting /tmp as tmpfs) is still default if not configured differently in fstab.conf.

@adriaandegroot
Copy link
Contributor

Please squash these PR's into something that follows the regular coding style and commit style. You can use git rebase -i to go through the commits to tidy them up.

@adriaandegroot
Copy link
Contributor

The code is good, and I appreciate your attention to getting defaults and not crashing on missing configuration keys (something other Python modules do a lot). Also updating the schema, thanks.

@abalfoort
Copy link
Contributor Author

Could you eleborate on: "squash these PR's"?
I don't know how to tidy up the PRs (using git rebase -i).

Indent

Overindent
@adriaandegroot
Copy link
Contributor

To tidy this up, I started by following the manual instructions for merging this branch. That leads to a situation that looks like this (in qgit, my preferred git-history-viewer; it looks slightly different in gitk).

image

The red nodes are your branch, which are mostly not needed: the "Configure.." and "Remove print.." commits are the useful ones. So I run git rebase -i calamares to re-do the branch on top of the calamares branch. This puts me in an editor where I can interactively edit the history:

image

I can replace the word pick there by a letter -- documentation is shown in the edit window -- to change the history. I delete the first two lines, since those are merges / noise. Then "Fix unencrypted.." gets an r because I want to edit the commit message slightly, and the last one gets an s to squash it with the previous commit (and I'll edit the commit message).

image

When squashing, you get all the commit messages together, and then you can edit all you like:

image

@abalfoort
Copy link
Contributor Author

Just a recap to see if I get it now:

  1. Rebase with calamares: git rebase -i calamares.
  2. Look for first commit you want to keep and label it "r".
  3. Remove commits above that one.
  4. The commit below the "r" labeled one does not change and keeps label "pick".
  5. Anything below that is squased by labelling them with "s".

Correct?

@kkofler
Copy link
Contributor

kkofler commented May 24, 2022

For what it's worth, on Fedora, /tmp is on tmpfs by default without any /etc/fstab entry, they decided to set this up with a native systemd unit instead.

@abalfoort abalfoort deleted the #1818-tmpfs-on-tmp branch May 25, 2022 06:33
lubuntu-lugito pushed a commit to lubuntu-team/calamares-packaging that referenced this pull request May 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lubuntu + Calamares mount tmpfs on /tmp
3 participants