In the past, header file paths were generated by the incdir command
Now they are generated by concatenating environment variables
In this way, when executing makefile, no shell command will be executed,
it will improve the speed of executing makfile
Signed-off-by: yinshengkai <yinshengkai@xiaomi.com>
## Summary
A lot of linker scripts were listed twice, once for unix, once for windows.
This PR cleans up the logic so they're only listed once.
## Impact
Any opportunity to use a single source of truth and reduce lines of code is a win!
## Testing
CI will test all build
This reverts commit 45672c269db13f59bdaa417e564837e8bbb6c8c1.
Because:
* It's very confusing to have cc as LD.
* I don't see what "-nostartfiles -nodefaultlibs" in LDFLAGS are
supposed to do when we use LD directly. It would be simpler to
remove them from our LDFLAGS.
Cleaning during `clean_context` had the issue of remaking everything
when `menuconfig` was issued. That's because `menuconfig` has a
`clean_context` on its way.
Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>
EXTRAFLAGS is already applied to *FLAGS in board's Make.defs (and
it applies to whole build, not just arch-code). EXTRAFLAGS is passed
around each make call to the complete build.
KDEFINE is already added to EXTRAFLAGS in main Makefile so no need
to add it again in arch-level Makefile
Make.dep file should be updated by .config changed after first make.
There are 2 cases affected for this problem:
1) Add source files by config symbol
2) Include header files in #ifdef directive
These 2 cases may not be included in Make.dep and this may prevent the
differential build from working correctly.
So call 'make EXTRAFLAGS=-Wno-cpp' could suppress the warnings with pre-processor
directive #warning in GCC.
Change-Id: Iaa618238924c9969bf91db22117b39e6d2fc9bb6
Signed-off-by: liuhaitao <liuhaitao@xiaomi.com>
This reverts commit 79af7fbf4ef0185fa501e23a900a2ad933f845a7.
Because:
* btashton reported some issues in local builds:
https://github.com/apache/incubator-nuttx/pull/603#issuecomment-602264860
* this might be related to the current CI breakage:
> /bin/sh: 1: /__w/incubator-nuttx/incubator-nuttx/nuttx/tools/mkdeps: not found
Squashed commit of the following:
Author: Gregory Nutt <gnutt@nuttx.org>
Eliminate complaints from tools/nxstyle
Manually fix up kruft left behind by indent.sh
Raw application of tools/indent.sh to all new .c and .h files. Still need fixups.
Convert all C++ style comments to C style; Covery hex constants to lower case.
Convert TABs to spaces
Beginning of code review process. The first small dent.
configs/misoc: Rename os_start to nx_start.
arch/misoc/src/minerva/minerva_exit.c: Make name change to match change made to master.
Author: Ramtin Amin <keytwo@gmail.com>
configs/Kconfig and misoc/nsh/defconfig: Add support for the Verilator board.
Author: Juha Niskanen (Haltian) <juha.niskanen@haltian.com>
arch/arm/src/stm32l4/stm32l4_lse.c: Ports Jussi Kivilinna's recent STM32F7 LSE change to STM32L4.
Author: David Sidrane <david_s5@usa.net>
tools/.gitignore: Ignore new tool binaries.
Author: Ramtin Amin <keytwo@gmail.com>
arch/misoc/src/minerva: Initial merge of minera port.