Page MenuHomeFreeBSD

bsddialog: in textbox mode differentiate between Exit and Extra buttons
ClosedPublic

Authored by glebius on Jan 24 2025, 7:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 27, 9:23 AM
Unknown Object (File)
Sat, Nov 22, 5:05 AM
Unknown Object (File)
Sat, Nov 22, 12:29 AM
Unknown Object (File)
Sun, Nov 9, 5:10 PM
Unknown Object (File)
Thu, Nov 6, 2:12 AM
Unknown Object (File)
Oct 24 2025, 7:32 PM
Unknown Object (File)
Oct 21 2025, 11:39 PM
Unknown Object (File)
Oct 12 2025, 4:13 PM
Subscribers

Details

Summary

If run --extra-button --textbox dialog will have two buttons, but
either of them would return 0 exit status. We definitely want the
Extra to report its value.

Diff Detail

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

Event Timeline

jlduran added a subscriber: jlduran.

After a simple:

grep -r -A3 "case 10" contrib/bsddialog/

I could not find any other occurrence that needs fixing.

This revision is now accepted and ready to land.Jan 25 2025, 2:23 AM

After a simple:

grep -r -A3 "case 10" contrib/bsddialog/

I could not find any other occurrence that needs fixing.

@jlduran Maybe the checks has to be performed inside the "clients" of bsddialog: bsdinstall, (mainly) bsdconfig, tzsetup, spkrtest, and kdbmap to avoid side effects.
Anyway the @glebius 's change makes sense. Honestly I forgot why textbox always returns OK, maybe for compatibility with cdialog. Anyway the next version (and import in contrib/) of bsddialog will have this change.

contrib/bsddialog/lib/textbox.c
203

Ok, this makes sense, honestly I forgot why textbox always returns OK, maybe compatibility with cdialog. Anyway I'll add 2 more changes/feature requests to bsddialog then I'll import a new version with this change.