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

Add remarvin mountpoints to xochitl application file #275

Merged
merged 1 commit into from
Jan 22, 2023

Conversation

plan5
Copy link
Contributor

@plan5 plan5 commented Dec 19, 2022

remarvin uses bind mounts and gocryptfs to add multiple profiles and encryption.

These mount points aren't included in the current chroot for xochitl, keeping it from seeing the decrypted files.

remarvin is not in toltec yet but compatibility would be nice to have and I experienced no issues with these changes on a non-encrypted setup.

Should fix plan5/remarvin#6

[remarvin](https://github.com/plan5/remarvin) uses bind mounts and gocryptfs to add multiple profiles and encryption.

These mount points aren't included in the current chroot for xochitl, keeping it from seeing the decrypted files.

remarvin is not in toltec yet but compatibility would be nice to have and I experienced no issues with these changes on a non-encrypted setup.
@plan5 plan5 changed the title Add remarvin mountpoints to application file Add remarvin mountpoints to xochitl application file Dec 19, 2022
@Eeems
Copy link
Collaborator

Eeems commented Dec 19, 2022

Please make sure to update the toltec copy of xochitl.oxide when remarvin is added to toltec. I would recommend adding instructions to your repo for how to manually add it to the chroot for now.

@plan5
Copy link
Contributor Author

plan5 commented Dec 19, 2022

Please make sure to update the toltec copy of xochitl.oxide when remarvin is added to toltec. I would recommend adding instructions to your repo for how to manually add it to the chroot for now.

Ok, thanks. Then I'll close the PR for now.

@plan5 plan5 closed this Dec 19, 2022
@Eeems
Copy link
Collaborator

Eeems commented Dec 19, 2022

Oh, I wasn't asking you to close the PR, I was going to merge it as soon as the build finished.

@plan5
Copy link
Contributor Author

plan5 commented Dec 19, 2022

Oops, okay, reopening then :)

@plan5 plan5 reopened this Dec 19, 2022
@Eeems Eeems merged commit f247753 into Eeems-Org:master Jan 22, 2023
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.

Files not accessible with oxide launcher
2 participants