lkml.org 
[lkml]   [2013]   [Nov]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: make oldnoconfig fix?
On 11/11/2013 10:13 AM, Yann E. MORIN wrote:
> David, All,

Hi Yann,

>
> On 2013-11-11 10:11 -0800, David Cohen spake thusly:
>> Is there a plan to fix 'make oldnoconfig' option?
>> I currently have need to it :)
>> I've tracked a thread fixing 'make olddefconfig' where 'oldnoconfig'
>> option is reported as a known bug, right?
>
> Sorry, I don't understand what your problem is.

Maybe I assumed it was more 'known' then it actually is :)

>
> Care to explain in details what problem you have:
> - what you start with
> - what you do
> - what you get
> - what you expected

The bug with 'oldnoconfig' is: 'make oldnoconfig' is just an alias to
'olddefconfig'. Any non-existent config is defined with its default
value instead of 'N'.

I work with embedded platforms. I need to maintain a set of defconfigs
for different products (customers some times want tailored configs, so
creating a generic one isn't an option always).

In order to have full control of the .config generated out of
defconfigs, I don't allow any CONFIG_ to be selected implicitly. So
everytime a Kconfig is changed in our internal tree, we need to update
all defconfigs either with CONFIG_... = [y|n] or # CONFIG_... is not set

If I managed to get 'make oldnoconfig' working, I could not care about
# CONFIG_... is not set and internal and changes to defconfigs would be
easily to understand and review.

>
> So we can try to reproduce the issue and investigate what is wrong.

All we need to know is explained in this commit:

commit fb16d8912db5268f29706010ecafff74b971c58d
Author: Adam Lee <adam8157@gmail.com>
Date: Sat Sep 1 01:05:17 2012 +0800

kconfig: replace 'oldnoconfig' with 'olddefconfig', and keep the
old name as an alias

As 67d34a6a391369269a2e5dba8a5f42cc4cd50231 said, 'oldnoconfig' doesn't
set new symbols to 'n', but instead sets it to their default values.

So, this patch replaces 'oldnoconfig' with 'olddefconfig', stop making
people confused, and keep the old name 'oldnoconfig' as an alias,
because people already are dependent on its behavior with the
counter-intuitive name.

Signed-off-by: Adam Lee <adam8157@gmail.com>
Signed-off-by: Michal Marek <mmarek@suse.cz>


>
> Since you mention an existing thread, please add a pointer (URL) to
> this thread in the archives (Use: http://marc.info/?l=linux-kbuild ).

This is the thread I was talking about:
http://comments.gmane.org/gmane.linux.kbuild.devel/8560

Master Yoda promised to fix it when the problem gets bigger :)

Br, David


\
 
 \ /
  Last update: 2013-11-11 20:21    [W:0.061 / U:0.932 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site