Bug 269160 - x11/workrave: add flavor for XFCE support
Summary: x11/workrave: add flavor for XFCE support
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: Fernando Apesteguía
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-01-25 22:54 UTC by Pau Amma
Modified: 2023-01-29 18:26 UTC (History)
5 users (show)

See Also:
cyberbotx: maintainer-feedback+


Attachments
Add xfce flavor (2.39 KB, patch)
2023-01-25 22:54 UTC, Pau Amma
cyberbotx: maintainer-approval+
Details | Diff
Add xfce flavor, this time with correct pkg-plist. (2.14 KB, patch)
2023-01-28 22:19 UTC, Pau Amma
cyberbotx: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pau Amma 2023-01-25 22:54:08 UTC
Created attachment 239713 [details]
Add xfce flavor
Comment 1 Naram Qashat 2023-01-25 23:02:59 UTC
Comment on attachment 239713 [details]
Add xfce flavor

Looks good to me.
Comment 2 Fernando Apesteguía freebsd_committer freebsd_triage 2023-01-26 18:28:52 UTC
^Triage: reporter is committer, assign accordingly.
Comment 3 Gleb Popov freebsd_committer freebsd_triage 2023-01-27 05:51:35 UTC
LGTM too, please push.
Comment 4 Pau Amma 2023-01-27 19:58:31 UTC
(In reply to Gleb Popov from comment #3)

Not worth the anxiety spike to me. Unassigning this. I've done what I could. If it doesn't happen, so be it.
Comment 5 Gleb Popov freebsd_committer freebsd_triage 2023-01-28 09:01:41 UTC
====> Running Q/A tests (stage-qa)
Warning: you may not need USES=desktop-file-utils
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: lib/xfce4/panel/plugins/libworkrave-plugin.a
Error: Orphaned: lib/xfce4/panel/plugins/libworkrave-plugin.so
Error: Orphaned: share/xfce4/panel/plugins/workrave-xfce-applet.desktop
===> Checking for items in pkg-plist which are not in STAGEDIR
===> Error: Plist issues found.
Comment 6 Pau Amma 2023-01-28 10:36:41 UTC
(In reply to Gleb Popov from comment #5)

USES=desktop-file-utils is there because portlint said to (and the PHB confirmed).

I'll look at the rest. (I may have forgotten the DEVELOPER= magic incantation in that VM.)

Thanks.
Comment 7 Pau Amma 2023-01-28 22:19:33 UTC
Created attachment 239772 [details]
Add xfce flavor, this time with correct pkg-plist.

====> Compressing man pages (compress-man)
====> Running Q/A tests (stage-qa)
Warning: you may not need USES=desktop-file-utils
===>  Installing for workrave-xfce-1.10.50
===>  Checking if workrave-xfce is already installed
===>  Switching to root credentials for 'install' target
Password:
===>   Registering installation for workrave-xfce-1.10.50
Installing workrave-xfce-1.10.50...
Generating GTK icon cache for /usr/local/share/icons/hicolor
Building cache database of MIME types
===>  Returning to user credentials
pauamma@FreeBSD-workrave-xfce-flavor:~/ports/x11/workrave % make FLAVOR=xfce check-plist
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
===> Checking for items in pkg-plist which are not in STAGEDIR
===> No pkg-plist issues found (check-plist)
Comment 8 Naram Qashat 2023-01-28 22:25:32 UTC
Comment on attachment 239772 [details]
Add xfce flavor, this time with correct pkg-plist.

Looks good to me.
Comment 9 Fernando Apesteguía freebsd_committer freebsd_triage 2023-01-29 18:26:24 UTC
Committed,

Thanks!
Comment 10 commit-hook freebsd_committer freebsd_triage 2023-01-29 18:26:35 UTC
A commit in branch main references this bug:

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

commit ab65e95cc8fd0ba5d1ebbe1b5b7e7fd905ec683e
Author:     Pau Amma <pauamma@FreeBSD.org>
AuthorDate: 2023-01-29 17:36:41 +0000
Commit:     Fernando Apesteguía <fernape@FreeBSD.org>
CommitDate: 2023-01-29 18:21:42 +0000

    x11/workrave: add flavor for XFCE support

    Deliberately not bumping PORTREVISION.

    PR:             269160
    Reported by:    pauamma@FreeBSD.org
    Reviewed by:    cyberbotx@cyberbotx.com (maintainer)

 x11/workrave/Makefile             | 28 +++++++++++++++++++++++-----
 x11/workrave/pkg-descr.xfce (new) |  6 ++++++
 2 files changed, 29 insertions(+), 5 deletions(-)