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

fix firefox installation #704

Merged
merged 2 commits into from
Dec 1, 2024

Conversation

Jeffrey04
Copy link
Contributor

Firefox dev installation is broken due to the change at mozilla's website

@Jeffrey04
Copy link
Contributor Author

Jeffrey04 commented Dec 1, 2024

Just out of curiosity, in recent firefox installs it says it requires this in /etc/apparmor.d/, how do I do that with umake?

# This profile allows everything and only exists to give the
# application a name instead of having the label "unconfined"
abi <abi/4.0>,

include <tunables/global>

profile umake-firefox
/home/jeffrey04/.local/share/umake/web/firefox-dev/{firefox,firefox-bin,updater}
flags=(attach_disconnected,unconfined) {

    userns, 

    # Site-specific additions and overrides. See local/README for details. 

    include if exists <local/firefox> 

}

@LyzardKing
Copy link
Collaborator

@Jeffrey04 Where does it mention this is needed for a new install?

@LyzardKing LyzardKing merged commit 4d34901 into ubuntu:master Dec 1, 2024
3 checks passed
@Jeffrey04
Copy link
Contributor Author

Jeffrey04 commented Dec 1, 2024

@Jeffrey04 Where does it mention this is needed for a new install?

https://support.mozilla.org/en-US/kb/install-firefox-linux?as=u&utm_source=inproduct#w_security-features-warning

same thing is needed for vscode (and possibly other electron apps) too actually (posted an example in my codeium windsurf PR), OTOH, ubuntu published this

https://ubuntu.com/blog/ubuntu-23-10-restricted-unprivileged-user-namespaces

also sometimes a sysctl reload apparmor.service is not enough, gotta reboot

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.

2 participants