Bug 267821 - x11/workrave: make stage missing-library-dependency errors on FreeBSD 13.1 with DEVELOPER=yes
Summary: x11/workrave: make stage missing-library-dependency errors on FreeBSD 13.1 wi...
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: 2022-11-17 04:39 UTC by Pau Amma
Modified: 2022-11-23 17:34 UTC (History)
2 users (show)

See Also:
cyberbotx: maintainer-feedback+


Attachments
Add missing library dependencies (659 bytes, patch)
2022-11-17 12:12 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 2022-11-17 04:39:57 UTC
====> Running Q/A tests (stage-qa)
Error: /usr/local/bin/workrave is linked to /usr/local/lib/libharfbuzz.so.0 from print/harfbuzz but it is not declared as a dependency
Warning: you need LIB_DEPENDS+=libharfbuzz.so:print/harfbuzz
Error: /usr/local/bin/workrave is linked to /usr/local/lib/libgdk_pixbuf-2.0.so.0 from graphics/gdk-pixbuf2 but it is not declared as a dependency
Warning: you need USE_GNOME+=gdkpixbuf2

(Working on https://lists.freebsd.org/archives/freebsd-ports/2022-January/001398.html, hence DEVELOPER=yes in /etc/make.conf.)
Comment 1 Naram Qashat 2022-11-17 05:02:11 UTC
I haven't had a chance to work on updating my ports in a whole, but these are probably easy enough for anyone to add to the port. If anyone does, I'll go ahead and accept the changes.
Comment 2 Pau Amma 2022-11-17 12:12:10 UTC
Created attachment 238136 [details]
Add missing library dependencies
Comment 3 Pau Amma 2022-11-17 12:19:16 UTC
Patch has no "make stage" errors, and I minimally tested it by running workrave, seeing it displays a window, and quitting. More testing would probably be better.
Comment 4 Naram Qashat 2022-11-17 16:01:49 UTC
Comment on attachment 238136 [details]
Add missing library dependencies

Patch looks good to me.
Comment 5 Pau Amma 2022-11-17 19:15:05 UTC
(In reply to Naram Qashat from comment #4)
Thanks. Can you set maintainer-approval+ on the patch to indicate that?
Comment 6 Naram Qashat 2022-11-17 21:54:59 UTC
Comment on attachment 238136 [details]
Add missing library dependencies

I did when I made the comment, apparently it didn't take because you didn't put my email on it? Not sure what happened there.
Comment 7 Pau Amma 2022-11-17 22:09:02 UTC
Comment on attachment 238136 [details]
Add missing library dependencies

Added now. Thanks for the reminder.
Comment 8 Naram Qashat 2022-11-17 22:12:13 UTC
Comment on attachment 238136 [details]
Add missing library dependencies

OK, this should do it, then.
Comment 9 Fernando Apesteguía freebsd_committer freebsd_triage 2022-11-21 07:01:21 UTC
^Triage: reporter is committer, assign accordingly.
Comment 10 commit-hook freebsd_committer freebsd_triage 2022-11-23 06:35:38 UTC
A commit in branch main references this bug:

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

commit 764c7201f2a99d2f57c7efcdcc26343cc3250f2b
Author:     Naram Qashat <cyberbotx@cyberbotx.com>
AuthorDate: 2022-11-22 16:13:20 +0000
Commit:     Fernando Apesteguía <fernape@FreeBSD.org>
CommitDate: 2022-11-23 06:31:10 +0000

    x11/workrave: Add missing library

    PR:             267821
    Reported by:    pauamma@FreeBSD.org

 x11/workrave/Makefile | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
Comment 11 Fernando Apesteguía freebsd_committer freebsd_triage 2022-11-23 06:36:13 UTC
Committed,

Thanks!
Comment 12 Pau Amma 2022-11-23 17:34:31 UTC
(In reply to Fernando Apesteguía from comment #11)
Noting FTR the author was me, not Naram Qashat.