trns1997 commented on PR #16976:
URL: https://github.com/apache/nuttx/pull/16976#issuecomment-3279261158

   > When running:
   > 
   > ```
   > ./tools/configure.sh -E -l stm32f4discovery:nsh && make export -j8
   > ```
   > 
   > with set -e enabled, the build stops with:
   > 
   > ```
   > make: *** [tools/Unix.mk:781: export] Error 1
   > ```
   > 
   > Verbose output shows the actual failure:
   > 
   > ```
   > if [ -d "/home/thomas/nuttxspace/nuttx/nuttx-export-12.10.0/startup" ]; 
then \
   >   cp -f crt0.o 
"/home/thomas/nuttxspace/nuttx/nuttx-export-12.10.0/startup/."; \
   > else \
   >   echo "/home/thomas/nuttxspace/nuttx/nuttx-export-12.10.0/startup does 
not exist"; \
   >   exit 1; \
   > fi
   > ```
   > 
   > it’s because the startup/ directory inside the export tree is never 
created, and the script exits when it tries to copy crt0.o there.
   > 
   > This looks to be the same root cause seen here in [#16970 
(comment)](https://github.com/apache/nuttx/pull/16970#issuecomment-3254893411) 
. Hopefully solving this will resolve both problems :)
   
   This wasn’t the actual root cause. The real issue was the use of 
`2>/dev/null`, which was hiding the underlying error. I’ve updated the script 
to replace `2>/dev/null` with checks, allowing us to use `set -e`. This ensures 
that any errors are caught immediately and helps prevent regressions in 
`mkexport.sh`. The next step will be to add an out-of-tree build test in CI to 
verify that the exported content continues to work over time.


-- 
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

Reply via email to