Page MenuHomeFreeBSD

Convert bsdinstall/partedit sade(8) for libbsddialog
AbandonedPublic

Authored by asiciliano on Dec 16 2021, 12:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 20, 12:04 AM
Unknown Object (File)
Sat, Sep 19, 11:23 PM
Unknown Object (File)
Sat, Sep 19, 11:06 PM
Unknown Object (File)
Sun, Sep 13, 8:45 PM
Unknown Object (File)
Thu, Sep 10, 5:33 AM
Unknown Object (File)
Sun, Sep 6, 8:52 PM
Unknown Object (File)
Sun, Sep 6, 6:33 PM
Unknown Object (File)
Sun, Sep 6, 3:58 PM
Subscribers

Details

Summary

First try for bsdinstall/partedit sade(8) with libbsddialog.

I am considering a call for testing, hackers@? (Test Plan below).

This review depends on libbsddialog 0.0.2 https://reviews.freebsd.org/D33580

Test Plan

Menu, checkelist, msgbox , yesno and infobox tested via tzsetup, kbdmap and portconfig.

We need to test forms (this is the first real world use/test case).
"Isolation" test with debug symbols:

% git clone https://gitlab.com/alfix/bsddialog.git
% cd bsddialog/
% git clone https://gitlab.com/alfix/freebsd-lab.git
% cd freebsd-lab/bsdinstall/partedit/
% make -DDEBUG
% ./sade

To check:

  • Forms: <Create> and <Modify>
  • ZFS

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

asiciliano edited the test plan for this revision. (Show Details)
asiciliano added a reviewer: allanjude.
asiciliano edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Dec 21 2021, 3:16 PM

Update diff:

  • fix: style(9) withespace and comment
  • delete: useless constant (diskmenu.h)
  • fix: focus on the last selected item after a diskmenu refresh (like diskeditor)
This revision now requires review to proceed.Dec 29 2021, 10:23 PM

Abandon Review: a complete libbsddialog version is almost ready, I' ll update a new snapshot and a revision for sade soon.