[riot-commits] [RIOT-OS/RIOT] d7834e: First apply board includes, then det. CC caps

Oleg Hahm gello at gmx.de
Tue Feb 18 13:45:51 CET 2014


  Branch: refs/heads/master
  Home:   https://github.com/RIOT-OS/RIOT
  Commit: d7834ecbf86363100d0ef8b006b5e5dea2cc4eb7
      https://github.com/RIOT-OS/RIOT/commit/d7834ecbf86363100d0ef8b006b5e5dea2cc4eb7
  Author: René Kijewski <rene.kijewski at fu-berlin.de>
  Date:   2014-02-16 (Sun, 16 Feb 2014)

  Changed paths:
    M Makefile.include

  Log Message:
  -----------
  First apply board includes, then det. CC caps

In #664 I added a test that determines if the supplied compiler
understands the `-fno-delete-null-pointer-checks` flag. The problem is
that the `$(CC)` supplied on command line or in the application's
Makefile is used, but not the one the `$(BOARD)`'s Makefile sets.

That problem was overlooked as all the boards use GCC, and GCC happens
to know the flag. But if some future board does not use GCC, then the
wrong order of the checks could pose a problem.


  Commit: c1bcfb3516d36618d72e84d3c83988fc4014c1bf
      https://github.com/RIOT-OS/RIOT/commit/c1bcfb3516d36618d72e84d3c83988fc4014c1bf
  Author: Oleg Hahm <gello at gmx.de>
  Date:   2014-02-18 (Tue, 18 Feb 2014)

  Changed paths:
    M Makefile.include

  Log Message:
  -----------
  Merge pull request #722 from Kijewski/fixup-664

First apply board includes, then det. CC caps


Compare: https://github.com/RIOT-OS/RIOT/compare/b42496475a9a...c1bcfb3516d3


More information about the commits mailing list