-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Mount a custom home partition #353
base: main
Are you sure you want to change the base?
Conversation
I really have no idea if the decryption thing works or not because I don't have any encrypted on my system. If someone does, please do let me know if it works. |
Update: works on my machine (also I finally don't have to mount my home partition by hand anymore 😁) You can test it by using the |
Added a reviewer by mistake. Anyway the changes looks good for now! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm really sorry, I see that work went into this, I just think it's way too overkill for what we're trying to do here.
In my opinion, we should just make /home a symlink to /var/home.
It's how Silverblue does it and would allow mounting something in /home (would mount to /var/home)
Also, /home doesn't need to exist before systemd so I strongly disagree with putting all the logic here.
That's also fair |
Maybe it's just me, but the fact that |
@muhdsalm Could you elaborate? I don't understand what you mean. If you were to switch from a different OS with a separate home partition to an ABRoot based OS you could just put the mount to /home in your fstab and systemd would automatically mount it in /var/home |
Some application configs break when switching from a system with $HOME set as |
With a symlink, we would still use /home for $HOME |
Oh ok, cool |
Would the decryption be a reason to put the logic here? |
@muhdsalm The only reason why we need /var this early is because it contains part of /etc |
85a4cab
to
47f75de
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to me like there's no checking if the symlink exists already.
If I'm wrong and you tested this across an update and multiple reboots then you can resolve this.
f1225b7
to
ad54b4f
Compare
The issue with this push is that the /home cannot exist otherwise it will not create the symlink (which is expected behavior), so while testing you have to remove /home manually. |
Perhaps encryption should be handled in a different PR |
This is kinda part one in the process to support custom home partitions. This a sort of roadmap of the process:
Up till this point, to add a custom home partition you have to mess around with
abroot.json
and filesystem labels yourself.