config shell & bt shell same time cause fail

[lgl88911]

Hi,pals:
I find eable the config same time, will fail:

CONFIG_SHELL=y

CONFIG_NET_SHELL=y
CONFIG_BT_SHELL=y
CONFIG_NET_L2_BT_SHELL=y

the log:

-- Generating zephyr/include/generated/generated_dts_board.h
-- Cache files will be written to: /home/frank/.cache/zephyr
CMake Error at /home/frank/work/project/zephyr/cmake/extensions.cmake:796 (add_subdirectory):
  The binary directory

    /home/frank/work/project/nrf52_moderate/apps/ipsptest/build/zephyr/subsys/shell

  is already used to build a source directory.  It cannot be used to build
  source directory

    /home/frank/work/project/zephyr/subsys/shell

  Specify a unique binary directory name.
Call Stack (most recent call first):
  /home/frank/work/project/zephyr/subsys/CMakeLists.txt:7 (add_subdirectory_ifdef)

if disable CONFIG_SHELL or other config will pass.
The means bt shell cmd and shell history function can't co-exist. This is a issue or sepc？Can support bt shell cmd using history function? Thanks !

[jakub-uC]

I see that bt commands have been recently moved to the new shell. Do you still have an issue with BT shell?

[jakub-uC]

I digged in a bit more.
New shell has different design compering to the old one. We are in the middle of transition so it may not still not work.
Current concept is that you can enable shell commands via Kconfig for NET, BT and so on. But shell instance itself needs to be created separately.

[lgl88911]

@jarz-nordic Yes, I tested look like you。 BT_SHELL is old shell style , but CONFIG_SHELL is new shell style. Is there a plan covert BT&Net shell to new shell style?

[jakub-uC]

@lgl88911 : yes of course. I hope that in 2 weeks we will be able to do it.
Once PR: #10511 is merged it will be very easy task.

[lgl88911]

@jarz-nordic Thanks, I‘ll wait your news

[jakub-uC]

@lgl88911 : Please track PR #9825. Once it will be merged you shall be able to use new shell.

[jakub-uC]

@lgl88911 : Please check now if it is working as you expect it.

[lgl88911]

@jarz-nordic Update code & test OK, Thanks a lot！
config:

CONFIG_BT_SHELL=y
CONFIG_NET_L2_BT_SHELL=y
CONFIG_PRINTK=y
CONFIG_KERNEL_SHELL=y
CONFIG_NET_SHELL=y
CONFIG_SHELL=y
CONFIG_LOG_BACKEND_UART=n
CONFIG_LOG=y
CONFIG_SHELL_HISTORY=y

Board run shell:

uart:~$ 
  bt       clear    gatt     help     history  kernel   l2cap    log
  net      net_bt   resize   shell    ticker