Call to deleted constructor of 'BT::Tree'clang(ovl_deleted_init) compiled error when using `clang-15` and FIXED

Bug report

Required Info:

Operating System:
- Ubuntu 20.04

ROS2 Version:

rolling

11:05:41 czg@czg-Lenovo-Legion-R7000-2020 ~ → dpkg -s ros-rolling-desktop 
Package: ros-rolling-desktop
Status: install ok installed
Priority: optional
Section: misc
Installed-Size: 44
Maintainer: Steven! Ragnarök <steven@openrobotics.org>
Architecture: amd64
Version: 0.9.3-1focal.20220122.094456

Version or commit hash:
- ci-jammy-rolling branch choosed based on ros-index
DDS implementation:
- I didn't specify any rmw_implementation， so just the default.

Steps to reproduce issue

use clang instead of gcc/g++


njtech@robot-ai-1:~/RMUA_WS$ tail ~/.bashrc -n 3
source /opt/ros/rolling/setup.bash
export CC=clang CXX=clang++
alias colcon-build="colcon build  --cmake-args '-DCMAKE_EXPORT_COMPILE_COMMANDS=1' --symlink-install "

njtech@robot-ai-1:~/RMUA_WS$ clang -v Ubuntu clang version 15.0.0-++20220309112335+52fb4f53c29e-1~exp1~20220309232347.184 Target: x86_64-pc-linux-gnu Thread model: posix InstalledDir: /usr/bin Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/10 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/11 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/8 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/9 Selected GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/11 Candidate multilib: .;@m64 Selected multilib: .;@m64


#### Expected behavior
```bash
$ colcon-build
Summary: 69 packages finished [16min 16s]                                                                                                         
  23 packages had stderr output: armor_vision costmap_converter cpp_pubsub livox_ros2_driver livox_sdk_vendor nav2_amcl nav2_behavior_tree nav2_br
ingup nav2_bt_navigator nav2_costmap_2d nav2_planner nav2_recoveries nav2_regulated_pure_pursuit_controller nav2_rotation_shim_controller nav2_sma
c_planner nav2_smoother nav2_system_tests nav2_waypoint_follower realsense2_camera roborts_sdk rplidar_sdk teb_local_planner turtlebot3_node

Actual behavior

clangd lint info

Call to deleted constructor of 'BT::Tree'clang(ovl_deleted_init)
bt_factory.h(141, 5): 'Tree' has been explicitly marked deleted here

colcon build errors


In file included from /home/njtech/RMUA_WS/src/navigation2/nav2_bt_navigator/src/bt_navigator.cpp:15:                                             
In file included from /home/njtech/RMUA_WS/src/navigation2/nav2_bt_navigator/include/nav2_bt_navigator/bt_navigator.hpp:27:                       
In file included from /home/njtech/RMUA_WS/src/navigation2/nav2_bt_navigator/include/nav2_bt_navigator/navigators/navigate_to_pose.hpp:24:        
In file included from /home/njtech/RMUA_WS/src/navigation2/nav2_bt_navigator/include/nav2_bt_navigator/navigator.hpp:27:                          
/home/njtech/RMUA_WS/install/nav2_behavior_tree/include/nav2_behavior_tree/bt_action_server.hpp:173:12: error: call to deleted constructor of 'BT:
:Tree'                                                                                                                                            
return tree_;                                                                                                                                 
       ^~~~~                                                                                                                                  
/opt/ros/rolling/include/behaviortree_cpp_v3/bt_factory.h:141:5: note: 'Tree' has been explicitly marked deleted here                             
Tree(const Tree& ) = delete;                                                                                                                  
^

#### Additional information
- **workaround** to comply with `// non-copyable. Only movable` comment in `behaviortree_cpp_v3/bt_factory.h` header which last modified in 
    ```diff
    diff --git a/nav2_behavior_tree/include/nav2_behavior_tree/bt_action_server.hpp b/nav2_behavior_tree/include/nav2_behavior_tree/bt_action_server.hpp
    index e423afcb..faf23843 100644
    --- a/nav2_behavior_tree/include/nav2_behavior_tree/bt_action_server.hpp
    +++ b/nav2_behavior_tree/include/nav2_behavior_tree/bt_action_server.hpp
    @@ -168,7 +168,7 @@ public:
        * @brief Getter function for the current BT tree
        * @return BT::Tree Current behavior tree
        */
    -  BT::Tree getTree() const
    +  const BT::Tree& getTree() const
    {
        return tree_;
    }

after git log --graph --all --oneline I found both main and ci-jammy-rolling bt_action_server.hpp last modified at Nov 20, 2021 c15ccbf don‘t change BT::Tree getTree() const
- then can modify similarly by
```
BT::Tree* foo_tree = &foo_class.getTree();
foo_tree->...
//    or call member function(s)
```
- or use BehaviorTreeFactory::createTree... official function
  - this function create a lvalue in the function， and return a rvalue and call non-deleted factory function, so no errors throw.
    - is my opinion right? :thinking:
      
      My coding habits may be bad.
      
      If you had better methods to solve.
      
      Welcome to point out and offer me some helpful links to better learning programming if any, with many thanks :grin:

ros-navigation / navigation2