Re: [PATCH 0/4] kconfig: Only activate UI save widgets when .config changed; Take 3

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 11 Dec 2006 14:08:51 +0100 Karsten Wiese wrote:

> Am Sonntag, 10. Dezember 2006 09:10 schrieb Andrew Morton:
> > 
> > So I'm pretending to be kbuild maintainer and I now realise I simply don't
> > know what this patch series does.
> > 
> > Can you please explain it a lot more?
> 
> lets "make xconfig" on a freshly untarred kernel-tree.
> look at the floppy disk icon of the qt application, that has just started:
> Its in a normal, active state.
> Mouse click on it:
> .config is being saved.
> Now in the current -mm head version, you'll notice something new:
> after the mouse click on the floppy disk icon, the icon is greyed out.
> If you mouse click on it now, nothing happens, which is ok IMHO,
> as nothing has changed.
> If you change some CONFIG_*, the floppy disk icon returns to "active state",
> that is, if you mouse click it now, .config is written.
> The "icon greying out" aka "save widget de/activation" is done by this
> patch series.
> Its done for the save-icons and the file-save menu entries of the
> applications launched after "make xconfig" or "make gconfig" is called.
> Purpose is: 
> "let the (gtk/qt kernel configuration) application show me if there are
>  unsaved changes to the .config,
>  so there's 1 thing less I might wonder about"
> 
> more?

I reviewed the patches and tested menu/x/gconfig.
They all look good to me.

Acked-by: Randy Dunlap <[email protected]>

Thanks,
---
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux