Bug 261268 - archivers/laszip: Use upstream release archive
Summary: archivers/laszip: Use upstream release archive
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: Rainer Hurling
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-01-17 10:12 UTC by Daniel Engberg
Modified: 2022-01-18 20:50 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (rhurlin)


Attachments
Patch for laszip (1.32 KB, patch)
2022-01-17 10:12 UTC, Daniel Engberg
no flags Details | Diff
Patch for laszip v2 (2.48 KB, patch)
2022-01-17 14:23 UTC, Daniel Engberg
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Engberg freebsd_committer 2022-01-17 10:12:18 UTC
Created attachment 231071 [details]
Patch for laszip

Use upstream release archive as recommended by Porters Handbook (USE_GITHUB section)

Compile tested on FreeBSD 13.0-STABLE #2 stable/13-n248607-93a95ebbf7c (amd64) (make, make check-plist)
Comment 1 Daniel Engberg freebsd_committer 2022-01-17 14:23:03 UTC
Created attachment 231073 [details]
Patch for laszip v2

Apply https://src.fedoraproject.org/rpms/laszip/blob/rawhide/f/laszip_restoreapi.patch to fix support with devel/liblas
Comment 2 Daniel Engberg freebsd_committer 2022-01-17 14:46:15 UTC
Patch from Fedora doesn't fix the compatibility issues with devel/liblas
Comment 3 commit-hook freebsd_committer 2022-01-18 20:49:03 UTC
A commit in branch main references this bug:

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

commit 8ecff286ac2818d0aaec3680d63234dae73c4766
Author:     Daniel Engberg <diizzy@FreeBSD.org>
AuthorDate: 2022-01-18 20:46:52 +0000
Commit:     Rainer Hurling <rhurlin@FreeBSD.org>
CommitDate: 2022-01-18 20:48:18 +0000

    archivers/laszip: Use upstream release archive

    PR:             261268

 archivers/laszip/Makefile | 11 ++++-------
 archivers/laszip/distinfo |  6 +++---
 2 files changed, 7 insertions(+), 10 deletions(-)
Comment 4 Rainer Hurling freebsd_committer 2022-01-18 20:50:30 UTC
Hi Daniel,

Thanks for your patch. It's now committed.