[v14,0/6] Fix the cross compiling errors
mbox series

Message ID 20180629172738.2121-1-gavin.hu@arm.com
Headers show
Series
  • Fix the cross compiling errors
Related show

Message

Gavin Hu June 29, 2018, 5:27 p.m. UTC
1. Pre-v5 of this patch set(two patches) is to fix the GNU Makefile based cross 
   compiling errors and add a guiding doc for this.
2. v6 add 5 more new patches to cover meson cross fixes
3. v7 Some minor changes to address the comments:
   a) trim the commit message
   b) at the start of the script, test ninja or ninja-build and use it to work out
      the command to work out.
   c) use host gcc and clang for the general v8a only to avoid overkill, and use gcc
      for others.
4. v8 drop one of the new patch as it a similar fix was already upstreamed
5. v9 some minor changes to address the comments, as well some commit message refined.
   a) move if clause out of the loop for a better program structure.
   b) echo the correct ninja command actually used.
6. v10 Fix the whitespace issues.
7. v11 Shorten the command to a single line with all hardcoded values in order to be
   shorter and more readable.
8. v12 Fix the following issues:
   a) return directly instead exiting with an error, if cross gcc is missing, meaning
      the cross compiling tests are skipped.
   b) move the check for cross gcc presence ahead to cover the clang case.
   c) remove host-gcc from the build dir for brevity.
9. v13 Fix the indention issue and merge lines to make the html doc more readable.
10. v14 Fix the following issues:
   a) remove the double hash to get attention for ninja
   b) cross compile only if the cross compiler is installed, just skip and no errors
      output for this if not installed.
   c) fix the errors with the building steps for numactl.

Gavin Hu (4):
  build: fix the meson build warning
  devtools: fix the missing ninja command error
  build: fix the meson cross compile error
  devtools: expand meson cross compiling test coverage

gavin hu (2):
  mk: fix makefile based cross build errors
  doc: add a guide doc for cross compiling from x86

 buildtools/pmdinfogen/Makefile                     |   2 +-
 config/arm/arm64_thunderx_linuxapp_gcc             |   1 +
 config/meson.build                                 |   3 +-
 devtools/test-meson-builds.sh                      |  33 ++++--
 .../linux_gsg/cross_build_dpdk_for_arm64.rst       | 132 +++++++++++++++++++++
 doc/guides/linux_gsg/index.rst                     |   1 +
 drivers/meson.build                                |   3 +
 examples/meson.build                               |   4 +
 lib/meson.build                                    |   4 +
 mk/toolchain/gcc/rte.toolchain-compat.mk           |   5 +
 mk/toolchain/gcc/rte.vars.mk                       |   9 ++
 test/test/meson.build                              |   7 +-
 12 files changed, 191 insertions(+), 13 deletions(-)
 create mode 100644 doc/guides/linux_gsg/cross_build_dpdk_for_arm64.rst

Comments

Thomas Monjalon July 11, 2018, 7:16 p.m. UTC | #1
29/06/2018 19:27, Gavin Hu:
> Gavin Hu (4):
>   build: fix the meson build warning
>   devtools: fix the missing ninja command error
>   build: fix the meson cross compile error
>   devtools: expand meson cross compiling test coverage
> 
> gavin hu (2):
>   mk: fix makefile based cross build errors
>   doc: add a guide doc for cross compiling from x86

Applied with minor changes (names and error message), thanks