8000 fix: `after-install.tpl`: Detect if apparmor is enabled instead of just file-exists check by gaaf · Pull Request #8932 · electron-userland/electron-builder · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: after-install.tpl: Detect if apparmor is enabled instead of just file-exists check #8932

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

Merged
merged 2 commits into from
Mar 4, 2025

Conversation

gaaf
Copy link
Contributor
@gaaf gaaf commented Mar 4, 2025

Merely the presence of a config directory does not imply a functional apparmor installation. Instead, just ask apparmor itself if it is enabled.

This fixes #8931

Merely the presence of a config directory does not imply a functional apparmor installation. Instead, just ask apparmor itself if it is enabled.
Copy link
changeset-bot bot commented Mar 4, 2025

🦋 Changeset detected

Latest commit: 3735f84

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
app-builder-lib Patch
dmg-builder Patch
electron-builder-squirrel-windows Patch
electron-builder Patch
electron-forge-maker-appimage Patch
electron-forge-maker-nsis-web Patch
electron-forge-maker-nsis Patch
electron-forge-maker-snap Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mmaietta mmaietta changed the title after-install.tpl: Fix detecting apparmor fix: after-install.tpl: Detect if apparmor is enabled instead of just file-exists check Mar 4, 2025
@mmaietta mmaietta merged commit e1ea62b into electron-userland:master Mar 4, 2025
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Debian package install fails with apparmor error
2 participants
0