Bug 245882

Summary: [PATCH] x11-drivers/xorg-drivers: make more user friendly options
Product: Ports & Packages Reporter: Ivan Rozhuk <rozhuk.im>
Component: Individual Port(s)Assignee: freebsd-x11 (Nobody) <x11>
Status: Closed Not Accepted    
Severity: Affects Some People CC: rozhuk.im, zeising
Priority: --- Flags: bugzilla: maintainer-feedback? (x11)
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch none

Description Ivan Rozhuk 2020-04-24 15:37:46 UTC
Created attachment 213755 [details]
patch

Make options desctiptions user friendly - extract ports COMMENT.

┌──────────────────────────── xorg-drivers-7.7_6 ──────────────────────────────┐
│ ┌──────────────────────────────────────────────────────────────────────────┐ │  
│ │────────────────────────────────── INPUT ─────────────────────────────────│ │  
│ │ [ ] EGALAX         eGalax touch screen input driver for X.Org            │ │  
│ │ [ ] ELOGRAPHICS    X.Org elographics input driver                        │ │  
│ │ [ ] EVDEV          X.Org event device input driver                       │ │  
│ │ [ ] JOYSTICK       X.Org joystick input driver                           │ │  
│ │ [x] KEYBOARD       X.Org keyboard input driver                           │ │  
│ │ [x] LIBINPUT       X.Org libinput input driver                           │ │  
│ │ [x] MOUSE          X.Org mouse input driver                              │ │  
│ │ [ ] SYNAPTICS      X.Org synaptics input driver                          │ │  
│ │ [ ] VMMOUSE        X.Org vmmouse input driver                            │ │  
│ │ [ ] VOID           X.Org void input driver                               │ │  
│ │ [ ] WACOM          X.Org legacy Wacom tablet driver                      │ │  
│ │────────────────────────────── Video support ─────────────────────────────│ │  
│ │ [ ] AMDGPU         X.Org amdgpu display driver                           │ │  
│ │ [ ] APM            X.Org apm display driver                              │ │  
│ └─────┴(+)─────────────────────────────────────────────────────────34%─────┘ │  
├──────────────────────────────────────────────────────────────────────────────┤  
│                       <  OK  >            <Cancel>                           │  
└──────────────────────────────────────────────────────────────────────────────┘
Comment 1 Niclas Zeising freebsd_committer freebsd_triage 2020-04-24 20:23:40 UTC
I'm not sure it's a good idea to run make several times just to get the ports comments.
Comment 2 Ivan Rozhuk 2020-04-24 20:40:19 UTC
(In reply to Niclas Zeising from comment #1)

Original "Install xxxx yyyy driver" in many cases mean nothing untill xxxxx googled.

To reduce CPU usage it is possible to make "config-opts" target that will be only called before options show to user and move all code that generates options description to it.
Comment 3 Niclas Zeising freebsd_committer freebsd_triage 2020-09-17 09:37:42 UTC
I don't like this, as I've stated earlier.  Running make in a loop just to get descriptions for the list feels both error prone and can be resource intensive.
Closing this.

Plus, in this case you rarely need to stray from the defaults, I think.