You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/09/09 09:17:24 UTC

[GitHub] [incubator-nuttx] Gary-Hobson opened a new pull request, #7048: Optimizing mkallsyms scripts with python

Gary-Hobson opened a new pull request, #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048

   ## Summary
   When compiling my project, the time to generate allsyms decreased from tens of seconds to 2 seconds
   
   ## Impact
   
   ## Testing
   Enable CONFIG_ALLSYMS
   and execute pip install pyelftools cxxfilt
   


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1259804762

   @Gary-Hobson please move the change for Dockerfile and cibuild.sh to the new patch.


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] Gary-Hobson commented on a diff in pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
Gary-Hobson commented on code in PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#discussion_r981319061


##########
arch/sim/src/.gitignore:
##########
@@ -1,3 +1,4 @@
+/allsyms.tmp

Review Comment:
   done



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1244847348

   > Does this mean that Python becomes a mandatory part of the build process?
   
   Yes, if you enable CONFIG_ALLSYMS. But I would said that python is more portable than shell script if you want to support Windows.


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] anchao commented on a diff in pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
anchao commented on code in PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#discussion_r982439328


##########
arch/arm/src/Makefile:
##########
@@ -157,7 +157,7 @@ board$(DELIM)libboard$(LIBEXT):
 	$(Q) $(MAKE) -C board libboard$(LIBEXT) EXTRAFLAGS="$(EXTRAFLAGS)"
 
 define LINK_ALLSYMS
-	$(Q) $(TOPDIR)/tools/mkallsyms.sh $(NUTTX) $(CROSSDEV) > allsyms.tmp

Review Comment:
   Using nm you need to filter the text segment,
   Seems that elftools can filter out the garbage symbols in the results you shared



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] Gary-Hobson commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
Gary-Hobson commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1260304489

   > @Gary-Hobson please move the change for Dockerfile and cibuild.sh to the new patch.
   
   done


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on code in PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#discussion_r982437745


##########
arch/sim/src/Makefile:
##########
@@ -300,7 +300,9 @@ nuttx-names.dat: nuttx-names.in
 	$(call PREPROCESS, nuttx-names.in, nuttx-names.dat)
 
 define LINK_ALLSYMS
-	$(Q) $(TOPDIR)/tools/mkallsyms.sh noconst $(NUTTX) $(CROSSDEV) > allsyms.tmp
+	$(if $(CONFIG_HOST_MACOS), \
+	$(Q) $(TOPDIR)/tools/mkallsyms.sh noconst $(NUTTX) $(CROSSDEV) > allsyms.tmp, \

Review Comment:
   why not use mkallsyms.py too



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1241774945

   @Gary-Hobson please fix the lint error:
   https://github.com/apache/incubator-nuttx/runs/8267006939?check_suite_focus=true


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] anchao commented on a diff in pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
anchao commented on code in PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#discussion_r982353664


##########
arch/arm/src/Makefile:
##########
@@ -157,7 +157,7 @@ board$(DELIM)libboard$(LIBEXT):
 	$(Q) $(MAKE) -C board libboard$(LIBEXT) EXTRAFLAGS="$(EXTRAFLAGS)"
 
 define LINK_ALLSYMS
-	$(Q) $(TOPDIR)/tools/mkallsyms.sh $(NUTTX) $(CROSSDEV) > allsyms.tmp

Review Comment:
   You need to compare the symbol output  of command nm, for example using arm-none-eabi-nm and  host "nm" are completely different



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1250232154

   > I gave up replacing the default mkallsyms.py because using it requires installing some dependencies. I may need to add the dependency in CI and then replace it
   
   You can install the dependencies from:
   https://github.com/apache/incubator-nuttx/blob/master/tools/ci/docker/linux/Dockerfile
   https://github.com/apache/incubator-nuttx/blob/master/tools/ci/cibuild.sh#L49
   


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] Gary-Hobson commented on a diff in pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
Gary-Hobson commented on code in PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#discussion_r982465049


##########
arch/sim/src/Makefile:
##########
@@ -300,7 +300,9 @@ nuttx-names.dat: nuttx-names.in
 	$(call PREPROCESS, nuttx-names.in, nuttx-names.dat)
 
 define LINK_ALLSYMS
-	$(Q) $(TOPDIR)/tools/mkallsyms.sh noconst $(NUTTX) $(CROSSDEV) > allsyms.tmp
+	$(if $(CONFIG_HOST_MACOS), \
+	$(Q) $(TOPDIR)/tools/mkallsyms.sh noconst $(NUTTX) $(CROSSDEV) > allsyms.tmp, \

Review Comment:
   The executable file for Mac is not in elf format, and pyelftools does not parse the symbol table correctly



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] Gary-Hobson commented on a diff in pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
Gary-Hobson commented on code in PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#discussion_r982348282


##########
arch/arm/src/Makefile:
##########
@@ -157,7 +157,7 @@ board$(DELIM)libboard$(LIBEXT):
 	$(Q) $(MAKE) -C board libboard$(LIBEXT) EXTRAFLAGS="$(EXTRAFLAGS)"
 
 define LINK_ALLSYMS
-	$(Q) $(TOPDIR)/tools/mkallsyms.sh $(NUTTX) $(CROSSDEV) > allsyms.tmp

Review Comment:
   pyelftools has no options for cross-compiling toolchains,it is for parsing of generic elf files
   



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on code in PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#discussion_r982437745


##########
arch/sim/src/Makefile:
##########
@@ -300,7 +300,9 @@ nuttx-names.dat: nuttx-names.in
 	$(call PREPROCESS, nuttx-names.in, nuttx-names.dat)
 
 define LINK_ALLSYMS
-	$(Q) $(TOPDIR)/tools/mkallsyms.sh noconst $(NUTTX) $(CROSSDEV) > allsyms.tmp
+	$(if $(CONFIG_HOST_MACOS), \
+	$(Q) $(TOPDIR)/tools/mkallsyms.sh noconst $(NUTTX) $(CROSSDEV) > allsyms.tmp, \

Review Comment:
   why not use mkallsyms.py too and remove mkallsyms.sh



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] anchao commented on a diff in pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
anchao commented on code in PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#discussion_r982325711


##########
arch/xtensa/src/Makefile:
##########
@@ -134,7 +134,7 @@ board/libboard$(LIBEXT):
 	$(Q) $(MAKE) -C board libboard$(LIBEXT) EXTRAFLAGS="$(EXTRAFLAGS)"
 
 define LINK_ALLSYMS
-	$(Q) $(TOPDIR)/tools/mkallsyms.sh $(NUTTX) $(CROSSDEV) > allsyms.tmp
+	$(Q)$(TOPDIR)/tools/mkallsyms.py $(NUTTX) allsyms.tmp

Review Comment:
   keep the space



##########
arch/arm/src/Makefile:
##########
@@ -157,7 +157,7 @@ board$(DELIM)libboard$(LIBEXT):
 	$(Q) $(MAKE) -C board libboard$(LIBEXT) EXTRAFLAGS="$(EXTRAFLAGS)"
 
 define LINK_ALLSYMS
-	$(Q) $(TOPDIR)/tools/mkallsyms.sh $(NUTTX) $(CROSSDEV) > allsyms.tmp

Review Comment:
    $(CROSSDEV) is mandatory, otherwise nm will print some unnecessary information



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] hartmannathan commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
hartmannathan commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1244839892

   Does this mean that Python becomes a mandatory part of the build process?


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] hartmannathan commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
hartmannathan commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1246928110

   > who can help me! Some CIs report errors, but I can't find the reason from the log, who can tell me what I should do?
   > 
   > ```
   > make[1]: *** [Makefile:324: nuttx] Error 2
   > make: *** [tools/Unix.mk:509: nuttx] Error 2
   > make: Target 'all' not remade because of errors.
   > /github/workspace/sources/nuttx/tools/testbuild.sh: line 290: /github/workspace/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory
   >   Normalize sim/minmea
   > ```
   
   Linker error? Forgot to add one or more files to CSRCS in a Makefile somewhere?


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on code in PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#discussion_r973812056


##########
.gitignore:
##########
@@ -38,6 +38,7 @@
 /.project
 /.version
 /_SAVED_APPS_config
+/allsyms.tmp

Review Comment:
   remove



##########
arch/sim/src/.gitignore:
##########
@@ -1,3 +1,4 @@
+/allsyms.tmp

Review Comment:
   ditto



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
pkarashchenko commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1243254101

   Yes, but that's what I see from CI logs


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
pkarashchenko commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1242673832

   Probably we need to add `allsyms.tmp` to `.gitignore`


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] Gary-Hobson commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
Gary-Hobson commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1243212181

   > Probably we need to add `allsyms.tmp` to `.gitignore`
   
   Normally you shouldn't see allsyms.tmp because it gets deleted after use.
   ```
   define LINK_ALLSYMS
   	$(Q) $(TOPDIR)/tools/mkallsyms.py noconst $(NUTTX) allsyms.tmp
   	$(Q) $(call COMPILE, -x c allsyms.tmp, allsyms$(OBJEXT))
   	$(if $(CONFIG_HAVE_CXX),\
   	$(Q) "$(CXX)" $(CFLAGS) $(LDFLAGS) -o $(NUTTX) \
   	      $(HEADOBJ) nuttx.rel $(HOSTOBJS) $(STDLIBS) allsyms$(OBJEXT),\
   	$(Q) "$(CC)" $(CFLAGS) $(LDFLAGS) -o $(NUTTX) \
   	      $(HEADOBJ) nuttx.rel $(HOSTOBJS) $(STDLIBS) allsyms$(OBJEXT))
   	$(Q) $(call DELFILE, allsyms.tmp allsyms$(OBJEXT))
   endef
   ```


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] Gary-Hobson commented on a diff in pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
Gary-Hobson commented on code in PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#discussion_r982348042


##########
arch/xtensa/src/Makefile:
##########
@@ -134,7 +134,7 @@ board/libboard$(LIBEXT):
 	$(Q) $(MAKE) -C board libboard$(LIBEXT) EXTRAFLAGS="$(EXTRAFLAGS)"
 
 define LINK_ALLSYMS
-	$(Q) $(TOPDIR)/tools/mkallsyms.sh $(NUTTX) $(CROSSDEV) > allsyms.tmp
+	$(Q)$(TOPDIR)/tools/mkallsyms.py $(NUTTX) allsyms.tmp

Review Comment:
   done



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] Gary-Hobson commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
Gary-Hobson commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1250194337

   I gave up replacing the default mkallsyms.py because using it requires installing some dependencies.
   I may need to add the dependency in CI and then replace it


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1260442076

   let rebase to the last.


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] Gary-Hobson commented on a diff in pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
Gary-Hobson commented on code in PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#discussion_r982410655


##########
arch/arm/src/Makefile:
##########
@@ -157,7 +157,7 @@ board$(DELIM)libboard$(LIBEXT):
 	$(Q) $(MAKE) -C board libboard$(LIBEXT) EXTRAFLAGS="$(EXTRAFLAGS)"
 
 define LINK_ALLSYMS
-	$(Q) $(TOPDIR)/tools/mkallsyms.sh $(NUTTX) $(CROSSDEV) > allsyms.tmp

Review Comment:
   There is a little difference between them, but I don't think it will make much difference
   
   [nm.md](https://github.com/apache/incubator-nuttx/files/9665404/nm.md)
   [pyelftools.md](https://github.com/apache/incubator-nuttx/files/9665405/pyelftools.md)
   



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] Gary-Hobson commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
Gary-Hobson commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1246913533

   who can help me!
   Some CIs report errors, but I can't find the reason from the log, who can tell me what I should do?
   ```
   make[1]: *** [Makefile:324: nuttx] Error 2
   make: *** [tools/Unix.mk:509: nuttx] Error 2
   make: Target 'all' not remade because of errors.
   /github/workspace/sources/nuttx/tools/testbuild.sh: line 290: /github/workspace/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory
     Normalize sim/minmea
   ```


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #7048: Optimizing mkallsyms scripts with python

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on PR #7048:
URL: https://github.com/apache/incubator-nuttx/pull/7048#issuecomment-1246938772

   You can repro the problem locally by ./tools/configure.sh sim:minmea


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org