Bug 267263 - sysutils/libudisks: Update to 2.9.4
Summary: sysutils/libudisks: Update to 2.9.4
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/storaged-project/u...
Keywords: needs-qa
Depends on:
Blocks:
 
Reported: 2022-10-22 00:44 UTC by Pau Amma
Modified: 2022-10-22 14:08 UTC (History)
2 users (show)

See Also:
eduardo: merge-quarterly+


Attachments
Upgrade to 2.9.4 (2.44 KB, patch)
2022-10-22 00:44 UTC, Pau Amma
pauamma: maintainer-approval+
Details | Diff
sugestion v0 (2.54 KB, patch)
2022-10-22 13:27 UTC, Nuno Teixeira
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pau Amma 2022-10-22 00:44:43 UTC
Created attachment 237517 [details]
Upgrade to 2.9.4

Summary of changes: https://github.com/storaged-project/udisks/releases/tag/udisks-2.9.3 and https://github.com/storaged-project/udisks/releases/tag/udisks-2.9.4

Full changes since 2.9.2: https://github.com/storaged-project/udisks/blob/c430dd9a27e158693cc783e9ee91bf6e5b2a8819/NEWS and https://github.com/storaged-project/udisks/blob/001c486e6d099ed33e2de4f5c73c03e3ee180f81/NEWS

Reported by: portscout.

While there:
- consult portclippy
- switch MAINTAINER to my @f.o address
- use %%PORTVERSION%% in pkg-plist to avoid changing it on each upstream upgrade.
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2022-10-22 00:44:43 UTC
Maintainer informed via mail
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2022-10-22 00:54:10 UTC
^Triage: If this is a bugfix or security release, or quarterly users are affected, please set merge-quarterly flag to: ?
Comment 3 commit-hook freebsd_committer freebsd_triage 2022-10-22 02:38:20 UTC
A commit in branch main references this bug:

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

commit 4b8953300ff9a327ad32a6d80fc3b8abcb6f603b
Author:     Pau Amma <pauamma@FreeBSD.org>
AuthorDate: 2022-10-22 02:34:32 +0000
Commit:     Nuno Teixeira <eduardo@FreeBSD.org>
CommitDate: 2022-10-22 02:34:32 +0000

    sysutils/libudisks: Update to 2.9.4

     - consult portclippy
     - switch MAINTAINER to my @f.o address
     - use %%PORTVERSION%% in pkg-plist to avoid changing it on each
       upstream upgrade

    ChangeLog:
    https://github.com/storaged-project/udisks/blob/udisks-2.9.4/NEWS

    PR:             267263
    MFH:            2022Q4 (bugfixes)

 sysutils/libudisks/Makefile  | 10 ++++++----
 sysutils/libudisks/distinfo  |  6 +++---
 sysutils/libudisks/pkg-plist |  2 +-
 3 files changed, 10 insertions(+), 8 deletions(-)
Comment 4 commit-hook freebsd_committer freebsd_triage 2022-10-22 03:17:26 UTC
A commit in branch 2022Q4 references this bug:

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

commit 387b0285cc2767ac693366f6dc4f99811fc1675c
Author:     Pau Amma <pauamma@FreeBSD.org>
AuthorDate: 2022-10-22 02:34:32 +0000
Commit:     Nuno Teixeira <eduardo@FreeBSD.org>
CommitDate: 2022-10-22 02:38:35 +0000

    sysutils/libudisks: Update to 2.9.4

     - consult portclippy
     - switch MAINTAINER to my @f.o address
     - use %%PORTVERSION%% in pkg-plist to avoid changing it on each
       upstream upgrade

    ChangeLog:
    https://github.com/storaged-project/udisks/blob/udisks-2.9.4/NEWS

    PR:             267263
    MFH:            2022Q4 (bugfixes)
    (cherry picked from commit 4b8953300ff9a327ad32a6d80fc3b8abcb6f603b)

 sysutils/libudisks/Makefile  | 10 ++++++----
 sysutils/libudisks/distinfo  |  6 +++---
 sysutils/libudisks/pkg-plist |  2 +-
 3 files changed, 10 insertions(+), 8 deletions(-)
Comment 5 Nuno Teixeira freebsd_committer freebsd_triage 2022-10-22 03:18:03 UTC
Committed, thanks!
Comment 6 Daniel Engberg freebsd_committer freebsd_triage 2022-10-22 08:54:18 UTC
Any reason why we're not using the official release archive?
https://github.com/storaged-project/udisks/releases/tag/udisks-2.9.4

Nitpick but Porters Handbook also suggests to use DISTVERSION
Comment 7 Pau Amma 2022-10-22 10:12:24 UTC
(In reply to Daniel Engberg from comment #6)

If you mean to get the source tarball: I use whatever 
> USE_GITHUB=	yes
> GH_ACCOUNT=	storaged-project
> GH_PROJECT=	udisks
gives me. I figure if there's ports framework support for it, then it must be an acceptable way to download source code, if not the best one in all cases.

Re your nitpick: DISTVERSION instead of... PORTVERSION and DISTVERSIONPREFIX? I seem to remember the way I use is what someone suggested when I first created the port. Comment just above explains their reason for it. (Keep in mind: first port, and so far only one.)
Comment 8 Nuno Teixeira freebsd_committer freebsd_triage 2022-10-22 13:27:08 UTC
Created attachment 237523 [details]
sugestion v0

- switch to DISTVERSION
- switch to MASTER_SITES (static tarball)

Fails to configure:
---
===>  Configuring for libudisks-2.9.4
cd /wrkdirs/usr/ports/sysutils/libudisks/work/udisks-2.9.4 && /usr/bin/env NOCONFIGURE=1 ./autogen.sh
env: ./autogen.sh: No such file or directory
---

Any clues? Can't figure out where NOCONFIGURE=1 cames from.

Cheers
Comment 9 Daniel Engberg freebsd_committer freebsd_triage 2022-10-22 14:02:42 UTC
Nuno, I forgot to link. :(

I've submitted a few improvements here:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=267268
Comment 10 Nuno Teixeira freebsd_committer freebsd_triage 2022-10-22 14:08:32 UTC
(In reply to Daniel Engberg from comment #9)

Nice! Looks good.

Thanks