Bug 281567 - net-p2p/rslsync: Update 2.7.3 → 2.8.1.1390
Summary: net-p2p/rslsync: Update 2.7.3 → 2.8.1.1390
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: Vladimir Druzenko
URL: https://help.resilio.com/hc/en-us/art...
Keywords:
Depends on:
Blocks:
 
Reported: 2024-09-18 09:54 UTC by Volodymyr Kostyrko
Modified: 2024-11-09 17:51 UTC (History)
2 users (show)

See Also:
arcade: maintainer-feedback+


Attachments
update to 2.8.1 (1.60 KB, patch)
2024-09-18 09:54 UTC, Volodymyr Kostyrko
no flags Details | Diff
fixed patch to 3.0.0.1409 (3.03 KB, patch)
2024-11-04 14:03 UTC, Volodymyr Kostyrko
arcade: maintainer-approval+
Details | Diff
update to 2.8.1 (3.33 KB, patch)
2024-11-07 17:33 UTC, Volodymyr Kostyrko
arcade: maintainer-approval+
Details | Diff
update to 2.8.1 (3.31 KB, patch)
2024-11-09 17:20 UTC, Volodymyr Kostyrko
arcade: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Volodymyr Kostyrko 2024-09-18 09:54:56 UTC
Created attachment 253642 [details]
update to 2.8.1

Changes: compat11x not needed anymore, COMPAT11 still required.
Generally this is a reroll of an older build, no major changes, just some updates. Newer versions are available but dist has the same binary as this build.
Comment 1 Vladimir Druzenko freebsd_committer freebsd_triage 2024-09-20 23:53:18 UTC
1. Use DISTVERSION in MASTER_SITES.

2. Lines in do-build are too long, better to split:
do-build:
        ${WRKSRC}/rslsync --nodaemon --storage ${WRKSRC} \
                --dump-sample-config > ${WRKSRC}/rslsync.conf.sample
        ${REINPLACE_CMD} -e \
                's;^//\([[:space:]]*"storage_path"[[:space:]]*:[[:space:]]*\)"/.*",$$;\1"/var/db/rslsync",;' \
                ${WRKSRC}/rslsync.conf.sample
        ${REINPLACE_CMD} -e \
                's;^//\([[:space:]]*"pid_file"[[:space:]]*:[[:space:]]*\)"/.*",$$;\1"/var/run/rslsync/rslsync.pid",;' \
                ${WRKSRC}/rslsync.conf.sample

2. make maintainer-makesum doesn't work.
This version work correctly:
maintainer-makesum:
        for arch in ${ONLY_FOR_ARCHS}; do \
                ${MAKE} DISTINFO_FILE=/dev/stdout ARCH=$$arch makesum; \
        done | ${GREP} -v '^===> ' | ${AWK} '$$1 != "TIMESTAMP" || NR == 1' \
                > ${DISTINFO_FILE}
Comment 2 Daniel Engberg freebsd_committer freebsd_triage 2024-09-21 04:44:29 UTC
The do-build section should probably be post-install?
https://cgit.freebsd.org/ports/tree/net/gerbera/Makefile#n85
Comment 3 Volodymyr Kostyrko 2024-09-21 05:56:15 UTC
I guess idea behind do-build was to provide sample config in the package without need to generate it during install. Not sure.
Comment 4 Vladimir Druzenko freebsd_committer freebsd_triage 2024-10-02 18:28:25 UTC
ping
Comment 5 Vladimir Druzenko freebsd_committer freebsd_triage 2024-11-03 18:35:49 UTC
ping
Comment 6 Volodymyr Kostyrko 2024-11-04 14:03:29 UTC
Created attachment 254930 [details]
fixed patch to 3.0.0.1409

Ahm, that was for me. Sorry, fixed Makefile a little.

Anyway I think this will be the last version. They dropped FreeBSD support. Will create rslsync-linux when I have time.
Comment 7 Vladimir Druzenko freebsd_committer freebsd_triage 2024-11-04 17:24:05 UTC
(In reply to Volodymyr Kostyrko from comment #6)
My suggestion is:
1. Add "NO_BUILD=yes" (before SUB_FILES).
2. Move all lines from do-build to do-install.
Comment 8 Vladimir Druzenko freebsd_committer freebsd_triage 2024-11-04 17:36:36 UTC
3. Deprecated support for Windows_x86, Linux_i386, FreeBSD, and Docker
https://help.resilio.com/hc/en-us/articles/31386579044755-Resilio-Sync-3-0-change-log
Comment 9 Volodymyr Kostyrko 2024-11-07 17:33:20 UTC
Created attachment 255009 [details]
update to 2.8.1

Ah, I see your point. Moved everything to install.

Also checked versions, newest blob for 3.0.0 actually contains binaries for 2.8.1, so there's no point in fooling everybody...
Comment 10 Vladimir Druzenko freebsd_committer freebsd_triage 2024-11-07 19:44:39 UTC
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: /var/db/%%USER%%/debug.txt
Error: Orphaned: /var/db/%%USER%%/settings.dat
Error: Orphaned: /var/db/%%USER%%/sync.lng
===> Checking for items in pkg-plist which are not in STAGEDIR
===> Error: Plist issues found.
Comment 11 Volodymyr Kostyrko 2024-11-09 17:20:36 UTC
Created attachment 255055 [details]
update to 2.8.1

Big thanks for testing, overlooked that part.
Comment 12 commit-hook freebsd_committer freebsd_triage 2024-11-09 17:51:34 UTC
A commit in branch main references this bug:

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

commit 1d5854a31b343cd2162b9795d528734303abb28d
Author:     Volodymyr Kostyrko <arcade@b1t.name>
AuthorDate: 2024-11-09 17:45:34 +0000
Commit:     Vladimir Druzenko <vvd@FreeBSD.org>
CommitDate: 2024-11-09 17:50:10 +0000

    net-p2p/rslsync: Update 2.7.3 → 2.8.1.1390

    Changelog:
    https://help.resilio.com/hc/en-us/articles/206216855-Sync-2-x-change-log

    Improve port: split long lines, add NO_BUILD, merge do-build and
    do-install.

    PR:     281567

 net-p2p/rslsync/Makefile | 25 +++++++++++++++----------
 net-p2p/rslsync/distinfo | 10 +++++-----
 2 files changed, 20 insertions(+), 15 deletions(-)
Comment 13 Vladimir Druzenko freebsd_committer freebsd_triage 2024-11-09 17:51:59 UTC
Thanks.