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)
Mon, Jun 22, 3:37 AM
Unknown Object (File)
Thu, Jun 18, 1:39 AM
Unknown Object (File)
Thu, Jun 4, 9:37 PM
Unknown Object (File)
May 15 2026, 2:47 PM
Unknown Object (File)
May 15 2026, 7:38 AM
Unknown Object (File)
May 14 2026, 9:41 PM
Unknown Object (File)
May 12 2026, 2:22 PM
Unknown Object (File)
May 11 2026, 8:06 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.