Bug 259786 - games/omega: Fix inventory display crash, Take MAINTAINER'ship
Summary: games/omega: Fix inventory display crash, Take MAINTAINER'ship
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Hiroki Tagato
URL:
Keywords: easy, needs-qa
Depends on:
Blocks:
 
Reported: 2021-11-11 23:35 UTC by Alex
Modified: 2021-12-12 07:59 UTC (History)
1 user (show)

See Also:
tagattie: merge-quarterly+

Attachments
maintainer update and bug fix (1.09 KB, patch)
2021-11-11 23:35 UTC, Alex 		koobs: maintainer-approval+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alex 2021-11-11 23:35:36 UTC 
Created attachment 229437 [details]
maintainer update and bug fix

Resolved crash / display issue when removing item from pack.
Adopting port.
Comment 1 Kubilay Kocak freebsd_committer freebsd_triage 2021-11-11 23:38:36 UTC 
Thank you for the report, patch and adopting the port Alex!
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2021-11-11 23:40:25 UTC 
Comment on attachment 229437 [details]
maintainer update and bug fix

Approved by: portmgr (blanket: bugfix, unmaintained port)
MFH: 2021Q4
Comment 3 Alex 2021-11-30 00:47:21 UTC 
Might be worth adding a PORTREVISION of 1 as well. The included change prevents crash / display problems, but the code still has some issues. For example, when compiling with clang, -O2 is required or display issues return (but no crash).
Comment 4 commit-hook freebsd_committer freebsd_triage 2021-12-12 07:56:14 UTC 
A commit in branch main references this bug:

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

commit dfc806ae70f1cd36860b6b90e4185d2198446f7a
Author:     Alex <r7st.guru@gmail.com>
AuthorDate: 2021-12-12 07:53:57 +0000
Commit:     Hiroki Tagato <tagattie@FreeBSD.org>
CommitDate: 2021-12-12 07:55:44 +0000

    games/omega: Fix inventory display crash and take maintainership

    PR:             259786
    Reported by:    Alex <r7st.guru@gmail.com> (new maintainer)
    Approved by:    portmgr (blanket: bugfix, unmaintained port)
    MFH:            2021Q4

 games/omega/Makefile          |  3 ++-
 games/omega/files/patch-inv.c | 14 ++++++++++++--
 2 files changed, 14 insertions(+), 3 deletions(-)
Comment 5 commit-hook freebsd_committer freebsd_triage 2021-12-12 07:58:15 UTC 
A commit in branch 2021Q4 references this bug:

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

commit a34084d590bbd0c46ca1509c17914d5865daaa81
Author:     Alex <r7st.guru@gmail.com>
AuthorDate: 2021-12-12 07:53:57 +0000
Commit:     Hiroki Tagato <tagattie@FreeBSD.org>
CommitDate: 2021-12-12 07:57:45 +0000

    games/omega: Fix inventory display crash and take maintainership

    PR:             259786
    Reported by:    Alex <r7st.guru@gmail.com> (new maintainer)
    Approved by:    portmgr (blanket: bugfix, unmaintained port)
    MFH:            2021Q4

    (cherry picked from commit dfc806ae70f1cd36860b6b90e4185d2198446f7a)

 games/omega/Makefile          |  3 ++-
 games/omega/files/patch-inv.c | 14 ++++++++++++--
 2 files changed, 14 insertions(+), 3 deletions(-)
Comment 6 Hiroki Tagato freebsd_committer freebsd_triage 2021-12-12 07:59:13 UTC 
Committed, thanks!