[riot-notifications] [RIOT-OS/RIOT] sys/Makefile.include: check for newlib_nano instead of USE_NANO_SPECS (#11883)

Gaëtan Harter notifications at github.com
Mon Jul 22 17:06:34 CEST 2019


### Contribution description

Check for the usage of `newlib_nano` module instead of the
`USE_NANO_SPECS` variable.

This allows also benefiting from the `printf_float` and `scanf_float`
behaviour on `arm7` and `riscv` cpus.


### Testing procedure

Verify it is actually handled correctly on `arm7` and `riscv` boards.


Run the `tests/unittests tests-scanf_float` on `arm7` and `riscv` boards.

```
BOARD=msba2 make --no-print-directory -C tests/unittests/ tests-scanf_float flash-only test

Booting (hardware reset)...

Reset CPU (into user code)
Programming done.
ssh -t vaduz.imp.fu-berlin.de 'source /srv/ilab-builds/workspace/workspace.rc && BOARD=msba2 QUIET=0 make --no-print-directory -C /srv/ilab-builds/boards term'
/srv/ilab-builds/boards/RIOT/dist/tools/pyterm/pyterm -tg -p "/dev/riot/ttyMSBA2"
Twisted not available, please install it if you want to use pyterm's JSON capabilities
2019-07-22 16:37:43,181 - INFO # Connect to serial port /dev/riot/ttyMSBA2
Welcome to pyterm!
Type '/exit' to exit.
2019-07-22 16:37:44,187 - INFO # main(): This is RIOT! (Version: 2019.10-devel-104-g294fb-HEAD)
2019-07-22 16:37:44,187 - INFO # ......
2019-07-22 16:37:44,188 - INFO # OK (6 tests)
```

Run other relevant tests:

* msba2
  * [ ] tests/unittests tests-scanf_float
* hifive1
  * [ ]

### Issues/PRs references

General fix for https://github.com/RIOT-OS/RIOT/pull/11882

`USE_NANO_SPECS` was added in https://github.com/RIOT-OS/RIOT/pull/5094
Not sure why it was not set with `newlib_nano` at the end.

You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/11883

-- Commit Summary --

  * sys/Makefile.include: check for newlib_nano instead of USE_NANO_SPECS
  * cortexm.inc.mk: remove deprecated USE_NANO_SPECS

-- File Changes --

    M makefiles/arch/cortexm.inc.mk (2)
    M sys/Makefile.include (4)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/11883.patch
https://github.com/RIOT-OS/RIOT/pull/11883.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11883
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190722/34e08dd4/attachment.htm>


More information about the notifications mailing list