Bug 257090

Summary: [NEW PORT] misc/komorebi: Animated wallpaper manager
Product: Ports & Packages Reporter: Fadel <fjachuf>
Component: Individual Port(s)Assignee: freebsd-ports-bugs (Nobody) <ports-bugs>
Status: Open ---    
Severity: Affects Only Me CC: fjachuf, grahamperrin, khng
Priority: --- Keywords: feature, needs-patch, needs-qa
Version: LatestFlags: koobs: maintainer-feedback? (fjachuf)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch
none
Komorebi
none
Komorebi 1.1 none

Description Fadel 2021-07-09 23:24:52 UTC
Created attachment 226335 [details]
patch

Animated wallpaper for your desktop like xfce, gnome, mate, etc.
Comment 1 Ka Ho Ng freebsd_committer freebsd_triage 2021-07-09 23:38:41 UTC
My first glance without building it yet.

> +USE_GITHUB=	yes
> +GH_ACCOUNT=	SrWither
> +GH_PROJECT=	KomorebiBSD
> +GH_TAGNAME= 1.1
> +
>
> +MAINTAINER=	fjachuf@gmail.com
> +COMMENT=	Animated Wallpaper for your desktop
> +USES= cmake pkgconfig
Wrong section orders.

> +#PLIST_FILES=
Just drop it if it does not constitute any meaning.

You miss the dependencies of your port to be properly build and run as well.
Consider running portlint -abct to check the sanity of the files. Handbook link:
https://docs.freebsd.org/en/books/porters-handbook/quick-porting/#porting-portlint
Comment 2 Fadel 2021-07-11 23:31:28 UTC
Created attachment 226380 [details]
Komorebi
Comment 3 Fadel 2021-07-11 23:33:39 UTC
Comment on attachment 226380 [details]
Komorebi

Animated Wallpaper Manager
Comment 4 Kubilay Kocak freebsd_committer freebsd_triage 2021-07-12 02:22:09 UTC
@Reporter Please provide a single patch will all changes included (second attachment missing pkg-plist). Confirmation of QA [1] passing would also be great

[1] poudriere and portlint at least: https://docs.freebsd.org/en/books/porters-handbook/testing/
Comment 5 Fadel 2021-07-12 15:55:11 UTC
Created attachment 226401 [details]
Komorebi 1.1
Comment 6 Fadel 2021-07-12 16:01:01 UTC
(In reply to Kubilay Kocak from comment #4)
I put the command make stage-qa to see what I got, this was the result

Out: 
❯ doas make stage-qa
Password:
====> Running Q/A tests (stage-qa)

komorebi git/master
❯