Bug 271348 - games/chessx: update to 1.6.0
Summary: games/chessx: update to 1.6.0
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: Robert Clausecker
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-05-10 11:43 UTC by Alexey Vyskubov
Modified: 2023-05-11 23:27 UTC (History)
1 user (show)

See Also:


Attachments
patch to upgrade games/chessx port to the version 1.6.0 (5.52 KB, patch)
2023-05-10 11:43 UTC, Alexey Vyskubov
no flags Details | Diff
New version of the patch, updating chessx to 1.6.0 (5.74 KB, patch)
2023-05-10 13:41 UTC, Alexey Vyskubov
no flags Details | Diff
ChessX changelog, as of version 1.6.0. (36.01 KB, text/plain)
2023-05-10 14:06 UTC, Alexey Vyskubov
no flags Details
Yet another version of the patch (7.11 KB, patch)
2023-05-11 12:50 UTC, Alexey Vyskubov
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Vyskubov 2023-05-10 11:43:25 UTC
Created attachment 242103 [details]
patch to upgrade games/chessx port to the version 1.6.0

Hello, the attached patch upgrades chessx to 1.6.0 (current version in FreeBSD is 1.5.7, and it is a beta version).

Since the port does not have a maintainer, and I am
- actively using FreeBSD,
- actively using chessx,
- have a lot of time,
I would like to volunteer to become a maintainer of this port.

-- 
Alexey
Comment 1 Robert Clausecker freebsd_committer freebsd_triage 2023-05-10 13:18:59 UTC
Thank you for your submission and your wish to maintain this port.  This is very appreciated!  Please check the following issues:

 - please use DISTVERSION over PORTVERSION if possible
 - please clear PORTREVISION when updating a port to a new version

Rest of the patch looks good!
Comment 2 Alexey Vyskubov 2023-05-10 13:21:20 UTC
Will do. I will also add installation of the application icon, which I removed due to the strange problems.
Comment 3 Alexey Vyskubov 2023-05-10 13:41:51 UTC
Created attachment 242105 [details]
New version of the patch, updating chessx to 1.6.0
Comment 4 Robert Clausecker freebsd_committer freebsd_triage 2023-05-10 13:46:36 UTC
Will commit with small changes.
Comment 5 Robert Clausecker freebsd_committer freebsd_triage 2023-05-10 13:47:01 UTC
Is there a changelog for this update?
Comment 6 Alexey Vyskubov 2023-05-10 13:50:38 UTC
Indeed, there is a changelog. How do I add it to port?
Comment 7 Alexey Vyskubov 2023-05-10 14:06:05 UTC
Created attachment 242106 [details]
ChessX changelog, as of version 1.6.0.
Comment 8 Robert Clausecker freebsd_committer freebsd_triage 2023-05-10 15:18:30 UTC
Add a link to the bug report.  I'll go and add it to the commit message.
Comment 9 Alexey Vyskubov 2023-05-10 15:22:05 UTC
I do not know how to do that. Changelog is a part of tarball, which is hosted on sourceforge. I'll try to find it in a linkable state.
Comment 10 Robert Clausecker freebsd_committer freebsd_triage 2023-05-10 15:23:55 UTC
If it's not available online, that's okay.  For this update, I'll just link to the changelog in attachment #242106 [details].  For future updates, we'll have to find a solution.
Comment 11 Alexey Vyskubov 2023-05-10 15:26:11 UTC
Maybe this -- https://github.com/Isarhamster/chessx/blob/master/ChangeLog.md -- but I am not sure it is "official".
Comment 12 Alexey Vyskubov 2023-05-10 15:29:58 UTC
It is official. The source code link at ChessX website points to this GitHub repository.
Comment 13 Robert Clausecker freebsd_committer freebsd_triage 2023-05-10 23:40:16 UTC
A test build failed with plist issues:

===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: share/metainfo/io.sourceforge.ChessX.metainfo.xml
Error: Orphaned: @dir share/icons/hicolor
===> Checking for items in pkg-plist which are not in STAGEDIR
===> Error: Plist issues found.
*** Error code 1


Please check and resubmit.
Comment 14 Alexey Vyskubov 2023-05-11 12:50:25 UTC
Created attachment 242119 [details]
Yet another version of the patch

Also added a link to the version 1.6.0 changelog to the commit message.
Comment 15 Alexey Vyskubov 2023-05-11 12:50:55 UTC
Is it better now?
Comment 16 Robert Clausecker freebsd_committer freebsd_triage 2023-05-11 14:21:26 UTC
Looks good.  Will commit with some small changes.
Comment 17 commit-hook freebsd_committer freebsd_triage 2023-05-11 23:27:12 UTC
A commit in branch main references this bug:

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

commit d40f8cab97ebcb5687ddf5a7744581a999bbd23d
Author:     Alexey Vyskubov <alexey@pentode.fi>
AuthorDate: 2023-05-10 11:27:45 +0000
Commit:     Robert Clausecker <fuz@FreeBSD.org>
CommitDate: 2023-05-11 23:23:30 +0000

    games/chessx: Update to 1.6.0.

    * Volunteer to be a maintainer
    * Do not include obsolete i18n: see
      https://sourceforge.net/p/chessx/wiki/HOWTO%20Translate/
    * Point to the proper license file
    * Fix bug preventing compilation if USE_SPEECH is not defined

    Changelog: https://github.com/Isarhamster/chessx/blob/master/ChangeLog.md#may-2023-version-160

    PR:             271348

 games/chessx/Makefile                              | 21 ++++--
 games/chessx/distinfo                              |  6 +-
 games/chessx/files/patch-chessx.pro                | 80 ++++++++++------------
 .../files/patch-src_gui_chessxsettings.cpp (new)   | 16 +++++
 .../files/patch-src_gui_chessxsettings.h (new)     | 19 +++++
 5 files changed, 90 insertions(+), 52 deletions(-)
Comment 18 Robert Clausecker freebsd_committer freebsd_triage 2023-05-11 23:27:59 UTC
Had to apply a little FreeBSD 12 hotfix to make it work.
Thank you for your contribution!