Bug 261778 - games/nbsdgames: update to 5
Summary: games/nbsdgames: update to 5
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: Fernando Apesteguía
URL: https://github.com/abakh/nbsdgames/re...
Keywords: easy
Depends on:
Blocks:
 
Reported: 2022-02-07 16:17 UTC by Robert Clausecker
Modified: 2022-02-08 09:18 UTC (History)
1 user (show)

See Also:


Attachments
games/nbsdgames: update to 5 (4.78 KB, patch)
2022-02-07 16:17 UTC, Robert Clausecker
fuz: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Clausecker freebsd_committer freebsd_triage 2022-02-07 16:17:29 UTC
Created attachment 231611 [details]
games/nbsdgames: update to 5

Upstream changes:

 - icon and menu shortcut included
 - tugow added
 - menu added
 - sudoku algorithm is replaced with a simple and fast algorithm that
   generates good puzzles in little time
 - miketron gameplay is improved upon
 - RedSquare now fits in the screen
 - command-line options are now in more conventional, getopt syntax
 - minor bug fixes

Tested with Poudriere on armv7 arm64 FreeBSD 13.
Comment 1 Fernando Apesteguía freebsd_committer freebsd_triage 2022-02-08 08:32:24 UTC
====> Running Q/A tests (stage-qa)
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: share/applications/nbsdgames.desktop
Error: Orphaned: share/icons/hicolor/scalable/apps/nbsdgames.svg
===> Checking for items in pkg-plist which are not in STAGEDIR
===> Error: Plist issues found.
*** Error code 1

^Triage: Please confirm this change passes QA (portlint, poudriere at least)

For details and instructions, see:

https://docs.freebsd.org/en/books/porters-handbook/#testing

No need to update a new patch. I'll fix it at commit time.
Comment 2 Robert Clausecker freebsd_committer freebsd_triage 2022-02-08 09:12:43 UTC
(In reply to Fernando Apesteguía from comment #1)

Hi,

Thank you for catching this.  I had forgotten to "git add" a last-minute change to distinfo.  The change reads:

diff --git a/games/nbsdgames/pkg-plist b/games/nbsdgames/pkg-plist
index 3649e7008be1..528cdc899286 100644
--- a/games/nbsdgames/pkg-plist
+++ b/games/nbsdgames/pkg-plist
@@ -25,6 +25,8 @@ bin/sudoku
 @sample(,games,0664) /var/games/nbsdgames/miketron_scores.sample
 @sample(,games,0664) /var/games/nbsdgames/pipes_scores.sample
 @sample(,games,0664) /var/games/nbsdgames/tugow_scores.sample
+share/applications/nbsdgames.desktop
+share/icons/hicolor/scalable/apps/nbsdgames.svg
 share/man/man6/battleship.6.gz
 share/man/man6/checkers.6.gz
 share/man/man6/darrt.6.gz

With this change, the patch passes QA on my machine.
Comment 3 Fernando Apesteguía freebsd_committer freebsd_triage 2022-02-08 09:18:22 UTC
Committed,

Thanks!
Comment 4 commit-hook freebsd_committer freebsd_triage 2022-02-08 09:18:29 UTC
A commit in branch main references this bug:

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

commit 03c4407ac9801a6d5619036daa70ba71bca942f6
Author:     Robert Clausecker <fuz@fuz.su>
AuthorDate: 2022-02-08 07:02:18 +0000
Commit:     Fernando Apesteguía <fernape@FreeBSD.org>
CommitDate: 2022-02-08 09:15:36 +0000

    games/nbsdgames: update to 5

    From ChangeLog: https://github.com/abakh/nbsdgames/releases/tag/v5

     * icon and menu shortcut included
     * tugow added
     * menu added
     * sudoku algorithm is replaced with a simple and fast algorithm that
       generates good puzzles in little time
     * miketron gameplay is improved upon
     * RedSquare now fits in the screen
     * command-line options are now in more conventional, getopt syntax
     * minor bug fixes

    PR:     261778
    Reported by:    fuz@fuz.su (maintainer)

 games/nbsdgames/Makefile             |  7 ++++++-
 games/nbsdgames/distinfo             |  6 +++---
 games/nbsdgames/files/patch-Makefile | 18 ++++++++++--------
 games/nbsdgames/pkg-plist            |  5 +++++
 4 files changed, 24 insertions(+), 12 deletions(-)