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

Run mutt under firejail #25

Merged
merged 1 commit into from
Aug 15, 2016
Merged

Run mutt under firejail #25

merged 1 commit into from
Aug 15, 2016

Conversation

KellerFuchs
Copy link
Member

This is part of hashbang/shell-etc#86.

@KellerFuchs
Copy link
Member Author

KellerFuchs commented Aug 3, 2016

Don't merge, profile should go in shell-etc.
See previous discussion.

KellerFuchs added a commit to KellerFuchs/shell-etc that referenced this pull request Aug 3, 2016
@KellerFuchs KellerFuchs changed the title Provide a firejail profile for mutt Run mutt under firejail Aug 3, 2016
@KellerFuchs
Copy link
Member Author

Merge after hashbang/shell-etc#102 is deployed

@lrvick
Copy link
Member

lrvick commented Aug 15, 2016

Seems straight forward enough and easy to revert if it ruins lives. +1

@KellerFuchs KellerFuchs merged commit 2ef6edd into hashbang:master Aug 15, 2016
@KellerFuchs KellerFuchs deleted the mutt branch August 15, 2016 04:38
@gmcclins
Copy link
Contributor

I think there is a permission problem.

Error: cannot open profile file /etc/firejail/mutt.profile.

4 -rw-r----- 1 root root 530 Aug 15 04:40 mutt.profile

KellerFuchs added a commit to hashbang/shell-etc that referenced this pull request Aug 15, 2016
@KellerFuchs
Copy link
Member Author

@gmcclins Thanks, there indeed was an issue. It is now fixed :-)

lrvick pushed a commit to hashbang/shell-etc that referenced this pull request Jun 15, 2017
lrvick pushed a commit to hashbang/shell-etc that referenced this pull request Jun 15, 2017
KellerFuchs added a commit to hashbang/shell-etc that referenced this pull request Sep 20, 2017
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.

3 participants