[ISSUE] man uses more instead of less when cross built

[ISSUE] man uses more instead of less when cross built

[lfdebrux]

[-- Attachment #1: Type: text/plain, Size: 1285 bytes --]

New issue by lfdebrux on void-packages repository

https://github.com/void-linux/void-packages/issues/37512

Description:
<!-- Don't request update of package. We have a script for that. https://repo-default.voidlinux.org/void-updates/void-updates.txt. However, a quality pull request may help. -->
### System

* xuname: Void 5.15.41_1 x86_64-musl GenuineIntel notuptodate rFFFF

* package:  mdocml-1.14.6_3

### Expected behavior

`man` uses `less` as the default pager.

### Actual behavior

`man` uses `more` as the default pager.

### Steps to reproduce the behavior

- check out latest version of void-packages repo
- crossbuild mdocml (for instance with `./xbps-src -a armv5tel-musl pkg mdocml`
- install mdocml on target device
- open a man page (for instance `man man`)

### Notes

I think this happens because although less is a build dependency of mdocml, the configure script for mandoc tries to run less to check whether it is present, but the cross version gets installed in `usr/arm-linux-musleabi/bin`, which isn't in PATH (and probably wouldn't run anyway being a non-native binary).

I'm not sure of the right way to fix this, setting `BINM_PAGER` in `configure.local` would probably do the job, but maybe there is a more standard way of fixing this sort of problem?

Re: man uses more instead of less when cross built

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 2425 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/issues/37512#issuecomment-1152957310

Comment:
`hostmakedepends="less"` solves this

Looks like there are a lot of execution failures on cross:

<details><summary>configure log of mdocml</summary>

```
file config.log: writing...
file configure.local: reading...
tested operating system: Linux -> OSENUM=MANDOC_OS_OTHER
tested aarch64-linux-gnu-gcc -W: tested noop: yes
selected CFLAGS="-fstack-clash-protection -D_FORTIFY_SOURCE=2 -O2 -pipe -march=armv8-a -fcommon  -I/usr/aarch64-linux-gnu/usr/include -fdebug-prefix-map=/builddir/mandoc-1.14.6=."
tested noop-static: yes
selected STATIC="-static"
tested attribute: yes
tested cmsg: yes
tested dirent-namlen: no (execution failed)
tested be32toh: yes
tested be32toh-DSYS_ENDIAN: no (execution failed)
tested EFTYPE: no (execution failed)
tested err: yes
tested getline: yes
tested getsubopt: yes
tested isblank: yes
tested mkdtemp: yes
tested mkstemps: yes
tested nanosleep: yes
tested ntohl: yes
tested O_DIRECTORY: yes
tested PATH_MAX: yes
tested pledge: no (execution failed)
tested sandbox_init: no (execution failed)
tested progname: no (execution failed)
tested reallocarray: yes
tested recallocarray: no (execution failed)
tested recallocarray-D_OPENBSD_SOURCE: no (execution failed)
tested recvmsg: yes
tested rewb-bsd: HAVE_REWB_BSD=0 (manual)
tested rewb-sysv: yes
tested strcasestr: no (execution failed)
tested strcasestr-D_GNU_SOURCE: yes
tested stringlist: no (execution failed)
tested strlcat: no (execution failed)
tested strlcpy: no (execution failed)
tested strndup: yes
tested strptime: no (execution failed)
tested strptime-D_GNU_SOURCE: yes
tested strsep: yes
tested strtonum: no (execution failed)
tested strtonum-D_OPENBSD_SOURCE: no (execution failed)
tested vasprintf: no (execution failed)
tested vasprintf-D_GNU_SOURCE: yes
tested fts-DFTS_COMPARE_CONST: no (execution failed)
tested fts: yes
tested less: yes
selected BINM_PAGER=less
tested less -T: yes
selected UTF8_LOCALE=en_US.utf8
tested wchar-DUTF8_LOCALE="en_US.utf8": no (execution failed)
tested wchar-D_GNU_SOURCE -DUTF8_LOCALE="en_US.utf8": yes
tested ohash: no (execution failed)
tested ohash-lutil: no (execution failed)
selected LDADD="    -lz"
file config.h: written
file Makefile.local: written
```

</details>

Re: [ISSUE] [CLOSED] man uses more instead of less when cross built

[leahneukirchen]

[-- Attachment #1: Type: text/plain, Size: 1695 bytes --]

Closed issue by lfdebrux on void-packages repository

https://github.com/void-linux/void-packages/issues/37512

Description:
<!-- Don't request update of package. We have a script for that. https://repo-default.voidlinux.org/void-updates/void-updates.txt. However, a quality pull request may help. -->
### System

* xuname: Void 5.15.41_1 x86_64-musl GenuineIntel notuptodate rFFFF

* package:  mdocml-1.14.6_3

### Expected behavior

`man` uses `less` as the default pager. (The mdocml package has less as a dependency so less should always be installed with man on a Void Linux system)

### Actual behavior

`man` uses `more` as the default pager.

### Steps to reproduce the behavior

- check out latest version of void-packages repo
- crossbuild mdocml (for instance with `./xbps-src -a armv5tel-musl pkg mdocml`
- install mdocml on target device
- open a man page (for instance `man man`)

### Notes

I think this happens because although less is a build dependency of mdocml, the configure script for mandoc tries to run less to check whether it is present, but the cross version gets installed in `usr/arm-linux-musleabi/bin`, which isn't in PATH (and probably wouldn't run anyway being a non-native binary).

I figured out what was going on by running `xbps-src -a armv5tel-musl configure mdocml`, and looking at `masterdir/builddir/mandoc*/config.log`. I also verified that the same thing happened when cross-compiling for different targets (and different architectures).

I'm not sure of the right way to fix this, setting `BINM_PAGER` in `configure.local` would probably do the job, but maybe there is a more standard way of fixing this sort of problem?