pkarashchenko commented on PR #6318: URL: https://github.com/apache/incubator-nuttx/pull/6318#issuecomment-1256318814
> > @hartmannathan do you have any test results? > > I'm not sure what happened here: > > ``` > 5126 git checkout master --force > 5127 git pull > 5128 wget https://github.com/apache/incubator-nuttx/pull/6318.patch > 5129 git apply 6318.patch > 5130 git status > 5131 make distclean > 5132 tools/configure.sh b-g474e-dpow1:nsh > 5133 make menuconfig > ``` > > (Added `CONFIG_BUILTIN`, `CONFIG_NSH_BUILTIN_APPS`, and `CONFIG_TESTING_OSTEST`.) > > ``` > 5134 make savedefconfig > 5135 make > ``` > > (Snip) > > ``` > CC: semaphore/sem_holder.c > semaphore/sem_holder.c: In function 'nxsem_restoreholderprio': > semaphore/sem_holder.c:439:26: error: 'struct tcb_s' has no member named 'waitsem'; did you mean 'waitdog'? > if ((stcb->waitsem->flags & PRIOINHERIT_FLAGS_DISABLE) == 0) > ^~~~~~~ > waitdog > semaphore/sem_holder.c:444:49: error: 'struct tcb_s' has no member named 'waitsem'; did you mean 'waitdog'? > if (pholder->sem == stcb->waitsem && > ^~~~~~~ > waitdog > make[1]: *** [Makefile:58: sem_holder.o] Error 1 > make[1]: Leaving directory '/home/nate/ramdrive/NuttX/nuttx/sched' > make: *** [tools/LibTargets.mk:59: sched/libsched.a] Error 2 > ``` > > This is strange because PR check above shows all checks passed. > > What configuration are you using to test with? Yeah... Recent changes on mainline removed mqwait and renamed semwait to generic void wait obj. I will rebase the changes and fix compilation -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
