[riot-notifications] [RIOT-OS/RIOT] tests/cortexm_common_ldscript: update code section for kinetis (#11588)

Francisco notifications at github.com
Mon May 27 16:40:41 CEST 2019


fjmolinas commented on this pull request.



> @@ -86,10 +86,15 @@ OFFSETS_TESTS = 0x1000 0x2000
 tests-offsets: $(OFFSETS_TESTS:%=test-offset_%)
 
 .PHONY: test-offset_%
+# Match the 'code' section. It is usually '.text' but is '.vector' on `kinetis`.
+# * [ 1] .text             PROGBITS        08001000 001000 001ef8 ...
+# * [ 1] .vector           PROGBITS        00001000 001000 000400 ...
+# So match with the section [ 1] being `PROGBITS`. Adapt if new cases appear.
+CODE_SECTION = \[ 1\] \.[a-zA-Z_.]* *PROGBITS

`[a-zA-Z_.]` Do sections have "." in the middle of their names? you are already matching the on leading the section name.

-- 
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/11588#pullrequestreview-242308306
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190527/4b2a487a/attachment-0001.html>


More information about the notifications mailing list