Probably the review title and description can be changed, because it wants to introduce GUI for timezone. (Otherwise, if its purpose is adding backtitle to tzsetup (timezone) we can add just 5-6 lines to tzsetup.c :D). Maybe:

In D44670#1018433, @khorben_defora.org wrote:

However I can look into creating a different version of this replacement of timezone with some tweaking in bsdconfig instead.

I fixed in upstream (https://gitlab.com/alfix/bsddialog/-/commit/ce220b82ad546d3518a805750e5ee6add73f1fbf) because the problem also appears in other places in this file. I'll test and import the new version 1.0.3 by 2-3 days.

Ok, I have the same patch locally.

LGTM

LGTM. I tested the new selectdists file in the my XFCE Terminal. I wrote some tip, you can consider or ignore.

New solution.

In D41507#949813, @pauamma_gundo.com wrote:

Maybe, now that the call for testing is out, add a link to https://lists.freebsd.org/archives/freebsd-accessibility/2023-August/000034.html at the end of the report so people know how to test?

Thanks for the review, I updated to fix.

In D41507#945746, @carlavilla wrote:

Please use one sentence per line and the package macro.
If you don’t know how to use the macro tell me and I’ll help.

Btw, looks good, thanks for this improvement!

@carlavilla I fixed one sentence per line, but I don't know package macro and how to use them.

In D35754#943208, @markj wrote:

In D35754#943181, @asiciliano wrote:

I am interested to take/continue this feature. Are others already working?

Not that I've seen. I think there are two major things which need to be addressed: the userspace interface should be converted to use file descriptors rather than sysctls, and we should find a way to avoid making term_char_t larger.

I am interested to take/continue this feature. Are others already working?

I am interested to take/continue this feature. Are others already working?

I am interested to take/continue this feature. Are others already working?

Thank you for the review, the updated diff should fix.

ping @dteske

Abandon: this replacement can be implied just by changing the $DIALOG variable in bsdconfig.

Abandon: uses struct sysctl_oid definition previous FreeBSD 1400071

Thanks!

Abandon: This review excessively impacts on current features.

Hi, thank you for the review. I like the lock update.

This review can close a PR for fetchmissingdists https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=265797

In D35776#817747, @hselasky wrote:

Should we pass arguments to espeak using an environment variable perhaps?

I just got your e-mail and will have a look at it!

Sorry for the late testing, I tested also with edbrowse and espeak in Italian[*] , it is all right.

Thanks for the reply, It seems all right.

Hi @hselasky this is an awesome feature, I am not an expert but I am interested in the subject.

add --hline string, see Summary.