Bug: `sort -V` not supported on NetBSD

[SirWumpus]

Is there an existing issue for this?

• I have searched for existing issues and did not find anything like this

Describe the bug

Low priority.

Copying from From #234 (comment) so as not to lose the thread.

After installing pandoc, attempted to generate gmake markdown the following error on NetBSD.

elf$ gmake markdown
=-=-=-=-= IOCCC begin gmake markdown =-=-=-=-=
You want to markdown?
bin/gen-top-html.sh markdown
sort: unknown option -- V
usage: sort [-bdfHilmnrSsu] [-k kstart[,kend]] [-o output] [-R char] [-T dir]
             [-t char] [file ...]
   or: sort -C|-c [-bdfilnru] [-k kstart[,kend]] [-o output] [-R char]
             [-t char] [file]
bin/pandoc-wrapper.sh: ERROR: pandoc: /usr/pkg/bin/pandoc version: 3.7.0.2 <= minimum allowed: 3.1.12.2
bin/pandoc-wrapper.sh: Warning: pandoc version must be >= 3.1.12.2
bin/pandoc-wrapper.sh: Warning: install a more up to date pandoc and/or use -p pandoc_tool to refer an up to date pandoc
bin/md2html.sh: ERROR: pandoc wrapper tool: bin/pandoc-wrapper.sh  -- .tmp.md2html.sh.STRIPPED_MD.1010.tmp - failed, error code: 5
bin/gen-top-html.sh: ERROR: md2html.sh: bin/md2html.sh -U https://www.ioccc.org/markdown.html -v 0 -t bin/gen-top-html.sh -D ./ -m markdown.md -- markdown.md markdown.html failed, error: 100
bin/gen-top-html.sh: Warning: EXIT_CODE set to: 1
bin/gen-top-html.sh: Warning: about to exit 1
gmake: *** [Makefile:704: markdown] Error 1


elf$ pkgin list | grep pandoc
...
pandoc-3.7.0.2       Conversion between documentation formats
pandoc-cli-3.7.0.2nb1 Conversion between documentation formats
elf$

From @xexyl
#234 (comment)

Hmmm ... that might be a flaw in the logic of verge(1). What do you think @lcn2?

If so one of us might have to fix it - but perhaps not until after this coming IOCCC?

What you expect

Don't know. Never ran this before.

Environment

• OS: Windows 10 Home hosting NetBSD 10.1 VM using VirtualBox
• Device: MSI Laptop
• Compiler:

Anything else?

No response

[xexyl]

Ahhh so maybe the problem is the sort command? I missed that earlier. It would make more sense - the commands I showed confirm that verge(1) is functioning right.

I'm not even sure (been a while) why we would need to sort versions. But if it's necessary I can see why it would fail without the option.

Must leave for now though.

[xexyl]

Although .. do you have verge installed?

Try from mkiocccentry:

cd jparse && make all && sudo make install

FYI also:

$ verge -h
usage: verge [-h] [-v level] [-V] major.minor.patch-1 major.minor.patch-2

	-h			print help message and exit
	-v level		set verbosity level (def level: 0)
	-V			print version strings and exit

	major.minor.patch-1	first version  (example: 0.1.1)
	major.minor.patch-2	second version (example: 1.3.2)

Exit codes:
     0   first version >= second version
     1   first version < second version
     2   -h and help string printed or -V and version strings printed
     3   command line error
     4   first or second version string is an invalid version
  >=10   internal error

verge version: 2.0.4 2025-09-01
jparse utils version: 2.1.7 2025-11-02
jparse UTF-8 version: 2.1.3 2025-07-04
jparse library version: 2.4.3 2025-11-10

[xexyl]

Okay looked at the script. It seems it does not even use verge(1). See:

# verify that the pandoc version is >= the minimum version
#
FIRST_VERSION=$(printf "%s\n%s" "$PANDOC_TOOL_VERSION" "$PANDOC_MIN_VERSION" | LC_ALL=C sort -V | head -1)
export FIRST_VERSION
if [[ $FIRST_VERSION != "$PANDOC_MIN_VERSION" ]]; then
    echo "$0: ERROR: pandoc: $PANDOC_TOOL version: $PANDOC_TOOL_VERSION <=" \
         "minimum allowed: $PANDOC_MIN_VERSION" 1>&2
    echo "$0: Warning: pandoc version must be >= $PANDOC_MIN_VERSION" 1>&2
    echo "$0: Warning: install a more up to date pandoc and/or use -p pandoc_tool to refer an up to date pandoc" 1>&2
    exit 5
fi

What do you think @lcn2? Should we make it use verge(1) instead?

[xexyl]

Okay looked at the script. It seems it does not even use verge(1). See:

verify that the pandoc version is >= the minimum version

FIRST_VERSION=$(printf "%s\n%s" "$PANDOC_TOOL_VERSION" "$PANDOC_MIN_VERSION" | LC_ALL=C sort -V | head -1)
export FIRST_VERSION
if [[ $FIRST_VERSION != "$PANDOC_MIN_VERSION" ]]; then
echo "$0: ERROR: pandoc: $PANDOC_TOOL version: $PANDOC_TOOL_VERSION <="
"minimum allowed: $PANDOC_MIN_VERSION" 1>&2
echo "$0: Warning: pandoc version must be >= $PANDOC_MIN_VERSION" 1>&2
echo "$0: Warning: install a more up to date pandoc and/or use -p pandoc_tool to refer an up to date pandoc" 1>&2
exit 5
fi
What do you think @lcn2? Should we make it use verge(1) instead?

... it seems like it would solve the problem. It's also what we have it for. It would require the users to install it but that's already the case for some of the other scripts anyway.

I must truly go now but maybe I can play with it tomorrow.

[xexyl]

I plan on fixing this in a bit. Not sure how long from now but should be this morning (assuming that it only takes the update to use verge which seems likely).

[xexyl]

I plan on fixing this in a bit. Not sure how long from now but should be this morning (assuming that it only takes the update to use verge which seems likely).

This in theory was fixed in 6cb22b9.

I tested it with make markdown myself. All good with my version which of course is the correct version (>= the min). It's very much like the other scripts that use verge(1).

The git log refers to the jparse repo as the source but notes that for IOCCC related things one should install from the mkiocccentry repo.

Please test this @SirWumpus and let us know if it is fine now. Thanks for reporting!

[xexyl]

Okay looked at the script. It seems it does not even use verge(1). See:

verify that the pandoc version is >= the minimum version

FIRST_VERSION=$(printf "%s\n%s" "$PANDOC_TOOL_VERSION" "$PANDOC_MIN_VERSION" | LC_ALL=C sort -V | head -1)
export FIRST_VERSION
if [[ $FIRST_VERSION != "$PANDOC_MIN_VERSION" ]]; then
echo "$0: ERROR: pandoc: $PANDOC_TOOL version: $PANDOC_TOOL_VERSION <="
"minimum allowed: $PANDOC_MIN_VERSION" 1>&2
echo "$0: Warning: pandoc version must be >= $PANDOC_MIN_VERSION" 1>&2
echo "$0: Warning: install a more up to date pandoc and/or use -p pandoc_tool to refer an up to date pandoc" 1>&2
exit 5
fi
What do you think @lcn2? Should we make it use verge(1) instead?

... it seems like it would solve the problem. It's also what we have it for.

Okay technically we have it for flex and bison but for the same idea .. :-)

Anyway I believe that the commit I just made will fix this bug.

[SirWumpus]

Is it worth installing the small natsort program? See https://github.com/sourcefrog/natsort/blob/master/natsort.c

[xexyl]

Is it worth installing the small natsort program? See https://github.com/sourcefrog/natsort/blob/master/natsort.c

Not worth it no. The verge(1) command is how it's done now - and how it was meant to be done all along.

[SirWumpus]

Is it worth installing the small natsort program? See https://github.com/sourcefrog/natsort/blob/master/natsort.c

Not worth it no. The verge(1) command is how it's done now - and how it was meant to be done all along.

I cannot find verge.sh

elf$ pwd
/home/achowe/git/ioccc_src/ioccc-winners
elf$ find . -name 'verge*'
elf$