RTEMS / Tools / RTEMS Tools¶
Go to Issues or Merge Requests
Issues Summary
Merge Requests Summary
Issues¶
9 - rtems-tools Do Not Compile on Cygwin¶
Id |
9 |
State |
closed |
Type |
ISSUE |
Author |
Joel Sherrill |
Assignee(s) |
Joel Sherrill |
Closed by |
Amar Takhar |
Created |
2024-06-27T19:05:28.339Z |
Closed |
2024-06-27T19:15:22.337Z |
Updated |
2024-06-27T20:03:05.844Z |
Milestone |
6.1 |
Labels |
tool::rtems-tools |
Link |
|
Merges |
1 |
Summary¶
rtems-tools do not build on Cygwin. Multiple files fail to compile and report this:
[ 15/270] Compiling rtemstoolkit/elftoolchain/libelf/gelf_cap.c
../rtemstoolkit/elftoolchain/libelf/gelf_cap.c:36:13: error: expected declaration specifiers or \ |md_0_8216|\ ...\ |md_0_8217|\ before string constant
36 | ELFTC_VCSID("$Id: gelf_cap.c 3977 2022-05-01 06:45:34Z jkoshy $");
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Waf: Leaving directory `/home/jrs007/rtems-work/rtems-tools/build'
Steps to reproduce¶
Simple to reproduce – just build on Cygwin.
Author: Joel Sherrill
2024-06-27T19:05:28.458Z
assigned to @joel
Author: Joel Sherrill
2024-06-27T19:14:34.527Z
mentioned in merge request !15
Author: Amar Takhar
2024-06-27T19:16:32.874Z
Should we upstream that fix? Though libelf makes no claims to build on cygwin that seems like a very simple change.
Author: Amar Takhar
2024-06-27T19:18:05.806Z
I don’t see an issue open for it:
Author: Amar Takhar
2024-06-27T20:03:05.820Z
Joel has submitted the patch here:
8 - Python tests GIT checks¶
Id |
8 |
State |
closed |
Type |
ISSUE |
Author |
Chris Johns |
Assignee(s) |
Amar Takhar |
Closed by |
Chris Johns |
Created |
2024-06-19T00:37:16.952Z |
Closed |
2024-06-19T22:54:39.236Z |
Updated |
2024-06-19T22:54:39.274Z |
Milestone |
6.1 |
Labels |
tool::rtems-tools |
Link |
|
Merges |
1 |
Summary¶
The python tests in RTEMS Tools check the state of the git repo and raise an error if there are untracked files or changes not staged and committed. This complicates using the tests for development because an error is raise.
A release is a tarball of files and there is no GIT repo associated to the tarball. The tests should pass for a release.
A release is created by using git to export the repo and adding a VERSION file to top. The version.py code knows to check and load the VERSION file set the various version values based on the contents.
Author: Chris Johns
2024-06-19T00:37:17.039Z
assigned to @amar
Author: Chris Johns
2024-06-19T00:39:20.573Z
changed the description
Author: Amar Takhar
2024-06-19T02:48:59.497Z
mentioned in merge request !11
7 - README does not document how to run the tests¶
Id |
7 |
State |
closed |
Type |
ISSUE |
Author |
Chris Johns |
Assignee(s) |
Amar Takhar |
Closed by |
Chris Johns |
Created |
2024-06-18T23:06:56.952Z |
Closed |
2024-06-19T00:22:58.991Z |
Updated |
2024-06-19T00:22:59.056Z |
Milestone |
6.1 |
Labels |
tool::rtems-tools |
Link |
|
Merges |
1 |
The README does not explain how to run the new tests.
If you wish to move the file to README.md please do so. Thanks.
Author: Chris Johns
2024-06-18T23:06:57.049Z
assigned to @amar
Author: Amar Takhar
2024-06-18T23:19:48.375Z
mentioned in merge request !9
6 - Unsupported DWARF version¶
Id |
6 |
State |
closed |
Type |
ISSUE |
Author |
Chris Johns |
Assignee(s) |
Chris Johns |
Closed by |
Amar Takhar |
Created |
2024-06-14T06:44:46.080Z |
Closed |
2024-06-15T02:00:15.661Z |
Updated |
2024-06-15T02:10:45.307Z |
Milestone |
6.1 |
Link |
|
Merges |
1 |
Summary¶
RTEMS Tools does not support the version of DWARF GCC is now gnerating
Steps to reproduce¶
$ rtems-exeinfo -D build/arm/xilinx_zynq_a9_qemu/testsuites/samples/hello.exe
RTEMS Executable Info 6.423d98d2058a
/opt/work/rtems/6/bin/rtems-exeinfo -D build/arm/xilinx_zynq_a9_qemu/testsuites/samples/hello.exe
May take a while ...
exe: build/arm/xilinx_zynq_a9_qemu/testsuites/samples/hello.exe
Compilation:
Producers: 0
Common flags: 0
|
DWARF Data:
error: dwarf:compilation_unit::sibling: Unsupported version [_dwarf_info_load(229)]
Resolution¶
Update elftoolchain to the latest in SVN on ELF Tool Chain on Sourceforge.
Author: Chris Johns
2024-06-14T06:44:46.155Z
assigned to @chris
Author: Chris Johns
2024-06-14T06:46:15.460Z
changed the description
Author: Chris Johns
2024-06-14T08:29:03.910Z
changed title from Unsupported D{-RAW-}F version to Unsupported D{+WAR+}F version
Author: Chris Johns
2024-06-14T08:32:14.273Z
mentioned in merge request !5
Author: Chris Johns
2024-06-15T02:06:57.461Z
changed the description
Author: Chris Johns
2024-06-15T02:11:19.941Z
$ rtems-exeinfo `find build -name hello.exe` RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/x86_64/amd64efi/testsuites/samples/hello.exe exe: build/x86_64/amd64efi/testsuites/samples/hello.exe Compilation: Producers: 2 | GNU AS 2.41: 5 objects | GNU C17 13.2.0 20230727 (RTEMS 6, RSB f1a36b851161480ff0a0a1addb22fb44e102d339, Newlib 3cacedb): 282 objects Common flags: 2 | -ftls-model=local-exec -march=x86-64$ rtems-exeinfo build/aarch64/xilinx_zynqmp_lp64_zu3eg/testsuites/samples/hello.exe RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/aarch64/xilinx_zynqmp_lp64_zu3eg/testsuites/samples/hello.exe exe: build/aarch64/xilinx_zynqmp_lp64_zu3eg/testsuites/samples/hello.exe Compilation: Producers: 2 | GNU AS 2.42: 11 objects | GNU C17 13.3.0 20240521 (RTEMS 6, RSB 423d98d2058a7dac0921f9a13113b024696236b9, Newlib 730703b): 266 objects Common flags: 7 | -ftls-model=local-exec -mno-outline-atomics -mcpu=cortex-a53 -mfix-cortex-a53-835769 -mfix-cortex-a53-843419 -mlittle-endian -mabi=lp64$ rtems-exeinfo `find build -name hello.exe` RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/powerpc/mvme2700/testsuites/samples/hello.exe exe: build/powerpc/mvme2700/testsuites/samples/hello.exe Compilation: Producers: 3 | GNU AS 2.42: 6 objects | GNU C17 13.3.0 20240521 (RTEMS 6, RSB 423d98d2058a7dac0921f9a13113b024696236b9, Newlib 730703b): 354 objects | GNU AS 2.41: 1 objects Common flags: 2 | -ftls-model=local-exec -mcpu=604$ rtems-exeinfo `find build -name hello.exe` RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/sparc/erc32/testsuites/samples/hello.exe exe: build/sparc/erc32/testsuites/samples/hello.exe Compilation: Producers: 3 | GNU AS 2.42: 5 objects | GNU C17 13.3.0 20240521 (RTEMS 6, RSB 8037d9ad83d74b5d5eee6ac13ead3580772609cd, Newlib 1ed1516): 272 objects | GNU AS 2.41: 3 objects Common flags: 1 | -ftls-model=local-exec$ rtems-exeinfo `find build -name hello.exe` RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/riscv/rv32imac/testsuites/samples/hello.exe exe: build/riscv/rv32imac/testsuites/samples/hello.exe Compilation: Producers: 2 | GNU AS 2.42: 5 objects | GNU C17 13.3.0 20240521 (RTEMS 6, RSB 8037d9ad83d74b5d5eee6ac13ead3580772609cd, Newlib 1ed1516): 291 objects Common flags: 4 | -ftls-model=local-exec -mabi=ilp32 -misa-spec=20191213 -march=rv32imac$ rtems-exeinfo `find build -name hello.exe` RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/riscv/rv64imac/testsuites/samples/hello.exe exe: build/riscv/rv64imac/testsuites/samples/hello.exe Compilation: Producers: 2 | GNU AS 2.42: 5 objects | GNU C17 13.3.0 20240521 (RTEMS 6, RSB 8037d9ad83d74b5d5eee6ac13ead3580772609cd, Newlib 1ed1516): 287 objects Common flags: 5 | -ftls-model=local-exec -mabi=lp64 -mcmodel=medany -misa-spec=20191213 -march=rv64imac
12 - rtl repositories no longer availab.e (opened)¶
Id |
12 |
State |
opened |
Type |
ISSUE |
Author |
Amar Takhar |
Assignee(s) |
Chris Johns |
Created |
2024-07-27T01:20:06.146Z |
Updated |
2024-07-27T01:20:06.146Z |
Milestone |
6.1 |
Labels |
doc |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/issues/12 |
Merges |
0 |
Summary¶
The repositories from git.rtems.org mentioned here are no longer available:
Steps to reproduce¶
Author: Amar Takhar
2024-07-27T01:20:06.254Z
assigned to @chris
Author: Amar Takhar
2024-07-27T01:20:23.853Z
Assigned to @chris since the repositories existed in his personal space.
10 - Coverity Reported a Leak in libdwarf (opened)¶
Id |
10 |
State |
opened |
Type |
ISSUE |
Author |
Joel Sherrill |
Assignee(s) |
Chris Johns |
Created |
2024-07-08T17:02:36.024Z |
Updated |
2024-07-09T07:02:00.584Z |
Milestone |
6.1 |
Labels |
tool::rtems-tools |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/issues/10 |
Merges |
0 |
Summary¶
Coverity Scan reports a resource leak.
** CID 1595466: Resource leaks (RESOURCE_LEAK) /rtemstoolkit/elftoolchain/libdwarf/dwarf_abbrev.c: 51 in dwarf_get_abbrev()
Details¶
This is the detailed information provided.
*** CID 1595466: Resource leaks (RESOURCE_LEAK)
/rtemstoolkit/elftoolchain/libdwarf/dwarf_abbrev.c: 51 in dwarf_get_abbrev()
45 ret = _dwarf_abbrev_parse(dbg, NULL, &offset, &ab, error);
46 if (ret != DW_DLE_NONE) {
47 if (ret == DW_DLE_NO_ENTRY) {
48 DWARF_SET_ERROR(dbg, error, DW_DLE_NO_ENTRY);
49 return (DW_DLV_NO_ENTRY);
50 } else
CID 1595466: Resource leaks (RESOURCE_LEAK)
Variable "ab" going out of scope leaks the storage it points to.
51 return (DW_DLV_ERROR);
52 }
53
54 *return_abbrev = ab;
55 *length = ab->ab_length;
56 *attr_count = ab->ab_atnum;
Author: Joel Sherrill
2024-07-08T17:02:36.185Z
assigned to @chris
Author: Amar Takhar
2024-07-09T07:02:00.720Z
changed the description
1 - rtems-ld created wrong RAP file on arm (opened)¶
Id |
1 |
State |
opened |
Type |
ISSUE |
Author |
Trac Migrate |
Assignee(s) |
xiaojun zheng |
Created |
2023-09-11T02:47:17.000Z |
Updated |
2024-05-10T15:48:33.371Z |
Milestone |
6.1 |
Labels |
priority::normal, tickettype::defect, tool |
Link |
|
Merges |
0 |
Original author: jameszxj
I use the latest rtems-tools to generate RAP file, I found the object file become too small(elf,2184bytes==>rap,736bytes) and the file loaded failed, error message like this:
load app.out error:offset past end of file: offset=736 size=736 error.
I reverted the rtl-rap.cpp commit (rtems/tools/rtems-tools@0ad4aaafc20afcb5aacb7a82b0b3a8150b638975).
At Chris’s suggestion, I found get_relocations(s) return all the count of relocation and write to rap file, but actual count is less, because ignore the reloc with section=0.
I tried to make the length fit the relocs write to rap file, the file can not be loaded, it crashed. So the relocs(reloc.symsect == 0) can not be ignored, at least on arm.
The command line I used to create RAP file is:
rtems-ld -n -e appmain -s -O rap --base rtems.elf app.elf -o app.out
Author: Chris Johns
2024-05-09T01:21:15.777Z
assigned to @zhengxiaojun
Author: xiaojun zheng
2024-05-03T12:06:06.253Z
mentioned in merge request rtems/tools/rtems-tools!1
Author: Chris Johns
2024-05-06T05:16:24.159Z
changed the description
Author: Chris Johns
2024-05-06T05:17:15.675Z
Could you please attach the
appsource that causes the issue?
Author: xiaojun zheng
2024-05-09T01:13:09.973Z
Just a printf in app.
#include <stdio.h> #include <stdlib.h> int appmain() { printf("app start\\n"); return 0; }
Author: Chris Johns
2024-05-09T01:13:09.945Z
Thanks. Do you have a script to build this code and then run the RTEMS linker?
Author: xiaojun zheng
2024-05-10T00:46:44.266Z
arm-rtems6-gcc -MMD -MP -std=gnu11 -O2 -march=armv7-a -mthumb -mlong-calls -Wall -Wformat-truncation=0 -Wa,-mimplicit-it=always --pipe -B/opt/rtems/6.1/arm-rtems6/imx6ull/lib -qrtems -Wno-pointer-sign -fno-strict-aliasing -c src/PT_Init.c -o build/PT_Init.c.o arm-rtems6-gcc -r -nostdlib -march=armv7-a -mthumb -mlong-calls -Wl,-dc --pipe -B/opt/rtems/6.1/arm-rtems6/imx6ull/lib -qrtems -o build/app.elf build/PT_Init.c.o arm-rtems6-strip -d build/app.elf rtems-ld -n -e appmain -s -O rap --base rtems.elf build/app.elf -o build/app.out
Author: xiaojun zheng
2024-05-10T01:11:40.313Z
The Makefile I used to build. Makefile
Author: Amar Takhar
2024-05-09T01:11:23.405Z
assigned to @zhengxiaojun and unassigned @tracmigrate
Author: Chris Johns
2024-05-09T01:21:16.388Z
moved from rtems/rtos/rtems#4952
Author: Chris Johns
2024-05-09T01:22:50.087Z
marked this issue as related to rtems/rtos/rtems#4952
Author: xiaojun zheng
2024-05-09T05:29:54.775Z
changed the description
Author: xiaojun zheng
2024-05-10T15:48:33.287Z
I compared the output information using the rtems-ld’s verbose option. when ignored reloc.symsect == 0, there is no “2/2… 3/3…” messages. the relocation is about extern symbol “puts”.
Outputs are as follows:
rap:relocation: section:.text relocs=4 rela=no relocs=4 sec.offset=0 sec.size=24 sec.align=4 build/arm-rtems6-imx6ull/app.elf 0/ 0: reloc.info=0x2f reloc.offset=2 reloc.addend=0 reloc.symtype=0 reloc.symsect=3 (.rodata.str1.4) reloc.symvalue=0 reloc.symbinding=0 0/ 0: rsym: sect=.const rap_symsect=1 sec.offset=0 sec.osecs=0 addend=0 0/ 0: reloc: info=0x12f offset=2 addend=0 reloc.info=0x2f reloc.offset=2 reloc.symtype=0 1/ 1: reloc.info=0x30 reloc.offset=6 reloc.addend=0 reloc.symtype=0 reloc.symsect=3 (.rodata.str1.4) reloc.symvalue=0 reloc.symbinding=0 1/ 1: rsym: sect=.const rap_symsect=1 sec.offset=0 sec.osecs=0 addend=0 1/ 1: reloc: info=0x130 offset=6 addend=0 reloc.info=0x30 reloc.offset=6 reloc.symtype=0 2/ 2: reloc.info=0x2f reloc.offset=10 reloc.addend=0 reloc.symtype=0 reloc.symsect=0 () reloc.symvalue=0 reloc.symbinding=1 2/ 2: reloc: info=0x8000042f offset=10 symname=puts (appended) reloc.info=0x2f reloc.offset=10 reloc.symtype=0 3/ 3: reloc.info=0x30 reloc.offset=14 reloc.addend=0 reloc.symtype=0 reloc.symsect=0 () reloc.symvalue=0 reloc.symbinding=1 3/ 3: reloc: info=0x80000430 offset=14 symname=puts (appended) reloc.info=0x30 reloc.offset=14 reloc.symtype=0
Merge Requests¶
6 - Add test suite for rtemstoolkit.¶
Id |
6 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-06-18T03:42:52.289Z |
Assignee(s) |
Amar Takhar |
Created |
2024-06-17T03:47:15.380Z |
Updated |
2024-07-30T02:51:05.653Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/6 |
Commit |
Summary¶
Add test suite for rtemstoolkit.
This was written in 2020.
To run do:
./waf configure¶
pytest -v¶
Author: Amar Takhar
2024-06-17T03:47:15.530Z
assigned to @amar
Author: Chris Johns
2024-06-18T03:42:42.996Z
This looks great and thanks for doing it. I some questions:
Would
rtemstoolkit/testsbe a better place for the code? It would be nice to adddoctestsupport to the C++ code at some point.There is test code at the end of each module which could be migrated over and deleted. It is run with
./waf test. Will these tests take over from those tests?
Author: Amar Takhar
2024-06-18T03:42:42.996Z
I was planning to eventually write tests for other components of
rtems-toolseventually which is why it’s toplevel. Also other directories that have required files likeconfig/it made sense to have it toplevel. Though nothing stops us from having lots of differenttestdirectories the tools aren’t all insular.Oh yes sorry I completely forgot to delete those! Yes this replaces all of them should I delete it in this commit or should I do it in another? I think it may be cleaner to do it in another so there is at least one commit where they live side-by-side makes for easier testing if required.
In my defense I wrote this 4 years ago and I just copied the test files but forgot there were source changes.
Author: Chris Johns
2024-06-18T03:42:42.996Z
The
rtemstoolkitis isolated sotestsis fine but should it betests/rtemstoolkitor eventestsand that referencesrtemstoolkit/tests. This way we can add tests to things that are not related to thertemstoolkit?The current tests should be migrated before being deleted or we will forget what those tests did.
Author: Amar Takhar
2024-06-18T03:42:42.996Z
I have renamed them to
test_rt_*.. subdirs won’t work because it messes with the python import it’s easier to just have them all in the same directory anyway.I think all the tests have been migrated but if any haven’t been in the next commit I’ll leave them behind and then migrate the rest. I’m pretty sure I did all of them unless more have been added.
Author: Amar Takhar
2024-06-18T03:42:42.996Z
Hold on something screwed up in my merge.
Author: Amar Takhar
2024-06-18T03:42:42.996Z
Interesting it seems the file rename didn’t carry through on my squash so it just deleted the files that’s not good.
Author: Amar Takhar
2024-06-18T03:42:42.996Z
haha I forgot to add the files after I renamed them so they got deleted I didn’t notice when I squashed so the files I renamed got left behind while the moved files got deleted. I’ve never done that before, funny.
Author: Chris Johns
2024-06-18T03:42:42.996Z
Yes I agree a sub-directory will not work. This is good.
Author: Amar Takhar
2024-06-17T05:51:48.945Z
added 1 commit
544d8970 - Add test suite for rtemstoolkit.
Author: Amar Takhar
2024-06-17T06:00:39.499Z
added 1 commit
d7b53bd6 - Add test suite for rtemstoolkit.
Author: Amar Takhar
2024-06-17T17:24:45.914Z
added 1 commit
4d69f1a7 - Add test suite for rtemstoolkit.
Author: Amar Takhar
2024-06-17T17:25:31.460Z
added 1 commit
0889179b - Add test suite for rtemstoolkit.
Author: Amar Takhar
2024-06-17T17:26:49.552Z
added 1 commit
fdeede90 - Add test suite for rtemstoolkit.
Author: Amar Takhar
2024-06-17T17:43:29.147Z
mentioned in merge request !7
Author: Amar Takhar
2024-06-17T17:43:41.715Z
requested review from @amar
Author: Gedare Bloom
2024-06-18T03:23:48.297Z
make it more clear the scope of what was copied, and from exactly where it comes from, so someone may update it easier.
Author: Amar Takhar
2024-06-18T03:23:48.297Z
This was already there I have no idea where it was taken from, either. I’ll ask @chris
Author: Amar Takhar
2024-06-18T03:23:48.297Z
Found the location and linked.
Author: Gedare Bloom
2024-06-18T03:22:24.780Z
suggest making the reason for commenting out upstream code more clear
Author: Amar Takhar
2024-06-18T03:22:24.780Z
This isn’t commented out? It’s in the commit message that this was upgraded the implementation is from upstream.
Author: Amar Takhar
2024-06-18T03:22:24.780Z
Sorry I see what you mean now no idea why that is commented. I wish GitLab had an easier to see colour for commented code.
Author: Amar Takhar
2024-06-18T03:22:24.780Z
I found it fixed the comment and linked the source.
Author: Gedare Bloom
2024-06-18T01:17:12.437Z
should be 2 blank lines. please pass through yapf or similar, I think we prefer PEP-8 unless there is another prevailing style to use. https://docs.rtems.org/branches/master/eng/python-devel.html#python-code-formatting
Author: Amar Takhar
2024-06-18T01:17:12.437Z
Fixed.
Author: Gedare Bloom
2024-06-18T01:19:56.314Z
?
Author: Amar Takhar
2024-06-18T01:16:41.867Z
changed this line in version 7 of the diff
Author: Amar Takhar
2024-06-18T01:19:56.314Z
Fixed
Author: Gedare Bloom
2024-06-18T01:17:19.019Z
IIRC, 1 line separates methods
Author: Amar Takhar
2024-06-18T01:16:42.177Z
changed this line in version 7 of the diff
Author: Amar Takhar
2024-06-18T01:17:19.019Z
Fixed
Author: Gedare Bloom
2024-06-18T01:17:23.723Z
line length, but I think
yapfshould fix it.
Author: Amar Takhar
2024-06-18T01:16:42.478Z
changed this line in version 7 of the diff
Author: Amar Takhar
2024-06-18T01:17:23.723Z
Fixed
Author: Gedare Bloom
2024-06-18T01:19:31.002Z
?
Author: Amar Takhar
2024-06-18T01:19:31.002Z
Fixed
Author: Gedare Bloom
2024-06-18T03:40:48.484Z
I think we want to explicitly
return None
Author: Amar Takhar
2024-06-18T03:40:48.484Z
The default behavior or Python is to return None. A return statement is useful if you want to let others know you didn’t forget anything but it will always be None.
Author: Chris Johns
2024-06-18T03:40:48.484Z
We will move to typing and return type interfaces once 3.10 (I think) and later is widely adopted. Before that version typing in python is not great to use. I have this in the release notes generator and after working on 3.12 and then testing on FreeBSD 3.9 I had to unwind a bunch of things.
Author: Gedare Bloom
2024-06-18T01:17:40.930Z
I would rather see proper path building
Author: Amar Takhar
2024-06-18T01:16:43.423Z
changed this line in version 7 of the diff
Author: Amar Takhar
2024-06-18T01:17:40.930Z
Fixed
Author: Gedare Bloom
2024-06-18T01:18:08.313Z
path construction
Author: Amar Takhar
2024-06-18T01:18:08.313Z
Fixed
Author: Gedare Bloom
2024-06-18T01:17:57.511Z
?
Author: Amar Takhar
2024-06-18T01:17:57.511Z
Fixed
Author: Gedare Bloom
2024-06-18T01:17:55.145Z
?
Author: Amar Takhar
2024-06-18T01:16:45.217Z
changed this line in version 7 of the diff
Author: Amar Takhar
2024-06-18T01:17:55.145Z
Fixed
Author: Gedare Bloom
2024-06-18T01:17:53.711Z
excessive blank lines
Author: Amar Takhar
2024-06-18T01:16:45.646Z
changed this line in version 7 of the diff
Author: Amar Takhar
2024-06-18T01:17:53.711Z
Fixed
Author: Gedare Bloom
2024-06-18T01:18:44.609Z
not clear to me why this is a string.
Author: Amar Takhar
2024-06-18T01:16:46.078Z
changed this line in version 7 of the diff
Author: Amar Takhar
2024-06-18T01:18:44.609Z
This was meant to be uncommented when I skipped the entire file.
Author: Gedare Bloom
2024-06-18T01:18:56.696Z
?
Author: Amar Takhar
2024-06-18T01:16:46.511Z
changed this line in version 7 of the diff
Author: Amar Takhar
2024-06-18T01:18:56.696Z
Fixed
Author: Gedare Bloom
2024-06-17T20:05:48.943Z
requested changes
Author: Amar Takhar
2024-06-18T01:16:46.676Z
added 1 commit
53383500 - Add test suite for rtemstoolkit.
Author: Amar Takhar
2024-06-18T01:20:10.653Z
removed review request for @amar
Author: Amar Takhar
2024-06-18T03:22:07.417Z
added 1 commit
3a639da1 - Add test suite for rtemstoolkit.
Author: Chris Johns
2024-07-30T02:51:05.608Z
Could
pytest -vbe linked to./waf test? This way you could do./waf configure test?I like
wafhandling the interface so we can add C++ tests and not break or add anything new for users.
Author: Amar Takhar
2024-07-30T02:51:05.608Z
pytest doesn’t like being run inside anything. I wouldn’t recommend running tests inside of another program like that. I think it’s OK as with python it’s really common for the test command to be something separate.
Author: Chris Johns
2024-06-18T03:40:48.504Z
resolved all threads
Author: Chris Johns
2024-06-18T03:42:33.364Z
approved this merge request
Author: Chris Johns
2024-06-18T03:42:43.017Z
resolved all threads
Author: Chris Johns
2024-07-30T02:51:05.633Z
resolved all threads
22 - Update to GitLab URLs.¶
Id |
22 |
State |
merged |
Merged by |
Kinsey Moore |
Merged at |
2024-07-27T21:31:45.003Z |
Assignee(s) |
Amar Takhar |
Created |
2024-07-27T01:22:24.852Z |
Updated |
2024-07-27T21:31:46.209Z |
Milestone |
6.1 |
Labels |
doc |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/22 |
Commit |
Summary¶
Author: Amar Takhar
2024-07-27T01:22:25.002Z
assigned to @amar
Author: Kinsey Moore
2024-07-27T21:31:34.509Z
approved this merge request
Author: Kinsey Moore
2024-07-27T21:31:39.516Z
added 2 commits
38a3e3a7 - 1 commit from branch
rtems/tools:main478c0727 - Update to GitLab URLs.
23 - Direct users to submit an MR¶
Id |
23 |
State |
merged |
Merged by |
Amar Takhar |
Merged at |
2024-07-27T21:29:45.616Z |
Assignee(s) |
Amar Takhar |
Created |
2024-07-27T20:15:20.341Z |
Updated |
2024-07-27T21:29:46.855Z |
Milestone |
6.1 |
Labels |
doc |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/23 |
Commit |
Summary¶
Direct users to submit an MR
Using the devel list is no longer the preferred method.
Author: Amar Takhar
2024-07-27T20:15:20.492Z
assigned to @amar
Author: Joel Sherrill
2024-07-27T20:52:27.539Z
approved this merge request
Author: Amar Takhar
2024-07-27T21:28:44.994Z
huh it says 1 of 1 so I’m not sure why this is blocked.
Author: Amar Takhar
2024-07-27T21:29:43.175Z
I’ll have to fix this later Joel wasn’t in the docs group but it’s not supposed to require the docs group if the general maintainer approved.
13 - Add API documentation using pdoc¶
Id |
13 |
State |
merged |
Merged by |
Gedare Bloom |
Merged at |
2024-07-10T05:22:17.775Z |
Assignee(s) |
Amar Takhar |
Created |
2024-06-20T21:06:17.065Z |
Updated |
2024-07-10T05:22:19.298Z |
Milestone |
6.1 |
Labels |
doc |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/13 |
Commit |
Summary¶
Add API documentation using pdoc
A lot of documentation is missing but it’s a start. If we need more complicated features in the future we can move to Sphinx but pdoc is great and a very small dependency.
Remove unused import
This also checks for the existence of a ‘python’ binary. It assumes it’s Python 2.7 if found and uses it. If it’s not found it skips building the documentation instead of erroring out.
Once we drop Python 2.7 support we can set this up properly using Wafs internal Python detection. It’s non-trivial to pickup and support two versions at the same time.
Author: Amar Takhar
2024-06-20T21:06:17.219Z
assigned to @amar
Author: Amar Takhar
2024-06-21T23:41:39.776Z
requested review from @amar
Author: Amar Takhar
2024-06-21T23:44:58.830Z
Testing comments vs review
Author: Amar Takhar
2024-06-21T23:45:00.289Z
This is a test to see if the MR is blocked due to a review
Author: Amar Takhar
2024-06-21T23:43:36.893Z
Using the finish review section.
Author: Amar Takhar
2024-06-21T23:45:00.305Z
resolved all threads
Author: Amar Takhar
2024-06-21T23:45:15.264Z
removed review request for @amar
Author: Amar Takhar
2024-06-23T06:29:32.909Z
Testing MR comments.
Author: Gedare Bloom
2024-07-02T04:43:02.949Z
Please edit the commit message (and MR title) to make the scope more clear (What API? I think it is
rtemstoolkit:)Is this something for using with the 6.1 release?
Author: Amar Takhar
2024-07-02T04:43:02.949Z
It’s not strictly limited to rtemstoolkit no but the only directory it scans right now is rtemstoolkit so I left it off. I wanted it in the repo first before I modified it to include other areas.
I don’t see why it can’t be in 6.1 it’s an innocuous change it can also be useful for anyone wanting to use rtemstoolkit as it does make useful documentation even though it is sparse.
Author: Chris Johns
2024-07-02T04:43:02.949Z
I think @gedare is discussing this commit which is why it would be good to have in the history which part is being covers?
Is there an example of the generated docs I can look at? Yes I am being lazy not running the change :smile:
Author: Amar Takhar
2024-07-02T04:43:02.949Z
I can add it but do we really split hairs on things like this? This commit adds the framework which is in the docs/ directory which coincidentally adds rtemstoolkit first but it’s mostly for the framework. I left it off because it’s not really fully part of either.. I can add it but I’m starting to feel these rules are kind of fuzzy
There is no example pdoc is a single dependency just install it,
waf configure build, that’s it! If it finds it then it will install the docs are inbuild/
Author: Amar Takhar
2024-07-02T04:43:02.949Z
I added a
doc:identifier.
Author: Amar Takhar
2024-07-02T04:43:02.949Z
You can look at some example docs online look under submodules on the left:
Author: Chris Johns
2024-07-02T04:43:02.949Z
We are only attempting to understand what is happening and nothing more.
The commit and MR title say “Add API documentation using pdoc” and it is not actually adding any documentation it is allowing us to support API generation hence the confusion (for me that is).
What about
waf: Add pdoc support for generating API documentation?
Author: Amar Takhar
2024-07-02T04:43:02.949Z
That is what my title says though.. Add API documentation using pdoc .. I can add the word generating for you.
Author: Amar Takhar
2024-07-02T04:43:02.949Z
I have changed the title to:
doc: Add API documentation generation using pdocI did not add waf because it’s in the ``doc/`` directory and relates to docs more than the build system.
Author: Chris Johns
2024-07-02T04:43:02.949Z
Yeah good point
Author: Amar Takhar
2024-07-02T02:01:38.529Z
added 1 commit
220de474 - doc: Add API documentation using pdoc
Author: Amar Takhar
2024-07-02T02:12:58.112Z
added 1 commit
9020d345 - doc: Add API documentation generation using pdoc
Author: Chris Johns
2024-07-02T04:43:02.971Z
resolved all threads
Author: Gedare Bloom
2024-07-10T05:21:52.427Z
approved this merge request
Author: Gedare Bloom
2024-07-10T05:21:59.251Z
added 6 commits
9020d345…71cf2428 - 5 commits from branch
rtems/tools:maind46ab837 - doc: Add API documentation generation using pdoc
17 - linkers: Improve generation of symbol map file¶
Id |
17 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-07-09T21:01:42.787Z |
Assignee(s) |
Sebastian Huber |
Created |
2024-07-09T08:54:20.467Z |
Updated |
2024-07-09T21:01:44.450Z |
Milestone |
6.1 |
Labels |
rtems::testing |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/17 |
Commit |
Summary¶
Use the user-provided symbol map file name as is. This simplifies the use in build systems by having clearly defined input and output names without some magic inbetween (adding a suffix).
Update #rtems/rtos/rtems/4941.
Author: Sebastian Huber
2024-07-09T08:54:20.587Z
assigned to @sebhub
Author: Sebastian Huber
2024-07-09T08:59:07.719Z
mentioned in merge request rtems/rtos/rtems!111
Author: Sebastian Huber
2024-07-09T09:00:21.608Z
changed title from {-tempfile: Make suffix append optional-} to {+linkers: Improve generation of symbol map file+}
Author: Sebastian Huber
2024-07-09T09:00:21.635Z
changed the description
Author: Chris Johns
2024-07-09T21:01:37.795Z
approved this merge request
15 - elftoolchain/common/_elftc.h: Add Cygwin definition of ELFTC_VCSID¶
Id |
15 |
State |
merged |
Merged by |
Amar Takhar |
Merged at |
2024-06-27T19:15:22.153Z |
Assignee(s) |
Joel Sherrill |
Created |
2024-06-27T19:14:31.197Z |
Updated |
2024-06-27T19:17:36.975Z |
Milestone |
6.1 |
Labels |
tool::rtems-tools |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/15 |
Commit |
Summary¶
elftoolchain/common/_elftc.h: Add Cygwin definition of ELFTC_VCSID
This can be the same as on other hosts with GNU tools but Cygwin was not listed in the conditional logic.
Closes #9
Author: Joel Sherrill
2024-06-27T19:14:31.354Z
assigned to @joel
Author: Amar Takhar
2024-06-27T19:15:14.466Z
I was just looking at that file!
Author: Amar Takhar
2024-06-27T19:15:17.630Z
approved this merge request
Author: Amar Takhar
2024-06-27T19:17:36.951Z
I don’t see an issue open for it:
Author: Amar Takhar
2024-06-27T19:17:55.469Z
That was meant for the issue.
11 - Don’t run Git tests in release version.¶
Id |
11 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-06-19T22:54:39.052Z |
Assignee(s) |
Amar Takhar |
Created |
2024-06-19T02:48:56.097Z |
Updated |
2024-06-19T22:54:40.442Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/11 |
Commit |
Summary¶
Don’t run Git tests in release version.
This checks for the existence of a ‘VERSION’ file if it exists then it will not run the Git tests.
Also remove ALL variable that was unused.
Author: Amar Takhar
2024-06-19T02:48:56.252Z
assigned to @amar
Author: Chris Johns
2024-06-19T22:52:50.759Z
added 3 commits
8b75c404…28e56379 - 2 commits from branch
rtems/tools:maind8ef79b9 - Don’t run Git tests in release version.
Author: Chris Johns
2024-06-19T22:54:35.041Z
approved this merge request
7 - Remove old tests that have been moved to pytest¶
Id |
7 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-06-19T22:51:52.566Z |
Assignee(s) |
Amar Takhar |
Created |
2024-06-17T17:42:31.331Z |
Updated |
2024-06-19T22:51:53.750Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/7 |
Commit |
Summary¶
Solaris and Darwin are covered under ‘unix’. I don’t currently have access to either to test but it should work.
If we need to add custom tests for these we can in the future.
Author: Amar Takhar
2024-06-17T17:43:28.949Z
This also has the changes in !6 .. however those changes aren’t merged yet I’m interested to see how GitLab handles this. I would assume it will automatically pick up the fact that !6 has been merged and remove the commit.
Author: Amar Takhar
2024-06-17T17:43:52.036Z
assigned to @amar
Author: Gedare Bloom
2024-06-18T03:06:33.120Z
Please rename this one or something, it’s a bit confusing atm.
I will not review this one, until the other one is more complete. It is extra work to review the same code in two places. Maybe rebase this MR on top of the other, and link it as a dependency
Author: Amar Takhar
2024-06-18T03:06:33.120Z
One thing that is annoying about GitLab is it’s easy to forget to edit the MR titles/description after fixing an incorrect initial push to the MR. It’s fixed now.
It was decided to keep these two separate. One adds the new test suite this one deletes it.
Author: Amar Takhar
2024-06-17T20:16:56.352Z
changed title from {-Add test suite for r-}te{-m-}st{-oolkit.-} to {+Remove old tests that have been moved to py+}test
Author: Amar Takhar
2024-06-17T20:16:56.373Z
changed the description
Author: Amar Takhar
2024-06-18T01:33:21.052Z
added 2 commits
53383500 - Add test suite for rtemstoolkit.
e7b8bd99 - Remove old tests that have been moved to pytest
Author: Amar Takhar
2024-06-18T03:06:33.137Z
resolved all threads
Author: Amar Takhar
2024-06-18T03:06:39.432Z
marked this merge request as draft
Author: Amar Takhar
2024-06-18T04:02:20.356Z
added 2 commits
3a639da1 - 1 commit from branch
rtems/tools:mainec1260ef - Remove old tests that have been moved to pytest
Author: Amar Takhar
2024-06-18T04:02:33.424Z
marked this merge request as ready
Author: Gedare Bloom
2024-06-18T23:22:25.742Z
@chris would you like to establish any commit message rules for this repo?
To be consistent, at a minimum, I would put the
rtemstoolkit:at the start of the commit message.
Author: Chris Johns
2024-06-18T23:22:25.742Z
Yes, I think prefixing on the first line is a good idea. The top level directory would be a good place to start.
Author: Amar Takhar
2024-06-18T23:20:55.264Z
added 6 commits
ec1260ef…93908c08 - 5 commits from branch
rtems/tools:main5a5ebf2c - Remove old tests that have been moved to pytest
Author: Chris Johns
2024-06-18T23:24:22.032Z
Moving the test code into pytest is a fantastic improvement. Thank you.
The only thing we should discuss now is the moving of some modules between the RSB and
rtemstoolkit. Some modules, not all, are manually kept in sync and removing the tests from here means updating is a little harder. I do not care for the tests at the end of the files if there are tests somewhere to fill that role. I am also happy if we establish this repo as the master copy and if a problem in for exampleexecute.pyis raised in the RSB we update this copy and run the tests before merging the commit into the RSB.I have considered in the past unifying the code in a single repo however in the end I felt the work required, the overheads to keep updated verses the time it takes to sync the small number of common files was not worth it. An example of the work to unify is
config.pyinrtemstoolkitis a better version and an improvement on the RSB but it would need changes deep in the RSB that are stable.
Author: Amar Takhar
2024-06-18T23:24:22.032Z
Yes I agree moving as much of our Python code to rtemstoolkit as possible makes the most sense so we can have a central way to manage everything even if other tools end up being mostly a frontend that’s fine. There’s certainly a lot of duplicated code.
Author: Chris Johns
2024-06-18T23:27:20.195Z
Should we have an issue to list the common files in the RSB and the ones we need to keep updated with
rtemstoolkit. At this point in time I think manual managment is fine.
Author: Amar Takhar
2024-06-18T23:33:05.693Z
Yes that sounds like a good idea but we also have to figure out how we want it to work if the RSB will now require rtemstoolkit to be installed.
Author: Amar Takhar
2024-06-18T23:22:03.683Z
added 1 commit
d8ae74b0 - rtemstoolkit: Remove old tests that have been moved to pytest
Author: Amar Takhar
2024-06-18T23:22:25.124Z
added 6 commits
d8ae74b0…93908c08 - 5 commits from branch
rtems/tools:maina958e40c - rtemstoolkit: Remove old tests that have been moved to pytest
Author: Chris Johns
2024-06-18T23:22:25.806Z
resolved all threads
Author: Amar Takhar
2024-06-18T23:22:59.536Z
I’ve updated the commit title.
Author: Amar Takhar
2024-06-18T23:24:22.048Z
resolved all threads
Author: Chris Johns
2024-06-19T00:49:38.522Z
I am seeing failures. Are there any expected failures? My host is FreeBSD 14.0 and python is 3.9.18
Author: Amar Takhar
2024-06-19T00:49:38.522Z
Hmm what’s the error?
Author: Chris Johns
2024-06-19T00:49:38.522Z
The first is no
svninstalled. I suggest that check can be removed.The second is the repo has an untracked and a change. I think having to have all files committed makes using the tests for development hard.
Author: Chris Johns
2024-06-19T00:49:38.522Z
Releases have a top level
VERSIONand no git repo. What would happen in a release?
Author: Amar Takhar
2024-06-19T00:49:38.522Z
Ohh yes I forgot about the svn error! We can remove that but this merge request is about deleting the old tests did the other one with the testsuite work?
I don’t understand the second part.
Author: Amar Takhar
2024-06-19T00:49:38.522Z
I don’t know but an issue may be a better place to talk about this since this ticket doesn’t have anything to do with the new test suite.
Author: Chris Johns
2024-06-19T00:49:38.522Z
I am fine to see the
svntest go.Sure, is there an issue to use to following up with?
Author: Amar Takhar
2024-06-19T00:49:38.522Z
Can you open one? A release will break for sure since some tests use Git.
Author: Chris Johns
2024-06-19T00:49:38.522Z
Yes and thanks.
Author: Amar Takhar
2024-06-19T00:49:38.522Z
Also can you add the procedure to create a release in that issue?
Author: Chris Johns
2024-06-19T00:49:38.540Z
resolved all threads
Author: Chris Johns
2024-06-19T00:49:50.586Z
added 3 commits
a958e40c…5c5d4927 - 2 commits from branch
rtems/tools:main02e58f7b - rtemstoolkit: Remove old tests that have been moved to pytest
Author: Amar Takhar
2024-06-19T02:48:59.536Z
mentioned in merge request !11
Author: Amar Takhar
2024-06-19T04:34:19.218Z
Is there anything left stopping this from being merged? Thanks.
Author: Chris Johns
2024-06-19T22:51:00.212Z
added 3 commits
02e58f7b…fa49a5e6 - 2 commits from branch
rtems/tools:main28e56379 - rtemstoolkit: Remove old tests that have been moved to pytest
Author: Chris Johns
2024-06-19T22:51:44.898Z
approved this merge request
12 - tests: Mark using xfail instead of skipping the file.¶
Id |
12 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-06-19T22:46:16.413Z |
Assignee(s) |
Amar Takhar |
Created |
2024-06-19T03:08:27.893Z |
Updated |
2024-06-19T22:46:17.464Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/12 |
Commit |
Summary¶
xfail is used when a test is failing but may work. Easier for development when the issue is fixed these will start working again.
Run pytest with:
pytest –runxfail
To see the output of the failed tests.
Remove global skip
Remove an old comment that was left behind
Author: Amar Takhar
2024-06-19T03:08:28.035Z
assigned to @amar
Author: Amar Takhar
2024-06-19T16:54:35.515Z
added 1 commit
fa49a5e6 - tests: Mark using xfail instead of skipping the file.
Author: Amar Takhar
2024-06-19T17:01:48.319Z
changed the description
Author: Chris Johns
2024-06-19T22:46:13.838Z
approved this merge request
8 - Simplify usage of patch_argv fixture.¶
Id |
8 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-06-18T22:59:52.357Z |
Assignee(s) |
Amar Takhar |
Created |
2024-06-18T22:46:17.287Z |
Updated |
2024-06-18T22:59:53.431Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/8 |
Commit |
Summary¶
Simplify usage of patch_argv fixture.
This moves the fixture to conftest and adds it as an argument
Version tests are still broken but this cleanup is required.
Also some whitespace fix.
Author: Amar Takhar
2024-06-18T22:46:17.806Z
assigned to @amar
Author: Chris Johns
2024-06-18T22:59:28.864Z
added 5 commits
8bc67d40…8138b87f - 4 commits from branch
rtems/tools:main93908c08 - Simplify usage of patch_argv fixture.
Author: Chris Johns
2024-06-18T22:59:48.373Z
approved this merge request
5 - rtemstoolkit/elftoolchain: Update to the latest version¶
Id |
5 |
State |
merged |
Merged by |
Amar Takhar |
Merged at |
2024-06-15T02:00:09.959Z |
Assignee(s) |
Chris Johns |
Created |
2024-06-14T08:32:05.824Z |
Updated |
2024-06-15T02:04:40.518Z |
Milestone |
6.1 |
Labels |
tool::rtems-tools |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/5 |
Commit |
Update to the latest elftoolchain version in ELF Sourceforge SVN Head.
The SVN ref is r4040-trunk
Close #6
Author: Chris Johns
2024-06-14T08:32:05.991Z
assigned to @chris
Author: Kinsey Moore
2024-06-15T01:46:26.519Z
This builds fine, but when run against the aarch64/xilinx_zynq_lp64_qemu hello.exe I get this error:
error: dwarf:debug_info_entry:attribute(string): Invalid attribute form [dwarf_attrval_string(107)]
Author: Chris Johns
2024-06-15T01:46:26.519Z
Thanks.
$ rtems-exeinfo `find build -name hello.exe` RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/x86_64/amd64efi/testsuites/samples/hello.exe exe: build/x86_64/amd64efi/testsuites/samples/hello.exe Compilation: Producers: 2 | GNU AS 2.41: 5 objects | GNU C17 13.2.0 20230727 (RTEMS 6, RSB f1a36b851161480ff0a0a1addb22fb44e102d339, Newlib 3cacedb): 282 objects Common flags: 2 | -ftls-model=local-exec -march=x86-64$ rtems-exeinfo build/aarch64/xilinx_zynqmp_lp64_zu3eg/testsuites/samples/hello.exe RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/aarch64/xilinx_zynqmp_lp64_zu3eg/testsuites/samples/hello.exe exe: build/aarch64/xilinx_zynqmp_lp64_zu3eg/testsuites/samples/hello.exe Compilation: Producers: 2 | GNU AS 2.42: 11 objects | GNU C17 13.3.0 20240521 (RTEMS 6, RSB 423d98d2058a7dac0921f9a13113b024696236b9, Newlib 730703b): 266 objects Common flags: 7 | -ftls-model=local-exec -mno-outline-atomics -mcpu=cortex-a53 -mfix-cortex-a53-835769 -mfix-cortex-a53-843419 -mlittle-endian -mabi=lp64$ rtems-exeinfo `find build -name hello.exe` RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/powerpc/mvme2700/testsuites/samples/hello.exe exe: build/powerpc/mvme2700/testsuites/samples/hello.exe Compilation: Producers: 3 | GNU AS 2.42: 6 objects | GNU C17 13.3.0 20240521 (RTEMS 6, RSB 423d98d2058a7dac0921f9a13113b024696236b9, Newlib 730703b): 354 objects | GNU AS 2.41: 1 objects Common flags: 2 | -ftls-model=local-exec -mcpu=604$ rtems-exeinfo `find build -name hello.exe` RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/sparc/erc32/testsuites/samples/hello.exe exe: build/sparc/erc32/testsuites/samples/hello.exe Compilation: Producers: 3 | GNU AS 2.42: 5 objects | GNU C17 13.3.0 20240521 (RTEMS 6, RSB 8037d9ad83d74b5d5eee6ac13ead3580772609cd, Newlib 1ed1516): 272 objects | GNU AS 2.41: 3 objects Common flags: 1 | -ftls-model=local-exec$ rtems-exeinfo `find build -name hello.exe` RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/riscv/rv32imac/testsuites/samples/hello.exe exe: build/riscv/rv32imac/testsuites/samples/hello.exe Compilation: Producers: 2 | GNU AS 2.42: 5 objects | GNU C17 13.3.0 20240521 (RTEMS 6, RSB 8037d9ad83d74b5d5eee6ac13ead3580772609cd, Newlib 1ed1516): 291 objects Common flags: 4 | -ftls-model=local-exec -mabi=ilp32 -misa-spec=20191213 -march=rv32imac$ rtems-exeinfo `find build -name hello.exe` RTEMS Executable Info 6.ed300ed60617-modified /opt/work/chris/rtems/rt/rtems-tools.git/build/linkers/rtems-exeinfo build/riscv/rv64imac/testsuites/samples/hello.exe exe: build/riscv/rv64imac/testsuites/samples/hello.exe Compilation: Producers: 2 | GNU AS 2.42: 5 objects | GNU C17 13.3.0 20240521 (RTEMS 6, RSB 8037d9ad83d74b5d5eee6ac13ead3580772609cd, Newlib 1ed1516): 287 objects Common flags: 5 | -ftls-model=local-exec -mabi=lp64 -mcmodel=medany -misa-spec=20191213 -march=rv64imac
Author: Chris Johns
2024-06-15T01:46:26.538Z
resolved all threads
Author: Chris Johns
2024-06-15T01:47:23.655Z
added 1 commit
9cd5599e - rtemstoolkit/elftoolchain: Update to the latest
Author: Amar Takhar
2024-06-15T02:00:00.839Z
approved this merge request
Author: Chris Johns
2024-06-15T02:03:22.681Z
changed the description
Author: Chris Johns
2024-06-15T02:04:40.568Z
changed the description
4 - Fix Python 3.12 PEP-701¶
Id |
4 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-06-08T04:06:22.890Z |
Assignee(s) |
Amar Takhar |
Created |
2024-06-08T04:05:19.115Z |
Updated |
2024-06-08T04:06:24.627Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/4 |
Commit |
Summary¶
Fix Python 3.12 PEP-701
Escapes are no longer allowed in strings.
Fixes #3
Author: Amar Takhar
2024-06-08T04:05:27.287Z
assigned to @amar
Author: Chris Johns
2024-06-08T04:06:04.500Z
approved this merge request
Author: Chris Johns
2024-06-08T04:06:15.797Z
added 2 commits
779d9312 - 1 commit from branch
rtems/tools:mained300ed6 - Fix Python 3.12 PEP-701