Re: [PATCH 2/2] kconfig: support out of tree KCONFIG_CONFIG

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

 



2017-09-03 7:21 GMT+09:00 Nicolas Porcel <nicolasporcel06@xxxxxxxxx>:
> On Sat, Sep 02, 2017 at 04:31:44PM +0900, Masahiro Yamada wrote:
>> 2017-08-19 7:20 GMT+09:00 Nicolas Porcel <nicolasporcel06@xxxxxxxxx>:
>>
>> If the kernel source is not writable, O= should be used.
>
> You are right, I didn't know O= worked for kernel config.
>
>> I think KCONFIG_CONFIG only contains a file name
>> and this restriction is intentional
>> because conf_write() might be called with a directory path (for
>> example from gconf.c)
>>
>>
>> If we change the behavior,
>> I think conf_write() should just error-out
>> when directory path is given.
>>
>
> When I build a kernel, I set KCONFIG_CONFIG to reference a file on a
> different file-system than the build directory. It used to work before
> kernel 4.12, hence my patch.
>
> I now realize that I misunderstood the purpose of this variable. It
> also means that it is not possible to store the configuration file
> outside the build directory when using silentconfig for instance.


Some targets can choose the save path of the .config file.

For example, run "make gconfig"
then choose "File  ->  Save as"
from the menu.


But, silentoldconfig does not have a way to save .config
out out the build tree.


> Of course, I can use a symlink with KCONFIG_OVERWRITECONFIG, but it
> may be an interesting feature to be able to specify an absolute path
> for the kernel config file.



conf_write() may be called with a directory path,
in this case, the KCONFIG_OVERWRITECONFIG is appended
as the base name.

If we support an absolute path in KCONFIG_OVERWRITECONFIG,
I think it is strange.




-- 
Best Regards
Masahiro Yamada
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux