Bug 266208 - multimedia/*pipe-viewer: Update to 0.3.1
Summary: multimedia/*pipe-viewer: Update to 0.3.1
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Nuno Teixeira
URL: https://github.com/trizen/pipe-viewer...
Keywords:
Depends on:
Blocks:
 
Reported: 2022-09-03 20:27 UTC by Jesús Daniel Colmenares Oviedo
Modified: 2022-09-11 23:31 UTC (History)
1 user (show)

See Also:


Attachments
patch (1.89 KB, patch)
2022-09-03 20:27 UTC, Jesús Daniel Colmenares Oviedo
no flags Details | Diff
pipe-viewer-0.3.1 (6.60 KB, patch)
2022-09-11 00:51 UTC, Jesús Daniel Colmenares Oviedo
DtxdF: maintainer-approval+
Details | Diff
Add GTK3_USES=desktop-file-utils (6.63 KB, patch)
2022-09-11 19:52 UTC, Jesús Daniel Colmenares Oviedo
DtxdF: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jesús Daniel Colmenares Oviedo 2022-09-03 20:27:05 UTC
Created attachment 236343 [details]
patch

pipe-viewer has a .desktop file, but in a different directory than the standard one. It is now a dynamic link to this file, including the icon.
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2022-09-03 20:27:05 UTC
Maintainer informed via mail
Comment 2 Jesús Daniel Colmenares Oviedo 2022-09-11 00:51:26 UTC
Created attachment 236489 [details]
pipe-viewer-0.3.1

Description:

* add .desktop file
* pet portclippy
* reformat with portfmt
* drop WWW::PipeViewer::Authentication: unused and unsupported
* drop WWW::PipeViewer::PlaylistItems::favorite_video: unused and unsupported

Changes: https://github.com/trizen/pipe-viewer/blob/0.3.1/Changes

QA:

* portlint: 1 fatal error and 2 warnings:
  - FATAL: Makefile: extra item "WWW" placed in the MAINTAINER section.
  - WARN: Makefile: unless this is a master port, MAINTAINER has to be set by "=", not by "?=".
  - WARN: Makefile: unless this is a master port, COMMENT has to be set by "=", not by "?=".
* poudriere: OK (13-1, amd64, GTK3 JSON_XS LOCAL_CACHE READLINE SHARE_DIR CHAR_WIDTH LINE_BREAK YOUTUBE_DL YT_DL tested)
Comment 3 Nuno Teixeira freebsd_committer freebsd_triage 2022-09-11 15:21:44 UTC
portlint -C:
---
FATAL: ###/ports/multimedia/pipe-viewer/pkg-plist: [39]: this port installs .desktop files. Please add `desktop-file-utils` to USES.
---
6.9.5.2. Updating Desktop Database

If a port has a MimeType entry in its portname.desktop, the desktop database must be updated after install and deinstall. To do this, define USES= desktop-file-utils.

Cheers
Comment 4 Jesús Daniel Colmenares Oviedo 2022-09-11 19:52:28 UTC
Created attachment 236504 [details]
Add GTK3_USES=desktop-file-utils

I have added desktop-file-utils to GTK3_USES to update the desktop database only when the GTK3 option is enabled.
Comment 5 Nuno Teixeira freebsd_committer freebsd_triage 2022-09-11 23:10:36 UTC
(In reply to Jesús Daniel Colmenares Oviedo from comment #4)

GTK3_USES=desktop-file-utils make sense since it only installs .desktop files when GTK3 is enabled:
---
%%GTK3%%%%SITE_PERL%%/auto/share/dist/WWW-PipeViewer/gtk-pipe-viewer.desktop
%%GTK3%%share/applications/gtk-pipe-viewer.desktop
---

I think I will open an issue at portlint about this.

Cheers
Comment 6 commit-hook freebsd_committer freebsd_triage 2022-09-11 23:30:18 UTC
A commit in branch main references this bug:

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

commit ae5b7072886eb9e743c22f814a85f46722b0d4f4
Author:     Jesús Daniel Colmenares Oviedo <DtxdF@disroot.org>
AuthorDate: 2022-09-11 23:24:54 +0000
Commit:     Nuno Teixeira <eduardo@FreeBSD.org>
CommitDate: 2022-09-11 23:24:54 +0000

    multimedia/*pipe-viewer: Update to 0.3.1

     - add .desktop file
     - pet portclippy
     - reformat with portfmt
     - drop WWW::PipeViewer::Authentication: unused and unsupported
     - drop WWW::PipeViewer::PlaylistItems::favorite_video: unused and unsupported

    ChangeLog:      https://github.com/trizen/pipe-viewer/blob/0.3.1/Changes
    PR:             266208

 multimedia/gtk-pipe-viewer/Makefile |  4 ++--
 multimedia/pipe-viewer/Makefile     | 34 ++++++++++++++++++++--------------
 multimedia/pipe-viewer/distinfo     |  6 +++---
 multimedia/pipe-viewer/pkg-plist    |  7 +++----
 4 files changed, 28 insertions(+), 23 deletions(-)
Comment 7 Nuno Teixeira freebsd_committer freebsd_triage 2022-09-11 23:31:42 UTC
Committed, thanks!