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 2021/05/22 18:33:13 UTC
[GitHub] [incubator-nuttx] a-lunev opened a new pull request #3763: boards: lm3s6965-ek: configuration to test nxflat
a-lunev opened a new pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763
boards: lm3s6965-ek: added config to run NuttX with enabled NXFLAT binary loader mode on QEMU;
Conditionally turn off -mpic-data-is-text-relative flag to access bss via the GOT (https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=139630111).
This is related to issue #3737.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637443997
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
@patacongo and @a-lunev thanks for explanation.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637438523
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
why remove it from ci?
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] a-lunev commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
a-lunev commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637440947
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
> But from the comment, the special compiler flags is supported by gcc now?
The special compiler flag is supported beginning from gcc-4.9.4. However, there are also required special NXFLAT tools (mknxflat and ldnxflat) that are not part of GNU GCC.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] a-lunev commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
a-lunev commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637440947
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
> But from the comment, the special compiler flags is supported by gcc now?
The special compiler flag is supported beginning from gcc-4.9.4. However, there are also required special NXFLAT tools (mknxflat and ldnxflat) that are not part of GNU GCC.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] a-lunev commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
a-lunev commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637439474
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
> why remove it from ci?
Hi @xiaoxiang781216,
Any nxflat configuration requires NXFLAT Toolchain for building. The standard GNU Toolchain will fail. (https://cwiki.apache.org/confluence/display/NUTTX/NxFlat)
I analyzed scripts in nuttx/tools/ci and I did not find NXFLAT Toolchain setup in the scripts.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637438530
##########
File path: tools/ci/testlist/arm-13.dat
##########
@@ -7,4 +7,5 @@
-eagle100:thttpd
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
same question.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] patacongo commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
patacongo commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637440757
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
>
>
> But from the comment, the special compiler flags is supported by gcc now?
This has nothing to do with GCC. These are the NxFLAT tools that are provided in the Bitbucket buildroot tools.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] patacongo commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
patacongo commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637440757
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
>
>
> But from the comment, the special compiler flags is supported by gcc now?
This has nothing to do with GCC. These are the NxFLAT tools that are provided in the Bitbucket buildroot tools. These: https://bitbucket.org/nuttx/buildroot/src/master/toolchain/nxflat/
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] a-lunev commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
a-lunev commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637441139
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
@btashton, it would be great!
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] patacongo commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
patacongo commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637575714
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
> From the header it says ldnxflat.c was originally from elf2flt.c which is GPL V2+ so I am not sure it could realistically become BSD. This makes it quite to hard to carry in a official capacity, especially with the additional dependency that it is built against binutils. Please do correct me if I am wrong here.
>
> https://github.com/uclinux-dev/elf2flt/blob/main/elf2flt.c
Yeah.. I forgot about that file.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] btashton commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
btashton commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637570828
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
> > If that is OK with you I can submit the PR to bring that into the CI pipeline.
>
> Sure. I we can put into a new incubator-nuttx-tools repository if you like. I have the copyright on all of the code and I think all of the nxflat code is BSD. There may be some references to some of the older XFLAT code which I released as GPL, but I wrote those too and also have the copyright. So I think that the licensing in the buildroot/toolchain/nxflat directory is okay.
>
From the header it says ldnxflat.c was originally from elf2flt.c which is GPL V2+ so I am not sure it could realistically become BSD. This makes it quite to hard to carry in a official capacity, especially with the additional dependency that it is built against binutils. Please do correct me if I am wrong here.
https://github.com/uclinux-dev/elf2flt/blob/main/elf2flt.c
> The vision that I had for the buildroot tools is that that there would be once place that you could just configure and make and get all of the tools that you need. So it also build the nuttx-eabi or nuttx-elf toolchain (which is a good thing), nxflat, and things like genromfs. It is a pretty handy concept and, I think, still worthy of support.
>
> The advantage of the nuttx-elf toolchain is that it does not include any foreign C library header files or logic; it builds against the NuttX C library and eliminates all of those incompatibilities.
Here though for nxflat tools itself I don't think we are gaining anything though as it is only using the BFD library, but please correct me if I am wrong. I do understand the benefits of having a NuttX toolchain in general.
As for the github.com/nuttx org I have no interest in it. We will likely have to remove that as part of graduation as it will be representing as the NuttX project without operating inline with the Apache requirements. These things need to be under the Apache project or be a different project. I am offering a home for this tool as I have a personal interest in it and if we do find a better place, happy to see it move there. I tried to be very upfront in README that this is your work.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637439690
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
But from the comment, the special compiler flags is supported by gcc now?
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] btashton commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
btashton commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637439839
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
It also needs the nxflat tools. I can get the CI side sorted out today if you would like @a-lunev at least for Linux.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] patacongo commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
patacongo commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637549192
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
> If that is OK with you I can submit the PR to bring that into the CI pipeline.
Sure. I we can put into a new incubator-nuttx-tools repository if you like. I have the copyright on all of the code and I think all of the nxflat code is BSD. There may be some references to some of the older XFLAT code which I released as GPL, but I wrote those too and also have the copyright. So I think that the licensing in the buildroot/toolchain/nxflat directory is okay.
The vision that I had for the buildroot tools is that that there would be once place that you could just configure and make and get all of the tools that you need. So it also build the nuttx-eabi or nuttx-elf toolchain (which is a good thing), nxflat, and things like genromfs. It is a pretty handy concept and, I think, still worthy of support.
The advantage of the nuttx-elf toolchain is that it does not include any foreign C library header files or logic; it builds against the NuttX C library and eliminates all of those incompatibilities.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] btashton commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
btashton commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637539547
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
@patacongo I created a fork of the nxflat portion of the buildroot toolchain which will build on its own and contains some minor fixes as well as some CI for make sure that it is functioning. It would be a lot easier to integrate this if that sounds ok to you. For some OS like ubuntu which is the reference build platform you do not even need to build binutils to use the nxflat utilities. I did also document building just the BFD library from binutils as other environments like Cygwin, but also Fedora (which I use) need to have it built from source.
If that is OK with you I can submit the PR to bring that into the CI pipeline.
https://github.com/btashton/nxflat
https://github.com/btashton/nxflat/runs/2649866623#step:6:1
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] a-lunev commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
a-lunev commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637550788
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
> The vision that I had for the buildroot tools is that that there would be once place that you could just configure and make and get all of the tools that you need. So it also build the nuttx-eabi or nuttx-elf toolchain (which is a good thing), nxflat, and things like genromfs. It is a pretty handy concept and, I think, still worthy of support.
>
> The advantage of the nuttx-elf toolchain is that it does not include any foreign C library header files or logic; it builds against the NuttX C library and eliminates all of those incompatibilities.
I also like the idea to have a dedicated nuttx-elf Toolchain (like for nxflat).
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] patacongo commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
patacongo commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637549520
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
> The vision that I had for the buildroot tools is that that there would be once place that you could just configure and make and get all of the tools that you need. So it also build the nuttx-eabi or nuttx-elf toolchain (which is a good thing), nxflat, and things like genromfs. It is a pretty handy concept and, I think, still worthy of support.
I still think we should use githumb.com/nuttx for this purpose. But in order to use that, I think we need to start over. Remove the garbage that has accumulated there and restrict access.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] btashton commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
btashton commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637538751
##########
File path: boards/arm/tiva/lm3s6965-ek/scripts/Make.defs
##########
@@ -41,6 +41,17 @@ ARCHCXXFLAGS = -fno-builtin -fno-exceptions -fcheck-new
ARCHWARNINGS = -Wall -Wstrict-prototypes -Wshadow -Wundef
ARCHWARNINGSXX = -Wall -Wshadow -Wundef
ARCHPICFLAGS = -fpic -msingle-pic-base -mpic-register=r10
+ifeq ($(CONFIG_NXFLAT),y)
+ # mpic-data-is-text-relative flag first time appeared in GCC source code tree on 2016-Jul-12
+ # and it was included in GCC 4.9.4 release on 2016-Aug-03.
+ ARCHCC_HAS_PIC_DATA_IS_TEXT_RELATIVE_FLAG = ${shell $(CC) --target-help 2>&1 | grep -c "mpic-data-is-text-relative"}
+
+ # Turn off -mpic-data-is-text-relative flag to access bss via the GOT
+ # (https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=139630111)
+ ifneq ($(ARCHCC_HAS_PIC_DATA_IS_TEXT_RELATIVE_FLAG),0)
+ ARCHPICFLAGS += -mno-pic-data-is-text-relative
+ endif
+endif
Review comment:
This needs the nxflat tools defined
```diff
diff --git a/boards/arm/tiva/lm3s6965-ek/scripts/Make.defs b/boards/arm/tiva/lm3s6965-ek/scripts/Make.defs
index 0d4d49fa1c..6460e812ff 100644
--- a/boards/arm/tiva/lm3s6965-ek/scripts/Make.defs
+++ b/boards/arm/tiva/lm3s6965-ek/scripts/Make.defs
@@ -60,6 +60,8 @@ CXXPICFLAGS = $(ARCHPICFLAGS) $(CXXFLAGS)
CPPFLAGS := $(ARCHINCLUDES) $(ARCHDEFINES) $(EXTRAFLAGS)
AFLAGS := $(CFLAGS) -D__ASSEMBLY__
+MKNXFLAT = mknxflat
+LDNXFLAT = ldnxflat
NXFLATLDFLAGS1 = -r -d -warn-common
NXFLATLDFLAGS2 = $(NXFLATLDFLAGS1) -T$(TOPDIR)/binfmt/libnxflat/gnu-nxflat-pcrel.ld -no-check-sections
LDNXFLATFLAGS = -e main -s 2048
```
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] btashton commented on a change in pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
btashton commented on a change in pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#discussion_r637539547
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
@patacongo I created a fork of the nxflat portion of the buildroot toolchain which will build on its own and contains some minor fixes as well as some CI for make sure that it is functioning. It would be a lot easier to integrate this if that sounds ok to you. For some OS like ubuntu which is the reference build platform you do not even need to build binutils to use the nxflat utilities. I did also document building just the BFD library from binutils as other environments like Cygwin, but also Fedora (which I use) need to have it built from source.
If that is OK with you I can submit the PR to bring that into the CI pipeline.
##########
File path: tools/ci/testlist/all.dat
##########
@@ -5,6 +5,7 @@
-imxrt1060-evk:libcxxtest
-launchxl-cc1310:nsh
-launchxl-tms57004:nsh
+-lm3s6965-ek:qemu-nxflat
Review comment:
@patacongo I created a fork of the nxflat portion of the buildroot toolchain which will build on its own and contains some minor fixes as well as some CI for make sure that it is functioning. It would be a lot easier to integrate this if that sounds ok to you. For some OS like ubuntu which is the reference build platform you do not even need to build binutils to use the nxflat utilities. I did also document building just the BFD library from binutils as other environments like Cygwin, but also Fedora (which I use) need to have it built from source.
If that is OK with you I can submit the PR to bring that into the CI pipeline.
https://github.com/btashton/nxflat
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #3763: boards: lm3s6965-ek: configuration to test nxflat on QEMU
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #3763:
URL: https://github.com/apache/incubator-nuttx/pull/3763#issuecomment-857872677
Let's merge this PR first. once the toolchain issue get resolved we can remove lm3s6965-ek:qemu-nxflat from the black list.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org