Bug 276089 - devel/z80-asm: update to 2.4.1
Summary: devel/z80-asm: update to 2.4.1
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: Fernando Apesteguía
URL: http://wwwhomes.uni-bielefeld.de/achi...
Keywords:
Depends on:
Blocks:
 
Reported: 2024-01-03 00:21 UTC by Laurent
Modified: 2024-04-26 10:29 UTC (History)
1 user (show)

See Also:


Attachments
git-format-patch (7.95 KB, patch)
2024-01-03 00:21 UTC, Laurent
no flags Details | Diff
Patch 2: fix poudriere build (2.35 KB, patch)
2024-01-03 14:17 UTC, Laurent
no flags Details | Diff
Update to 2.4.1 (9.48 KB, patch)
2024-04-25 10:08 UTC, Laurent
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Laurent 2024-01-03 00:21:56 UTC
Created attachment 247426 [details]
git-format-patch

The currently in git is from 2006. This updates the port to the latest version, from 2018. 

I tried to deal with a problem the best way I could: the tar file is named z80-asm-current.tar.gz, but it untars into z80-asm-2.4.1. So I have a less than ideal `DISTNAME=${PORTNAME}-current` in the Makefile, but the danger of having a version named `current` is mitigated by explicitely setting `WRKSRC=${PWD}/work/${PORTNAME}-${PORTVERSION}`, PORTVERSION being 2.4.1. It's the best way that I found to deal with the discrepency. If there is a better way to do it, let me know!

I have contacted the maintainer upstream to see if he would take some patches, and if the tar file could be renamed to include a version number, but so far no answer. In a future update I might fork the source to my own github repo for this type of maintenance.
Comment 1 Laurent 2024-01-03 14:17:11 UTC
Created attachment 247434 [details]
Patch 2: fix poudriere build
Comment 2 Laurent 2024-01-03 14:20:44 UTC
Also removed LICENSE_FILE because the one bundled with the source is a template that's not been filled.
Comment 3 Fernando Apesteguía freebsd_committer freebsd_triage 2024-01-04 08:53:40 UTC
^Triage: [tags] in issue Titles are deprecated.

^Triage: Simplifying title

^Triage: If there is a changelog or release notes URL available for this version, please add it to the URL field.


Thanks!
Comment 4 Fernando Apesteguía freebsd_committer freebsd_triage 2024-04-21 12:03:10 UTC
Hi Laurent,

The patch does not apply. Would you mind having a look at it?
Comment 5 Laurent 2024-04-22 09:07:20 UTC
Yes there was a commit made after I created the patch. I'll fix it.
Comment 6 Laurent 2024-04-25 10:08:49 UTC
Created attachment 250220 [details]
Update to 2.4.1

Updated the patch to take into account a commit that was made recently and broke the previous patch.
Comment 7 Laurent 2024-04-25 10:11:01 UTC
Ready for commit

devel/z80-asm: Update to 2.4.1

Changes:
 - Update 2.4.p3 to 2.4.1
 - Fix various (but not all) compilation warnings

QA:
 - portlint:  no new warnings
 - poudriere: OK
Comment 8 Fernando Apesteguía freebsd_committer freebsd_triage 2024-04-26 10:28:40 UTC
Committed,

Thanks!
Comment 9 commit-hook freebsd_committer freebsd_triage 2024-04-26 10:29:01 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=88505e8a3f59e6dd0a71984f65d6bbefc2b9add2

commit 88505e8a3f59e6dd0a71984f65d6bbefc2b9add2
Author:     Laurent <laurent.chardon@gmail.com>
AuthorDate: 2024-04-25 16:28:36 +0000
Commit:     Fernando Apesteguía <fernape@FreeBSD.org>
CommitDate: 2024-04-26 10:28:09 +0000

    devel/z80-asm: update to 2.4.1

    ChangeLog: http://wwwhomes.uni-bielefeld.de/achim/z80-asm/CHANGELOG-2.4.1

    Submitter becomes maintainer. Already maintaining several ports.

    PR:             276089
    Reported by:    laurent.chardon@gmail.com

 devel/z80-asm/Makefile                            | 35 ++++--------------
 devel/z80-asm/distinfo                            |  7 ++--
 devel/z80-asm/files/patch-Makefile (new)          | 43 +++++++++++++++++++++++
 devel/z80-asm/files/patch-hardware_Makefile (new) | 29 +++++++++++++++
 devel/z80-asm/files/patch-memory.c (new)          | 38 ++++++++++++++++++++
 devel/z80-asm/files/patch-z80-asm.c (new)         | 11 ++++++
 devel/z80-asm/files/patch-z80-mon.c (new)         | 11 ++++++
 7 files changed, 141 insertions(+), 33 deletions(-)