On Sat, Apr 25, 2020, 3:01 PM Abdelatif Guettouche < abdelatif.guettou...@gmail.com> wrote:
> We have addressed the concerns raised here. > Brennan, could you give it another go? This time it should be RC1, right? > Happy to cut another release, although I think this next one should be sent directly for the 72hr vote. if people are OK with it. The one concern I have is we did not do much to send someone totally new in the right direction with regards to building the code. I know some of it can be addressed outside of the release code, but do we need to have a really short guide to send someone on the right track to build the SIM or something like the STM32DISCOVERY4? Something that gets you to shell from 0 in 10min. > BTW; as you can see in Github we have merged some of the apps' > backported PRs with failing tests. Those PRs were low risk since they > changed only text files. It seems that they are failing because PRs > made against apps/ releases/9.0 pull nuttx/ master instead of > releases/9.0. The CI changes I made only made it to the OS repo and not to the Apps repo. I just opened two PR and tagged you that include the change for master and the release branch. I think with those the app CI should be green. --Brennan >