Bug 274599 - x11-themes/plasma5-oxygen: add the sounds that have been swapped out since 5.25 back to the Oxygen style
Summary: x11-themes/plasma5-oxygen: add the sounds that have been swapped out since 5....
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-kde (group)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-10-19 16:16 UTC by Lars Herschke
Modified: 2023-10-25 23:30 UTC (History)
3 users (show)

See Also:
bugzilla: maintainer-feedback? (kde)

Attachments
add the sounds back to the Oxygen style (4.43 KB, patch)
2023-10-19 16:16 UTC, Lars Herschke 		no flags Details | Diff
plasma5-oxygen-sounds (4.90 KB, patch)
2023-10-25 13:11 UTC, Lars Herschke 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Herschke 2023-10-19 16:16:35 UTC 
Created attachment 245746 [details]
add the sounds back to  the Oxygen style

With the KDE update 5.25 the sounds from the oxygen style were moved to an extra package. However, x11/kf5-frameworkintegration refers to some of these sounds in plasma_workspace.notifyrc. This leads to error messages, e.g. 
"kf.notifications: Audio notification requested, but sound file from notifyrc file was not found, aborting audio notification". 
With the patch the sounds are again as before with in the plasma5-oxygen package.
Comment 1 Max Brazhnikov freebsd_committer freebsd_triage 2023-10-23 19:31:07 UTC 
Cool! It would be nice to have new port port x11-themes/plasma5-oxygen-sounds, which installs a sound theme. Lars, could you rework you patch to accomplish this?
Comment 2 Lars Herschke 2023-10-25 13:11:14 UTC 
Created attachment 245862 [details]
plasma5-oxygen-sounds

Now as a single port.
Comment 3 commit-hook freebsd_committer freebsd_triage 2023-10-25 23:28:14 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=e6b83d42a0f2ea6fe8cafee098302324a8435fd9

commit e6b83d42a0f2ea6fe8cafee098302324a8435fd9
Author:     Dima Panov <fluffy@FreeBSD.org>
AuthorDate: 2023-10-25 23:10:54 +0000
Commit:     Dima Panov <fluffy@FreeBSD.org>
CommitDate: 2023-10-25 23:10:54 +0000

    audio/plasma5-oxygen-sounds: add default Plasma sound theme (+)

    PR:     274599

 audio/Makefile                              |  1 +
 audio/plasma5-oxygen-sounds/Makefile (new)  | 17 ++++++++++
 audio/plasma5-oxygen-sounds/distinfo (new)  |  3 ++
 audio/plasma5-oxygen-sounds/pkg-descr (new) |  1 +
 audio/plasma5-oxygen-sounds/pkg-plist (new) | 50 +++++++++++++++++++++++++++++
 5 files changed, 72 insertions(+)
Comment 4 commit-hook freebsd_committer freebsd_triage 2023-10-25 23:28:16 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=91f6c6375c89da96b226a63330d418e5b206930d

commit 91f6c6375c89da96b226a63330d418e5b206930d
Author:     Dima Panov <fluffy@FreeBSD.org>
AuthorDate: 2023-10-25 23:27:07 +0000
Commit:     Dima Panov <fluffy@FreeBSD.org>
CommitDate: 2023-10-25 23:27:07 +0000

    x11/kf5-frameworkintegration: depend on audio/plasma5-oxygen-sounds (+)

    PR:     274599

 x11/kf5-frameworkintegration/Makefile | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Comment 5 Dima Panov freebsd_committer freebsd_triage 2023-10-25 23:30:15 UTC 
Have placed sounds theme under audio/ cathegory as it could be.


Thanks for submission!