Page MenuHomeFreeBSD

games/kanatest: revive port
ClosedPublic

Authored by fuz on Feb 11 2023, 7:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 15, 8:59 PM
Unknown Object (File)
Mon, Apr 15, 4:22 PM
Unknown Object (File)
Thu, Apr 11, 7:19 PM
Unknown Object (File)
Fri, Mar 22, 11:30 PM
Unknown Object (File)
Fri, Mar 22, 11:30 PM
Unknown Object (File)
Mar 9 2024, 7:46 AM
Unknown Object (File)
Mar 9 2024, 7:33 AM
Unknown Object (File)
Jan 27 2024, 2:27 PM
Subscribers

Details

Summary
[New Port] games/kanatest: revive port

During test the Kanatest displays randomly selected kana char
(respecting mode and lesson) and waits for user answer expected as
romaji equivalent. This process continues until all questions will be
answered or all questions will be answered correctly (depends on
options). At the end of test a short info about drilling time and
correctness ratio is displayed. The results are stored and user can
review his performance in any time.

Submitter becomes maintainer.

Changelog: https://sourceforge.net/p/kanatest/code/ci/master/tree/ChangeLog

PR:		268895
Test Plan

Tested on i386 amd64 FreeBSD 12.4 13.1. arm64 pending.
See http://fuz.su/~fuz/freebsd/batch1/

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 49733
Build 46623: arc lint + arc unit

Event Timeline

fuz requested review of this revision.Feb 11 2023, 7:21 PM

PR mentions a fork at https://github.com/Clockwork6400/kanatest

This diff puts WWW back to original url https://kanatest.sourceforge.io/ with develpment stoped at 2018-xx-xx: version 0.4.10.
(Changelog: https://sourceforge.net/p/kanatest/code/ci/master/tree/ChangeLog)

From where 0.4.10.g20200611 comes from and why are we reviving a port with upstream dead?

PR mentions a fork at https://github.com/Clockwork6400/kanatest

This diff puts WWW back to original url https://kanatest.sourceforge.io/ with develpment stoped at 2018-xx-xx: version 0.4.10.
(Changelog: https://sourceforge.net/p/kanatest/code/ci/master/tree/ChangeLog)

From where 0.4.10.g20200611 comes from and why are we reviving a port with upstream dead?

The submitter has taken the last commit from the repository I use as the upstream, did some unknown modifications (supposedly portability fixes) to it and then uploaded it to github. The repository used by the submitter does not preserve any history or metadata and is not meant to be a fork, merely a rehost due to the difficulty in obtaining the state of the upstream before it was discontinued. In communication with the submitter I had modified and fixed his patch to ensure that it builds and that the official upstream is used instead of the submitter's rehosted version.

The version number follows Porter's Handbook § 5.4.3 Ex. 18 and 19, indicating “a version before the unreleased 0.4.10 taken from a git commit dated 2020-06-11.” This is as I understood general policy for generating version numbers that are taken from a git commit. The port was revived because the submitter believed it to be useful and agreed to maintain it.

The version number follows Porter's Handbook § 5.4.3 Ex. 18 and 19, indicating “a version before the unreleased 0.4.10 taken from a git commit dated 2020-06-11.” This is as I understood general policy for generating version numbers that are taken from a git commit. The port was revived because the submitter believed it to be useful and agreed to maintain it.

OK, I found that commit (g20200611) at https://sourceforge.net/p/kanatest/code/ci/860e790a35f547cc96669f805d371a5ba3d8daff/

  • Build (stage) fails with NLS=OFF

I remember having same problem when gettext not present: NLS_USES= gettext

Making install in po
gmake[2]: Entering directory '/wrkdirs/usr/ports/games/kanatest/work/kanatest-0.4.10.g20200611/po'
gmake[3]: Entering directory '/wrkdirs/usr/ports/games/kanatest/work/kanatest-0.4.10.g20200611/po'
gmake[3]: Nothing to be done for 'install-exec-am'.
/bin/sh: msgfmt: not found
(...)
for i in de.mo el.mo es.mo fi.mo fr.mo hr.mo hu.mo it.mo pl.mo pt.mo ru.mo zh_CN.mo; do \
        lang=`echo $i | sed 's/\.mo$//'`; \
        /bin/sh /wrkdirs/usr/ports/games/kanatest/work/kanatest-0.4.10.g20200611/install-sh -d /wrkdirs/usr/ports/games/kanatest/work/stage/usr/local/share/locale/$lang/LC_MESSAGES; \
        install  -m 0644 $i /wrkdirs/usr/ports/games/kanatest/work/stage/usr/local/share/locale/$lang/LC_MESSAGES/kanatest.mo; \
done
install: de.mo: No such file or directory
(...)
  • pkg-plist should be sorted alphabetically:
bin/kanatest
share/applications/...
share/icons/hicolor/...
%%NLS%%share/locale/...
share/pixmaps/kanatest.png
fuz planned changes to this revision.Feb 13 2023, 10:17 AM

Build (stage) fails with NLS=OFF

I think it's best to just remove the NLS knob entirely at this point. Clearly the package does not have support for building without NLS support.

pkg-plist should be sorted alphabetically

It is sorted mostly alphabetically, but taking the % signs into account. Will try to adjust the sort as per your guidance.

In D38519#877035, @fuz_fuz.su wrote:

Build (stage) fails with NLS=OFF

I think it's best to just remove the NLS knob entirely at this point. Clearly the package does not have support for building without NLS support.

pkg-plist should be sorted alphabetically

It is sorted mostly alphabetically, but taking the % signs into account. Will try to adjust the sort as per your guidance.

It still take some advantage of NLS by installing or not %%NLS%% files in pkg-plist but gettext must be included in USES and NLS_USES= gettext removed.

Updated the patch to unconditionally have USES=gettext.
Sorted the pkg-plist alphabetically as per guidance.

I'm following PR discussion about chosen do-fetch mechanism.
I approve this method since it's the only way to get a snapshot from SF but a local snapshot will be good and safer too.

Please feel free to choose the best fetch method for this port.

This revision is now accepted and ready to land.Feb 14 2023, 9:02 AM

Closed through bb71c4161707a95f215dc5f24165310265cb6cf0 which for some reason didn't trigger the auto-close.