Summary: | net-im/openfire: Update to 4.6.4 | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Nikita Druba <nikita> | ||||||
Component: | Individual Port(s) | Assignee: | Neel Chauhan <nc> | ||||||
Status: | Closed FIXED | ||||||||
Severity: | Affects Only Me | CC: | diizzy, nc, nikita, pi | ||||||
Priority: | --- | ||||||||
Version: | Latest | ||||||||
Hardware: | Any | ||||||||
OS: | Any | ||||||||
URL: | http://download.igniterealtime.org/openfire/docs/4.6.4/changelog.html | ||||||||
Attachments: |
|
Description
Nikita Druba
2021-06-21 15:07:25 UTC
Openfire release link: https://github.com/igniterealtime/Openfire/releases please note that your version of the distinfo file misses the TIMESTAMP entry. To be more exact: The patch for distinfo has 1624266366 and should probably have TIMESTAMP = 1624266366 (In reply to Kurt Jaeger from comment #2) Sorry, I generates all update by my script and probably something goes wrong... Now, I uploaded new diff version. What about src name? Created attachment 225976 [details]
git diff for net-im/openfire (corrected)
Comment on attachment 225965 [details]
git diff for net-im/openfire
Set old patch as obsolete.
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=3968e59f6c62174afa7f526544d902f2b507d0bb commit 3968e59f6c62174afa7f526544d902f2b507d0bb Author: Nikita Druba <nikita@druba.su> AuthorDate: 2021-06-26 06:23:40 +0000 Commit: Neel Chauhan <nc@FreeBSD.org> CommitDate: 2021-06-26 06:23:40 +0000 net-im/openfire: Update to 4.6.4 PR: 256748 net-im/openfire/Makefile | 7 +++---- net-im/openfire/distinfo | 10 +++++----- net-im/openfire/pkg-plist | 6 +++--- 3 files changed, 11 insertions(+), 12 deletions(-) Committed! Hi, This patch utilizes MASTER_SITES incorrectly A port should _never_ only have version in its file name as this will potential clash with other ports (hopefully not as this naming convention is a really bad idea) and it's also close to impossible to figure out where the tarball comes from unless you inspect it manually. You should use upstreams bug tracker to report this tarball naming issue https://igniterealtime.atlassian.net/browse/OF Either revert this update until upstream fixes the issue or submit a patch that makes use of USE_GITHUB instead of MASTER_SITES https://docs.freebsd.org/en/books/porters-handbook/book/#makefile-master_sites-github-description Best regards, Daniel Thanks for reporting this, Daniel! I was committing this from the plane (well, with SSH into my desktop at home), with (obvious) high latency. A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=138948297672f43af0fa75647288abc84602385d commit 138948297672f43af0fa75647288abc84602385d Author: Neel Chauhan <nc@FreeBSD.org> AuthorDate: 2021-06-26 18:31:06 +0000 Commit: Neel Chauhan <nc@FreeBSD.org> CommitDate: 2021-06-26 18:33:09 +0000 net-im/openfire: Switch to USE_GITHUB PR: 256748 Reported by: Daniel Engberg <daniel.engberg.lists AT pyret DOT net> net-im/openfire/Makefile | 8 ++++---- net-im/openfire/distinfo | 8 +++----- 2 files changed, 7 insertions(+), 9 deletions(-) (In reply to commit-hook from comment #10) Wonderfull! Do you understand that with this commit you oblige the user to download the sources during the build process, and not before? |