Page MenuHomeFreeBSD

bsdconfig(8): add bsddialog(1)
ClosedPublic

Authored by asiciliano on Nov 23 2022, 8:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 13 2024, 4:19 AM
Unknown Object (File)
Mar 13 2024, 4:11 AM
Unknown Object (File)
Mar 13 2024, 3:50 AM
Unknown Object (File)
Dec 20 2023, 4:15 AM
Unknown Object (File)
Dec 10 2023, 7:07 PM
Unknown Object (File)
Nov 30 2023, 1:00 AM
Unknown Object (File)
Nov 30 2023, 12:53 AM
Unknown Object (File)
Nov 27 2023, 1:27 PM
Subscribers

Details

Summary

Add bsddialog(1) to bsdconfig(8). This can be considered an increment not a replacement, `$DIALOG=dialog' restores dialog(1).

Test Plan

bsddialog 0.4 was released to match the features of bsdconfig.

To test, with a recent CURRENT:
$DIALOG=bsddialog changing /usr/share/bsdconfig/dialog.subr like the review.

otherwise

$ git clone https://gitlab.com/alfix/bsddialog.git
$ cd bsddialog
$ make

$DIALOG=/absolute/path/bsddialog changing /usr/share/bsdconfig/dialog.subr like the review.

Of course it is possible to apply the diff in src/.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Nov 24 2022, 7:48 AM
jrtc27 added inline comments.
usr.sbin/bsdconfig/share/dialog.subr
82–90

Does this not break Xdialog?

2305

Wrong dialog?

usr.sbin/bsdconfig/share/dialog.subr
82–90

Ok. I think you refer to DIALOG_ESC for Xdialog. Otherwise, please let me know how I can reproduce the fault.

I'll fix it under, where -X is parsed/processed and USE_XDIALOG and $DIALOG=Xdialog are set.

2305

OK. I' ll fix.

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

This revision now requires review to proceed.Jan 21 2023, 12:48 PM
This revision was not accepted when it landed; it landed in state Needs Review.Oct 5 2023, 7:31 AM
This revision was automatically updated to reflect the committed changes.