Bug 282784 - filesystems/zrepl-dsh2dsh: update to 0.9.1
Summary: filesystems/zrepl-dsh2dsh: update to 0.9.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: Robert Clausecker
URL: https://github.com/dsh2dsh/zrepl/rele...
Keywords:
Depends on:
Blocks:
 
Reported: 2024-11-15 15:08 UTC by Denis Shaposhnikov
Modified: 2024-11-18 01:02 UTC (History)
2 users (show)

See Also:


Attachments
zrepl-dsh2dsh-0.9.1.patch (4.77 KB, patch)
2024-11-15 15:08 UTC, Denis Shaposhnikov
dsh: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Denis Shaposhnikov 2024-11-15 15:08:23 UTC
Created attachment 255196 [details]
zrepl-dsh2dsh-0.9.1.patch

Upgrading needs to modify configuration file:

  /usr/local/etc/zrepl/zrepl.yml

Please read more about this change here:
https://github.com/dsh2dsh/zrepl?tab=readme-ov-file#breaking-changes

https://github.com/dsh2dsh/zrepl/releases/tag/v0.8.10
https://github.com/dsh2dsh/zrepl/releases/tag/v0.9.0
https://github.com/dsh2dsh/zrepl/releases/tag/v0.9.1
Comment 1 Robert Clausecker freebsd_committer freebsd_triage 2024-11-15 17:05:01 UTC
What is COPY supposed to do?
The macro does not seem to be defined; I'll use INSTALL_DATA instead.
I'll go ahead and add an entry to UPDATING when committing this patch.
Comment 2 Denis Shaposhnikov 2024-11-15 17:55:34 UTC
(In reply to Robert Clausecker from comment #1)
> What is COPY supposed to do?
> The macro does not seem to be defined;

I got it from the definition of INSTALL_DATA in bsd.port.mk:

INSTALL_DATA=	${INSTALL} ${COPY} -m ${_SHAREMODE}

That file should be installed with 0600 mode, instead of 0644, which INSTALL_DATA uses. That's why I don't use INSTALL_DATA for that case.
Comment 3 Robert Clausecker freebsd_committer freebsd_triage 2024-11-15 18:32:50 UTC
I understand.  I'll address this by adding

@sample(,,0600) %%ETCDIR%%/keys.yaml.sample

to pkg-plist so the file is installed with the correct mode.
Comment 4 Denis Shaposhnikov 2024-11-15 18:34:16 UTC
(In reply to Robert Clausecker from comment #3)
> @sample(,,0600) %%ETCDIR%%/keys.yaml.sample

Great! That's even better. Thanks.
Comment 5 commit-hook freebsd_committer freebsd_triage 2024-11-18 00:59:18 UTC
A commit in branch main references this bug:

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

commit 03e8423f08859e54d5ac1263b1ba8e0e2faff6d0
Author:     Denis Shaposhnikov <993498+dsh2dsh@users.noreply.github.com>
AuthorDate: 2024-11-15 15:03:20 +0000
Commit:     Robert Clausecker <fuz@FreeBSD.org>
CommitDate: 2024-11-18 00:57:54 +0000

    filesystems/zrepl-dsh2dsh: update to 0.9.1

    Upgrading needs to modify configuration file:

      /usr/local/etc/zrepl/zrepl.yml

    Please read more about this change here:
    https://github.com/dsh2dsh/zrepl?tab=readme-ov-file#breaking-changes

    Changelogs:
    https://github.com/dsh2dsh/zrepl/releases/tag/v0.8.10
    https://github.com/dsh2dsh/zrepl/releases/tag/v0.9.0
    https://github.com/dsh2dsh/zrepl/releases/tag/v0.9.1

    PR:             282784

 UPDATING                                       | 11 +++++++++++
 filesystems/zrepl-dsh2dsh/Makefile             |  6 ++++--
 filesystems/zrepl-dsh2dsh/distinfo             | 10 +++++-----
 filesystems/zrepl-dsh2dsh/files/pkg-message.in | 14 ++++++++++++--
 filesystems/zrepl-dsh2dsh/pkg-plist            |  4 ++--
 5 files changed, 34 insertions(+), 11 deletions(-)
Comment 6 Robert Clausecker freebsd_committer freebsd_triage 2024-11-18 01:02:17 UTC
Thank you for your contribution.