Prashant Rahul created a merge request: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1063

Project:Branches: prashantrahul141/rtems:contrib/microshell to 
rtems/rtos/rtems:main
Author:   Prashant Rahul



## Summary

Issue: #5312 

This ports microshell as a alternative shell in rtems. This pr is to get 
initial thoughts and reviews.

There are few things which needs to be clarified/fixed here
1. Microshell build should be conditional, right now it gets build everytime. 
How exactly should this be setup?

2. Currently `microshell/src/inc/ush_preconfig.h` file includes 
`"config/ush_config_platform.h"`, that would mean creating a `config` folder 
inside `$PREFIX/7/arch/board/lib/include` (example: 
`$PREFIX/7/aarch64-rtems7/zynqmp_qemu_ilp32/lib/include`). Which I have done in 
the buildspec for microshell, but it seems redundent and I was thinking of 
instead patching microshell to include something like 
`"microshell/config/ush_config_platform.h"`. That way `config` folder can be 
moved inside `microshell` folder which seems "cleaner" to me.

## TODO:
- [ ] conditional build for microshell
- [ ] decide header location
- [ ] DOCS: Existence of microshell inside rtems
- [ ] DOCS: Usage example

## Testing
I have been testing this on sim for the arm/xilinx_zynq_a9_qemu bsp. I am 
planning to test this real hardware very soon.

## Generative AI

<!-- If you have used AI please use the "AI Contribution" template otherwise 
leave this blank see our fulls statement at 
https://www.rtems.org/generative-ai/-->


<!-- Default settings, if it is a dropdown it will set after submission -->

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1063
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to