[PR #5905] [MERGED] profiles: fix misc in kmail/transmission-qt & add kontact.profile #5718

New issue

Closed

opened 2026-05-05 10:43:38 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 10:43:38 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/netblue30/firejail/pull/5905
Author: @Sadoon-AlBader
Created: 7/17/2023
Status: ✅ Merged
Merged: 12/18/2024
Merged by: @netblue30

Base: master ← Head: master

📝 Commits (10+)

d578e92 remove nosound from kmail.profile
f49c3eb Enable notifications to kmail.profile
1e92326 Create kontact.profile
7b5d946 Add systray option as a commented config to transmission-qt
1f40d91 Fix kmail.profile ordering based on template
f6b3857 Fix kontact.profile ordering based on template
0e9c8ea Prompt the user to add lines to .local to enable notifications
4d00897 Enable systray support with condition ?ALLOW_TRAY
5057ab1 Add kontact to firecfg
2c80515 make kontact.profile a redirect of kmail.profile

📊 Changes

4 files changed (+30 additions, -2 deletions)

View changed files

📝 etc/profile-a-l/kmail.profile (+6 -2)
➕ etc/profile-a-l/kontact.profile (+20 -0)
📝 etc/profile-m-z/transmission-qt.profile (+3 -0)
📝 src/firecfg/firecfg.config (+1 -0)

📄 Description

This removes "nosound" from kmail.profile to allow notification sounds, and also adds a "dbus-user.talk org.freedesktop.Notifications" to allow notifications to start.
Also adds kontact.profile based on kmail.profile, have been using it and works well.
Also edits transmission-qt.profile to allow systray, commented out as is usually the case with profiles, but better to at least let the user know that it might be necessary to add to .local if they need systray.

EDIT: if the maintainers also prefer to keep "nosound" for kmail, it might make sense to also add it to kontact.profile since I didn't add it there to start, I think it makes sense to either remove it or add a comment letting the user know that sound notifications won't work unless they "ignore nosound"

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/netblue30/firejail/pull/5905 **Author:** [@Sadoon-AlBader](https://github.com/Sadoon-AlBader) **Created:** 7/17/2023 **Status:** ✅ Merged **Merged:** 12/18/2024 **Merged by:** [@netblue30](https://github.com/netblue30) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (10+) - [`d578e92`](https://github.com/netblue30/firejail/commit/d578e9235d4f0a4a139b1aebfa24e3ba1a890470) remove nosound from kmail.profile - [`f49c3eb`](https://github.com/netblue30/firejail/commit/f49c3ebfddfeca029bc4560fee4b343d09ebc1de) Enable notifications to kmail.profile - [`1e92326`](https://github.com/netblue30/firejail/commit/1e9232662c9d45da488a5746b67ee75b3b094216) Create kontact.profile - [`7b5d946`](https://github.com/netblue30/firejail/commit/7b5d9468baa7e435e049f86169823c962a4b7a28) Add systray option as a commented config to transmission-qt - [`1f40d91`](https://github.com/netblue30/firejail/commit/1f40d9135bb897bc2199e13a445d5e5d0caaa084) Fix kmail.profile ordering based on template - [`f6b3857`](https://github.com/netblue30/firejail/commit/f6b3857592462eec066924d59d836af41e5f29ce) Fix kontact.profile ordering based on template - [`0e9c8ea`](https://github.com/netblue30/firejail/commit/0e9c8ea743c6534cb53673fa5e6e41ead0abcf48) Prompt the user to add lines to .local to enable notifications - [`4d00897`](https://github.com/netblue30/firejail/commit/4d00897f607eca1b752ed9982eac3f5eb0a79d1b) Enable systray support with condition ?ALLOW_TRAY - [`5057ab1`](https://github.com/netblue30/firejail/commit/5057ab1ce97d1aa30e7172eb78c9c58464aa96ad) Add kontact to firecfg - [`2c80515`](https://github.com/netblue30/firejail/commit/2c80515aa5ab41dd00d0eed3dea869321be9f5a6) make kontact.profile a redirect of kmail.profile ### 📊 Changes **4 files changed** (+30 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `etc/profile-a-l/kmail.profile` (+6 -2) ➕ `etc/profile-a-l/kontact.profile` (+20 -0) 📝 `etc/profile-m-z/transmission-qt.profile` (+3 -0) 📝 `src/firecfg/firecfg.config` (+1 -0) </details> ### 📄 Description This removes "nosound" from kmail.profile to allow notification sounds, and also adds a "dbus-user.talk org.freedesktop.Notifications" to allow notifications to start. Also adds kontact.profile based on kmail.profile, have been using it and works well. Also edits transmission-qt.profile to allow systray, commented out as is usually the case with profiles, but better to at least let the user know that it might be necessary to add to .local if they need systray. EDIT: if the maintainers also prefer to keep "nosound" for kmail, it might make sense to also add it to kontact.profile since I didn't add it there to start, I think it makes sense to either remove it or add a comment letting the user know that sound notifications won't work unless they "ignore nosound" --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

gitea-mirror

2026-05-05 10:43:39 -06:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/firejail#5718

No description provided.

Rows
Columns