linux-kernel - Re: Search function in xconfig is partially broken after recent changes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 25 Jun 2020 17:05:46 +0200
From:   Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
To:     Maxim Levitsky <mlevitsk@...hat.com>
Cc:     linux-kernel@...r.kernel.org
Subject: Re: Search function in xconfig is partially broken after recent
 changes

Em Thu, 25 Jun 2020 15:53:46 +0300
Maxim Levitsky <mlevitsk@...hat.com> escreveu:

> On Thu, 2020-06-25 at 13:17 +0200, Mauro Carvalho Chehab wrote:
> > Em Thu, 25 Jun 2020 12:59:15 +0200
> > Mauro Carvalho Chehab <mchehab+huawei@...nel.org> escreveu:
> >   
> > > Hi Maxim,
> > > 
> > > Em Thu, 25 Jun 2020 12:25:10 +0300
> > > Maxim Levitsky <mlevitsk@...hat.com> escreveu:
> > >   
> > > > Hi!
> > > > 
> > > > I noticed that on recent kernels the search function in xconfig is partially broken.
> > > > This means that when you select a found entry, it is not selected in the main window,
> > > > something that I often do to find some entry near the area I would like to modify,
> > > > and then use main window to navigate/explore that area.
> > > > 
> > > > Reverting these commits helps restore the original behavier:
> > > > 
> > > > b311142fcfd37b58dfec72e040ed04949eb1ac86 - kconfig: qconf: fix support for the split view mode
> > > > cce1faba82645fee899ccef5b7d3050fed3a3d10 - kconfig: qconf: fix the content of the main widget
> > > > 
> > > > I have Qt5 5.13.2 from fedora 31 (5.13.2-1.fc31)
> > > > 
> > > > Could you explain what these commits are supposed to fix?
> > > > I mostly use the split view mode too and it does appear to work for me with these commits reverted as well.
> > > >   
> > > 
> > > There are three view modes for qconf:
> > > 
> > > 	- Single
> > > 	- Split
> > > 	- Full
> > > 
> > > those got broken when gconf was converted to use Qt5, back on Kernel 3.14.
> > > Those patches restore the original behavior.  
> You mean xconfig/qconf? (gconf is another program that is GTK based as far as I know).

Yeah, I meant the Qt one (qconfig).

> Could you expalin though what was broken? What exactly didn't work?

Try to switch between the several modes and switch back. There used to
have several broken things there, because the Qt5 port was incomplete.

One of the things that got fixed on the Qt5 fixup series is the helper
window at the bottom. It should now have the same behavior as with the
old Qt3/Qt4 version.

Basically, on split mode, it should have 3 screen areas:

	+------------+-------+
	|            |       |
	| Config     |  menu |
	|            |       |
	+------------+-------+
	|                    |
	| Config description +
	|                    |
	+--------------------+

The contents of the config description should follow up any changes at 
the "menu" part of the split mode, when an item is selected from "menu",
or follow what's selected at "config", when the active window is "config".

The Kernel 3.14 conversion broke the "config description", and caused
several other issues.

When the config description area got fixed, I had to fix each of the
modes, for all of them to update the right area at the screen, as they
were pointing to the wrong places on several parts of the code.

> I do seem to be able to select menus on the left and the config items to the right,
> change the config item values, etc, in the split mode at least with these commits reverted.

You should also be able to see the helper window at the bottom changing
as modes change.

Anyway, the patch I just sent should fix it.

> Could you check that you also have the issue with search in qconf/xconfig?

Yeah, before that patch, search was working only on the two other
modes.

Btw, I'm also using Fedora here (Fedora 32). So, I should have a
result close to what you would be experiencing.

Thanks,
Mauro