Page MenuHomeFreeBSD

bsdinstall/partedit: Replace libdialog with libbsddialog
ClosedPublic

Authored by asiciliano on Mar 22 2022, 9:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 29, 5:50 AM
Unknown Object (File)
Tue, May 20, 5:19 AM
Unknown Object (File)
Tue, May 13, 11:28 PM
Unknown Object (File)
Tue, May 13, 6:34 AM
Unknown Object (File)
Fri, May 9, 9:46 AM
Unknown Object (File)
Apr 21 2025, 2:02 PM
Unknown Object (File)
Apr 20 2025, 5:23 PM
Unknown Object (File)
Apr 20 2025, 5:14 PM
Subscribers

Details

Summary

bsdinstall/partedit and sade(8): Replace (LGPL) libdialog with (BSD-2-Clause) libbsddialog. change only User Iterface everything else is left unchanged (for example an unused variable scripted.c:73:6: warning: variable 'error' set but not used [-Wunused-but-set-variable] int error;). Rewrite diskeditor.c because it uses an API totally incompatible with bsddialog.

Test Plan

The forms are the new components, all others are already in use. Probably sade uses all the dialogs so it is a good test case to continue the replacement process.

Diff Detail

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

Event Timeline

simple remark, not mandatory to follow ;), otherwise LGTM

usr.sbin/bsdinstall/partedit/diskmenu.c
60

using asprintf would simplify this

This revision is now accepted and ready to land.Mar 23 2022, 8:14 AM
This revision now requires review to proceed.Mar 23 2022, 3:39 PM

Apply tip

usr.sbin/bsdinstall/partedit/diskmenu.c
60

updated

This revision is now accepted and ready to land.Mar 23 2022, 3:43 PM

I am not familiar with dialogs or this code. I've tried to look for GEOM-related changes, but haven't found any.