RTEMS / Tools / RTEMS Tools¶
Go to Issues or Merge Requests
Issues¶
16 - Xilinx MPSoC BSP Names Need Updating¶
Id |
16 |
State |
closed |
Type |
ISSUE |
Author |
Joel Sherrill |
Assignee(s) |
Kinsey Moore |
Closed by |
Kinsey Moore |
Created |
2024-11-13T15:48:26.547Z |
Closed |
2024-11-22T00:45:31.978Z |
Updated |
2024-11-22T00:45:31.989Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/issues/16 |
Merges |
0 |
Summary¶
The configuration files for rtems-test use the old names for these BSPs. Please update.
Steps to reproduce¶
Pre-set options¶
Author: Joel Sherrill
2024-11-13T15:48:26.636Z
assigned to @opticron
Author: Kinsey Moore
2024-11-13T16:06:31.750Z
mentioned in merge request !39
17 - GDB Python rtems.pprinter is missing¶
Id |
17 |
State |
closed |
Type |
ISSUE |
Author |
Chris Johns |
Assignee(s) |
Chris Johns |
Closed by |
Amar Takhar |
Created |
2024-11-15T05:29:12.954Z |
Closed |
2024-11-15T06:48:05.891Z |
Updated |
2024-11-15T06:48:05.951Z |
Milestone |
6.1 |
Labels |
tool::rtems-tools |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/issues/17 |
Merges |
1 |
Summary¶
There is a module missing from the RTEMS GDB C++ Pretty Printer support. Starting gdb results in:
Find the GDB manual and other documentation resources online at:
<http://www.gnu.org/software/gdb/documentation/>.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from build/arm-rtems6-xilinx_zynq_microzed/app/spu/spu...
RTEMS GDB Support
Python Exception <class 'ModuleNotFoundError'>: No module named 'rtems.pprinter'
(gdb)
This issue is set to the %6.1 milestone because it is fixing the support being documented by https://gitlab.rtems.org/rtems/rtos/rtems-release/-/merge_requests/9.
Steps to reproduce¶
Start an RTEMS gdb with an RTEMS executable.
Author: Chris Johns
2024-11-15T05:29:13.073Z
assigned to @chris
Author: Chris Johns
2024-11-15T06:14:00.357Z
The problem is the some files where added in https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/26 the MR did not update the
wafscript to install them.I have also fixed the case of the RSB failing to install the version specific GCC support. A warning is now printed:
Type "apropos word" to search for commands related to "word"... Reading symbols from build/arm-rtems6-xilinx_zynq_microzed/app/spu/spu... RTEMS GDB Support warning: no C++ pretty printers; invalid install (gdb)
Author: Chris Johns
2024-11-15T06:30:27.810Z
mentioned in merge request !40
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
Merge Requests¶
43 - waf: update to 2.1.4¶
Id |
43 |
State |
merged |
Merged by |
Gedare Bloom |
Merged at |
2024-11-20T23:42:00.506Z |
Assignee(s) |
Gedare Bloom |
Reviewer(s) |
Chris Johns |
Created |
2024-11-20T23:32:37.760Z |
Updated |
2024-11-21T00:11:53.263Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/43 |
Commit |
Summary¶
Bump waf version from 2.1.2 to 2.1.4
Tested on Windows/MSYS2.
Author: Gedare Bloom
2024-11-20T23:32:37.948Z
assigned to @gedare
Author: Gedare Bloom
2024-11-20T23:34:26.670Z
requested review from @chris
Author: Chris Johns
2024-11-20T23:35:05.129Z
approved this merge request
Author: Chris Johns
2024-11-20T23:35:19.711Z
Milestone?
Author: Gedare Bloom
2024-11-20T23:41:48.381Z
Tested on Ubuntu
Author: Gedare Bloom
2024-11-21T00:11:53.220Z
mentioned in merge request !44
40 - GDB Python: Install all python files¶
Id |
40 |
State |
merged |
Merged by |
Amar Takhar |
Merged at |
2024-11-15T06:48:05.500Z |
Assignee(s) |
Chris Johns |
Created |
2024-11-15T06:30:20.644Z |
Updated |
2024-11-15T07:21:23.103Z |
Milestone |
6.1 |
Labels |
tool::rtems-tools |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/40 |
Commit |
Install all python files to C++ pretty printers work. Also reformat the python code.
Closes #17
Author: Chris Johns
2024-11-15T06:30:20.845Z
assigned to @chris
Author: Amar Takhar
2024-11-15T06:47:50.192Z
added 4 commits
729284d2 - 1 commit from branch
rtems/tools:maind5a6976a - tools/gdb/python: Install all python files
a84c2a34 - tools/gdb/python: Print an error if the C++ support is not installed
f4729624 - tools/gdb/python: Format using yapf
Author: Amar Takhar
2024-11-15T06:47:54.624Z
approved this merge request
Author: Chris Johns
2024-11-15T07:21:23.158Z
changed title from GDB Py{-rh-}ton: Install all python files to GDB Pyt{+h+}on: Install all python files
41 - trace/record: Fix warnings on FreeBSD 14.1¶
Id |
41 |
State |
merged |
Merged by |
Sebastian Huber |
Merged at |
2024-11-15T06:39:12.909Z |
Assignee(s) |
Chris Johns |
Reviewer(s) |
Sebastian Huber |
Created |
2024-11-15T06:37:53.747Z |
Updated |
2024-11-15T06:39:14.888Z |
Milestone |
6.1 |
Labels |
tool::rtems-tools |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/41 |
Commit |
Fix warnings on FreeBSD 14.1 in trace recorder from https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/36.
Author: Chris Johns
2024-11-15T06:37:53.922Z
assigned to @chris
Author: Chris Johns
2024-11-15T06:38:14.177Z
requested review from @sebhub
Author: Sebastian Huber
2024-11-15T06:39:05.874Z
approved this merge request
36 - Improve event record handling¶
Id |
36 |
State |
merged |
Merged by |
Kinsey Moore |
Merged at |
2024-11-15T00:41:56.821Z |
Assignee(s) |
Sebastian Huber |
Created |
2024-10-29T10:39:12.709Z |
Updated |
2024-11-15T06:37:59.883Z |
Milestone |
6.1 |
Labels |
api |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/36 |
Commit |
Summary¶
The event record handling had some reliability issues especially on SMP systems. This is fixed by a new approach to fetch the event records. See also: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/282
Author: Sebastian Huber
2024-10-29T10:39:12.781Z
assigned to @sebhub
Author: Sebastian Huber
2024-10-29T10:40:41.872Z
mentioned in merge request rtems/rtos/rtems!282
Author: Sebastian Huber
2024-10-29T10:41:02.588Z
changed title from {-Update company name-} to {+Improve event record handling+}
Author: Sebastian Huber
2024-10-29T10:41:02.611Z
changed the description
Author: Gedare Bloom
2024-11-06T05:10:10.529Z
tools/4.12/newlib/newlib-cygwin-git-461152e4ebd94f7d8b1e20f4cd1c704138ba2083.patchthis seems unrelated. I think this patch is merged upstream. Please send an MR to delete this file if it is not longer needed.
Author: Sebastian Huber
2024-11-06T05:10:10.529Z
Yes, sorry, this was an oversight. I will remove it from the commit.
Author: Sebastian Huber
2024-11-01T05:26:47.187Z
added 4 commits
8096c5d9 - Update company name
9d0b7002 - record: Fix base64 decoding
8d8c3660 - record: Use event records v10
043e266a - record: Add -t option
Author: Sebastian Huber
2024-11-06T05:09:26.274Z
added 8 commits
043e266a…56e81e76 - 2 commits from branch
rtems/tools:mainffff1583 - Update company name
0cd1b241 - record: Fix base64 decoding
65deca21 - record: Fix zlib decompression
9681de2b - record: Use memset() to initialize the zlib stream
034f8d9d - record: Add -t option
f24f8c29 - record: Use event records v10
Author: Sebastian Huber
2024-11-06T05:10:10.557Z
resolved all threads
Author: Sebastian Huber
2024-11-06T05:10:36.715Z
I fixed another bug in the zlib decompression filter.
Author: Sebastian Huber
2024-11-06T05:20:22.014Z
added 1 commit
3a962ce5 - trace: Check for latest C++ standard early
Author: Sebastian Huber
2024-11-06T05:21:19.735Z
I added a fix to improve the llvm/DebugInfo/Symbolize/Symbolize.h detection.
Author: Chris Johns
2024-11-08T01:34:26.777Z
Why not use smart pointers?
Author: Sebastian Huber
2024-11-08T01:34:26.777Z
I program in C++ once in a year. There are probably plenty of opportunities to turn this into a real C++ code, but is this really necessary?
Author: Sebastian Huber
2024-11-08T01:33:53.450Z
changed this line in version 5 of the diff
Author: Chris Johns
2024-11-08T01:34:29.401Z
Why not use shared or unique?
Author: Sebastian Huber
2024-11-08T01:33:54.015Z
changed this line in version 5 of the diff
Author: Sebastian Huber
2024-11-08T01:33:54.228Z
added 1 commit
51b500f4 - trace: Use smart pointers
Author: Sebastian Huber
2024-11-08T01:34:29.433Z
resolved all threads
Author: Gedare Bloom
2024-11-09T19:52:52.862Z
approved this merge request
Author: Sebastian Huber
2024-11-11T06:16:12.289Z
added 5 commits
0970ae1a - record: Add -t option
7d952150 - record: Use smart pointers
fc6ee411 - record: Add error messages
a7a2553d - trace: Check for latest C++ standard early
f5f3d683 - record: Use event records v10
Author: Sebastian Huber
2024-11-11T06:16:28.259Z
reset approvals from @gedare by pushing to the branch
Author: Sebastian Huber
2024-11-12T07:39:33.786Z
added 1 commit
55e045c8 - record: Add -o option
Author: Sebastian Huber
2024-11-13T00:50:48.261Z
Due to rtems/rtos/rtems!282, this should be merged as well. I have to update the RSB afterwards.
Author: Chris Johns
2024-11-15T00:41:47.887Z
Why not use std::make_unique? I think C++ is better with no direct use of
newanddelete.
Author: Sebastian Huber
2024-11-15T00:41:47.887Z
For me this is good enough, I have to move to other topics on my TODO list.
Author: Sebastian Huber
2024-11-14T23:36:17.827Z
resolved all threads
Author: Chris Johns
2024-11-15T00:41:37.659Z
approved this merge request
Author: Chris Johns
2024-11-15T00:41:47.906Z
resolved all threads
Author: Chris Johns
2024-11-15T06:37:59.851Z
mentioned in merge request !41
39 - tester: Update ZynqMP and Versal BSP names¶
Id |
39 |
State |
merged |
Merged by |
Kinsey Moore |
Merged at |
2024-11-15T00:47:07.137Z |
Assignee(s) |
Kinsey Moore |
Created |
2024-11-13T16:06:25.879Z |
Updated |
2024-11-15T00:47:08.578Z |
Milestone |
6.1 |
Labels |
arch:aarch64, tool::test |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/39 |
Commit |
Summary¶
tester: Update ZynqMP and Versal BSP names
CLoses #16
Author: Kinsey Moore
2024-11-13T16:06:26.133Z
assigned to @opticron
Author: Kinsey Moore
2024-11-13T16:06:42.431Z
added 2 commits
56e81e76 - 1 commit from branch
rtems/tools:main293054e8 - tester: Update ZynqMP and Versal BSP names
Author: Chris Johns
2024-11-15T00:46:54.221Z
added 11 commits
293054e8…55e045c8 - 10 commits from branch
rtems/tools:mainc01e39d2 - tester: Update ZynqMP and Versal BSP names
Author: Chris Johns
2024-11-15T00:47:01.081Z
approved this merge request
38 - misc/bin2c: Silence warning on FreeBSD (clang)¶
Id |
38 |
State |
merged |
Merged by |
Amar Takhar |
Merged at |
2024-11-01T01:54:09.584Z |
Assignee(s) |
Chris Johns |
Created |
2024-11-01T01:52:45.141Z |
Updated |
2024-11-01T01:54:12.007Z |
Milestone |
6.1 |
Labels |
tool |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/38 |
Commit |
Silence the following warning on FreeBSD:
../misc/bin2c/rtems-bin2c.c:143:16: warning: '_Generic' is a C11 extension [-Wc11-extensions]
143 | ifbasename = basename(ifbasename_to_free);
| ^
/usr/include/libgen.h:60:21: note: expanded from macro 'basename'
60 | #define basename(x) __generic(x, const char *, __old_basename, basename)(x)
| ^
/usr/include/sys/cdefs.h:322:2: note: expanded from macro '__generic'
322 | _Generic(expr, t: yes, default: no)
| ^
1 warning generated.
Author: Chris Johns
2024-11-01T01:52:45.336Z
assigned to @chris
Author: Amar Takhar
2024-11-01T01:54:03.891Z
approved this merge request
37 - rtemstoolkit/elf: Update support machine types to match RTEMS¶
Id |
37 |
State |
merged |
Merged by |
Kinsey Moore |
Merged at |
2024-11-01T00:34:44.168Z |
Assignee(s) |
Chris Johns |
Reviewer(s) |
Kinsey Moore |
Created |
2024-11-01T00:15:19.459Z |
Updated |
2024-11-01T00:45:34.795Z |
Milestone |
6.1 |
Labels |
arch:microblaze |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/37 |
Commit |
Update support machine types to match RTEMS
Match the machine types to the libdl arch support.
Updates rtems/rtos/rtems#5141
Author: Chris Johns
2024-11-01T00:15:19.629Z
requested review from @opticron
Author: Chris Johns
2024-11-01T00:15:19.644Z
assigned to @chris
Author: Chris Johns
2024-11-01T00:15:47.833Z
added 33 commits
1337023c…db3a6e34 - 32 commits from branch
rtems/tools:main8ff9a508 - rtemstoolkit/elf: Update support machine types to match RTEMS
Author: Kinsey Moore
2024-11-01T00:34:39.426Z
approved this merge request
Author: Amar Takhar
2024-11-01T00:45:29.433Z
Related to https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/285
35 - Remove import of host¶
Id |
35 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-10-28T04:03:50.195Z |
Assignee(s) |
Amar Takhar |
Reviewer(s) |
Chris Johns, Gedare Bloom |
Created |
2024-10-28T03:36:27.500Z |
Updated |
2024-10-28T04:03:51.956Z |
Milestone |
6.1 |
Labels |
tool::rtems-tools |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/35 |
Commit |
Summary¶
Remove import of host
This avoids a circular dependency in execute.py since other modules that will also use host import it as well.
This uses os.name to detect if we’re on Windows which is “nt”
Author: Amar Takhar
2024-10-28T03:36:27.643Z
requested review from @chris
Author: Amar Takhar
2024-10-28T03:36:27.660Z
assigned to @amar
Author: Amar Takhar
2024-10-28T03:36:44.916Z
requested review from @gedare
Author: Amar Takhar
2024-10-28T03:36:54.949Z
This executes but I’m not on a Windows machine to test if this works OK until @gedare tests it.
Author: Chris Johns
2024-10-28T04:03:05.190Z
Does this work on Cygwin?
Author: Amar Takhar
2024-10-28T04:03:05.190Z
No it won’t it’ll return posix. Native Python will return ‘nt’ and mingw python will return ‘nt’. Does it need to? I guess I should have asked didn’t think about it since it didn’t come up in conversation.
Author: Amar Takhar
2024-10-28T04:03:05.190Z
I could change it to use sys.platform and check for
win32,cygwinandmsysthat would cover those 3.
Author: Chris Johns
2024-10-28T04:03:05.190Z
Great. The Cyginw vs MSYS2 verses native Windows can become tricky.
Author: Amar Takhar
2024-10-28T03:49:10.728Z
added 1 commit
db3a6e34 - Remove import of host
Author: Gedare Bloom
2024-10-28T03:57:39.942Z
This MR works for me on MSYS2 and on Ubuntu. The os.name == ‘nt’ should only pick up MSYS2, not pick up Cygwin, so this should be non-functional change for Cygwin.
Author: Gedare Bloom
2024-10-28T04:00:31.930Z
approved this merge request
Author: Chris Johns
2024-10-28T04:03:05.209Z
resolved all threads
33 - Fixes for tester running on windows MSYS¶
Id |
33 |
State |
merged |
Merged by |
Amar Takhar |
Merged at |
2024-10-28T02:44:36.527Z |
Created |
2024-10-28T02:42:40.253Z |
Updated |
2024-10-28T02:50:16.139Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/33 |
Commit |
Summary¶
Windows path needs to be corrected for configparser.load() to work. and the execute.py in rtemstoolkit seems to need to invoke commands in a shell.
Author: Amar Takhar
2024-10-28T02:44:31.288Z
approved this merge request
34 - execute.py: restore accidentally deleted command construction¶
Id |
34 |
State |
merged |
Merged by |
Amar Takhar |
Merged at |
2024-10-28T02:49:37.856Z |
Created |
2024-10-28T02:47:59.988Z |
Updated |
2024-10-28T02:50:11.515Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/34 |
Commit |
Summary¶
Author: Amar Takhar
2024-10-28T02:49:33.690Z
approved this merge request
31 - Reformat code using yapf¶
Id |
31 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-10-21T04:42:51.379Z |
Assignee(s) |
Amar Takhar |
Reviewer(s) |
Chris Johns |
Created |
2024-10-21T04:22:20.469Z |
Updated |
2024-10-21T04:42:53.405Z |
Milestone |
6.1 |
Labels |
python, tool::rtems-tools |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/31 |
Commit |
Summary¶
Reformat code using yapf
Done using default settings.
Author: Amar Takhar
2024-10-21T04:22:20.710Z
assigned to @amar
Author: Chris Johns
2024-10-21T04:25:01.573Z
The following are imported and should not be reformatted:
misc/tool\ |md_0_8206|\ s/getmac
tester/r\ |md_0_8206|\ t/pygdb
Author: Chris Johns
2024-10-21T04:42:38.182Z
These files are python:
tester/rtems-bsp-builder
tester/rtems-test
tester/rtems-run
tester/rtems-tftp-server
misc/rtems-tftp-proxy
misc/rtems-boot-image
Author: Amar Takhar
2024-10-21T04:42:38.182Z
Updated.
Author: Chris Johns
2024-10-21T04:29:29.708Z
requested review from @chris
Author: Amar Takhar
2024-10-21T04:33:06.125Z
oh, duh wonder why i didn’t think to check for the binaries i just did
*.pydoh thanks I’ll grab it.I tried to exclude imported code didn’t notice those thanks.
Author: Amar Takhar
2024-10-21T04:35:55.651Z
added 1 commit
88c7f5df - Reformat code using yapf
Author: Chris Johns
2024-10-21T04:42:38.200Z
resolved all threads
Author: Chris Johns
2024-10-21T04:42:43.245Z
approved this merge request
28 - stty: fix open and read calls¶
Id |
28 |
State |
merged |
Merged by |
Kinsey Moore |
Merged at |
2024-10-07T14:34:17.358Z |
Created |
2024-09-28T21:18:43.316Z |
Updated |
2024-10-07T14:34:18.794Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/28 |
Commit |
Summary¶
This merge request fixes the open() and read() calls for tty devices on rtems tester and run.
Opening a device in ‘rw’ mode causes:
error: opening tty dev: /dev/ttyACM0: must have exactly one of create/read/write/append mode
Reading from device:
Exception in thread tty[/dev/ttyACM0]:
[...]
File "/usr/lib/python3.10/codecs.py", line 321, in decode
data = self.buffer + input
TypeError: can't concat NoneType to bytes
Author: Ranulfo Raphael
2024-09-28T21:20:12.206Z
changed the description
Author: Kinsey Moore
2024-10-07T14:34:10.859Z
approved this merge request
24 - Added niosvc10lp BSP support¶
Id |
24 |
State |
merged |
Merged by |
Joel Sherrill |
Merged at |
2024-08-23T17:18:39.368Z |
Created |
2024-08-06T20:12:59.200Z |
Updated |
2024-09-02T12:43:42.923Z |
Milestone |
6.1 |
Labels |
arch:riscv |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/24 |
Commit |
Summary¶
Added niosvc10lp BSP support for Issue #5101 1. Add niosvc10lp bsp and tier 2. Added support for Intel’s juart_terminal.exe JTAG UART application as a console device 3. Changed the way GDB shuts down after closing to work with ash-riscv-gdb-server.exe GDB Server a. Ashling ash-riscv-gdb-server.exe GDB Server doesn’t shut down properly when killing it through SIGTERM. b. When closing gdb.py, call stop() instead of kill() to allow graceful shutdown. c. Ashling ash-riscv-gdb-server.exe GDB Server doesn’t respond to the -exec-interrupt command. d. Send SIGINT signal to break active GDB session e. Added disconnect before quiting GDB session f. commented out print() when sending signals to keep output clean with these changes 4. When opening and closing the console, added the current test index and total test value to allow console to perform actions on first and last test case. Important for opening and closing the juart_terminal.exe only once since to lower the chance of a failed initialization when it runs over the JTAG cable.
Author: Joel Sherrill
2024-08-11T13:59:02.253Z
Looks like an accidental insertion.
Author: Kevin Kirspel
2024-08-07T13:35:20.490Z
changed this line in version 2 of the diff
Author: Kevin Kirspel
2024-08-11T13:59:02.253Z
Fixed accidental insertion
Author: Joel Sherrill
2024-08-11T13:59:34.427Z
I am not sure rtems-tools is uniformly doing it, but please add an SPDX comment as line 1 followed by a blank line.
Author: Kevin Kirspel
2024-08-11T13:59:34.427Z
Added SPDX comment
Author: Joel Sherrill
2024-08-11T14:00:14.805Z
This is your contribution. :)
Author: Kevin Kirspel
2024-08-07T13:35:21.346Z
changed this line in version 2 of the diff
Author: Kevin Kirspel
2024-08-11T14:00:14.805Z
Updated copyright to reflect my contribution
Author: Joel Sherrill
2024-08-11T14:00:31.342Z
SPDX line please – add to all new files.
Author: Kevin Kirspel
2024-08-11T14:00:31.342Z
Added SPDX comment
Author: Joel Sherrill
2024-08-06T20:53:14.497Z
This will need a more thorough review by @kiwichris.
Author: Kevin Kirspel
2024-08-07T13:35:21.790Z
added 1 commit
b0e71d98 - Fixed issues identified by Joel: accidental insertion, SPDX comments, and contribuition
Author: Kevin Kirspel
2024-08-11T14:00:31.369Z
resolved all threads
Author: Kevin Kirspel
2024-08-23T17:15:25.762Z
added 6 commits
b0e71d98…24475571 - 4 commits from branch
rtems/tools:main518e09e0 - Added niosvc10lp BSP support
18e38d5d - Fixed issues identified by Joel: accidental insertion, SPDX comments, and contribuition
Author: Joel Sherrill
2024-08-23T17:18:34.532Z
approved this merge request
Author: Frank Kuehndel
2024-09-02T12:43:42.825Z
mentioned in issue #13
19 - linkers: Fix symbol table size placement¶
Id |
19 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-07-20T00:01:01.418Z |
Assignee(s) |
Sebastian Huber |
Created |
2024-07-18T18:40:11.225Z |
Updated |
2024-08-28T22:37:54.939Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/19 |
Commit |
Summary¶
linkers: Fix symbol table size placement
The symbol table size label must be in the same section as the symbol table base label. Otherwise relocation errors may occur on some targets.
Author: Sebastian Huber
2024-07-18T18:40:12.237Z
assigned to @sebhub
Author: Sebastian Huber
2024-07-19T06:02:19.289Z
added 1 commit
d282b57d - linkers: Single asm() block for the symbol table
Author: Chris Johns
2024-07-20T00:00:40.526Z
approved this merge request
Author: Chris Johns
2024-07-20T00:00:47.477Z
added 4 commits
d282b57d…98ddf340 - 2 commits from branch
rtems/tools:main9a732207 - linkers: Fix symbol table size placement
0483fe3a - linkers: Single asm() block for the symbol table
Author: Sebastian Huber
2024-08-28T22:37:54.778Z
mentioned in issue rtems/rtos/rtems#5123
21 - Add script for updating ACPICA code in the RTEMS source tree¶
Id |
21 |
State |
merged |
Merged by |
Joel Sherrill |
Merged at |
2024-08-23T14:36:53.533Z |
Created |
2024-07-22T17:34:59.729Z |
Updated |
2024-08-23T14:36:55.949Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/21 |
Commit |
Summary¶
This MR also contains changes in rtemstoolkit:
Adding a missing import in path.py which was needed for the “removeall” method
Adding methods to git.py which are used by the script
I tested the script on Gentoo Linux, FreeBSD and Windows with Cygwin. I don’t have any means to test it on macOS.
Closes #11
Author: Matheus Pecoraro
2024-07-22T17:40:44.446Z
added 1 commit
793a5aaf - misc: Add rtems-gen-acpica-patches script
Author: Matheus Pecoraro
2024-07-22T19:56:59.617Z
Author: Matheus Pecoraro
2024-07-22T20:06:42.131Z
changed the description
Author: Chris Johns
2024-08-07T02:57:07.678Z
marked this merge request as draft
Author: Chris Johns
2024-08-23T14:35:44.179Z
Please run the python code thru
yapfwith the default arguments. Thanks.
Author: Matheus Pecoraro
2024-08-23T14:35:44.179Z
Done
Author: Matheus Pecoraro
2024-08-07T15:54:51.340Z
added 5 commits
793a5aaf…478c0727 - 2 commits from branch
rtems/tools:main132c9c36 - rtemstoolkit: Add missing import to path.py
bc50035b - rtemstoolkit: Add more methods to git.py
97493568 - misc: Add rtems-gen-acpica-patches script
Author: Joel Sherrill
2024-08-23T14:35:44.198Z
resolved all threads
Author: Joel Sherrill
2024-08-23T14:35:49.934Z
approved this merge request
Author: Joel Sherrill
2024-08-23T14:36:20.882Z
marked this merge request as ready
Author: Joel Sherrill
2024-08-23T14:36:36.736Z
added 4 commits
78266a24 - 1 commit from branch
rtems/tools:main2c5ae13c - rtemstoolkit: Add missing import to path.py
77d32062 - rtemstoolkit: Add more methods to git.py
24475571 - misc: Add rtems-gen-acpica-patches script
2 - rld: recognize aarch64.¶
Id |
2 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-05-15T23:46:44.734Z |
Created |
2024-05-15T06:24:12.717Z |
Updated |
2024-08-10T00:40:54.524Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/2 |
Commit |
Summary¶
close #2
Author: Kinsey Moore
2024-05-15T15:20:53.164Z
These are in alphabetical order and aarch64 comes before arm. This addition needs to be moved up one line.
Author: xiaojun zheng
2024-05-15T14:47:00.503Z
changed this line in version 1 of the diff
Author: xiaojun zheng
2024-05-15T14:50:19.008Z
added 1 commit
2f1c2ebb - rld: recognize aarch64.
Author: Kinsey Moore
2024-05-15T15:20:53.181Z
resolved all threads
Author: Kinsey Moore
2024-05-15T15:21:13.298Z
approved this merge request
Author: Kinsey Moore
2024-05-15T17:40:21.194Z
I’d like @chris to double check this before it gets merged as I’m not familiar at all with the codebase or what else might be required when adding an architecture.
Author: Chris Johns
2024-05-15T23:46:40.273Z
approved this merge request
9 - Convert README to MarkDown and update.¶
Id |
9 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-06-19T00:22:58.774Z |
Created |
2024-06-18T23:19:45.647Z |
Updated |
2024-08-10T00:40:27.535Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/9 |
Commit |
Summary¶
Convert README to MarkDown and update.
Also fix some URL references and general cleanup. Point to Discord instead of IRC.
Closes #7
Author: Chris Johns
2024-06-19T00:22:52.056Z
Is there something I need to install to run
pytest?(r-py-3) chris@torea:/opt/work/chris/rtems/rt/rtems-tools.git $ python --version Python 3.9.18 (r-py-3) chris@torea:/opt/work/chris/rtems/rt/rtems-tools.git $ type pytest pytest: not found
Author: Amar Takhar
2024-06-19T00:22:52.056Z
Yes you need to install pytest I will update the readme.
Author: Amar Takhar
2024-06-18T23:35:18.051Z
added 1 commit
5c5d4927 - Add note about installing pytest first and it’s location.
Author: Chris Johns
2024-06-19T00:22:52.072Z
resolved all threads
Author: Chris Johns
2024-06-19T00:22:55.470Z
approved this merge request
10 - rtemstoolkit: Drop requirement for ‘svn’.¶
Id |
10 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-06-19T00:51:36.443Z |
Assignee(s) |
Amar Takhar |
Created |
2024-06-19T00:36:22.107Z |
Updated |
2024-08-10T00:40:27.438Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/10 |
Commit |
Summary¶
rtemstoolkit: Drop requirement for ‘svn’.
We don’t use or require subversion for anything no need to force users to install it.
Author: Amar Takhar
2024-06-19T00:36:28.547Z
assigned to @amar
Author: Chris Johns
2024-06-19T00:51:00.644Z
added 3 commits
ffcf694d…5c5d4927 - 2 commits from branch
rtems/tools:main61b383f9 - rtemstoolkit: Drop requirement for ‘svn’.
Author: Chris Johns
2024-06-19T00:51:31.371Z
approved this merge request
3 - tester/amd64: Add amd64 tester configurations¶
Id |
3 |
State |
merged |
Merged by |
Gedare Bloom |
Merged at |
2024-06-18T16:07:28.353Z |
Created |
2024-06-03T19:39:26.806Z |
Updated |
2024-08-10T00:40:27.242Z |
Milestone |
6.1 |
Labels |
arch:x86_64 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/3 |
Commit |
Summary¶
Add tester configurations for both amd64 (using the FreeBSD bootloader) and amd64efi (using GRUB).
Additionally fix a typo causing @FEXE@ to not expand to the correct value and add an option to qemu.cfg allowing the image path to be specified by the BSP config.
Author: Matheus Pecoraro
2024-06-05T18:13:22.566Z
added 1 commit
58d0d6c9 - tester/amd64: Add amd64_qemu config
Author: Matheus Pecoraro
2024-06-12T10:19:03.693Z
mentioned in merge request rtems-source-builder!24
Author: Matheus Pecoraro
2024-06-13T19:26:09.051Z
Author: Joel Sherrill
2024-06-18T16:06:41.525Z
Looks ok. The process of creating a bootable executable and booting it should be documented in the Users Guide. Please create an issue for that.
Once I see the other issue, I will approve this.
Author: Matheus Pecoraro
2024-06-18T16:06:41.525Z
Author: Matheus Pecoraro
2024-06-15T20:41:56.631Z
mentioned in issue rtems/docs/rtems-docs#15
Author: Gedare Bloom
2024-06-18T16:06:41.541Z
resolved all threads
Author: Gedare Bloom
2024-06-18T16:06:46.755Z
approved this merge request
Author: Gedare Bloom
2024-06-18T16:06:56.107Z
added 7 commits
58d0d6c9…3a639da1 - 3 commits from branch
rtems/tools:maine0931e62 - tester: Make @FEXE@ expand to filtered executable
37653db6 - tester: Add option to specify image loaded by QEMU
e4ab5711 - tester/amd64: Add amd64efi_grub_qemu config
8138b87f - tester/amd64: Add amd64_qemu config
16 - rtems-test: Fix –jobs option in –help output¶
Id |
16 |
State |
merged |
Merged by |
Kinsey Moore |
Merged at |
2024-06-27T21:12:54.933Z |
Created |
2024-06-27T20:19:04.371Z |
Updated |
2024-08-10T00:40:17.286Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/16 |
Commit |
Summary¶
This merge request fixes the --jobs option in --help output.
$ rtems-test --help | grep jobs
--jobs=[0..n,none,half,full] : Run with specified number of jobs, default: num CPUs.
$ rtems-test --jobs=full
error: invalid jobs option: full
The valid jobs option is max.
Author: Ranulfo Raphael
2024-06-27T20:35:22.868Z
added 1 commit
9ca98805 - rtems-test: Fix –jobs option in –help output
Author: Kinsey Moore
2024-06-27T21:12:51.300Z
approved this merge request
20 - linkers: Constify TLS offsets¶
Id |
20 |
State |
merged |
Merged by |
Chris Johns |
Merged at |
2024-07-19T21:14:10.483Z |
Assignee(s) |
Sebastian Huber |
Created |
2024-07-19T06:30:13.360Z |
Updated |
2024-08-10T00:40:04.479Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/20 |
Commit |
Summary¶
Author: Sebastian Huber
2024-07-19T06:30:14.277Z
assigned to @sebhub
Author: Chris Johns
2024-07-19T21:14:03.828Z
approved this merge request
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:
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
44 - Update waf to 2.1.4 (closed)¶
Id |
44 |
State |
closed |
Assignee(s) |
Amar Takhar |
Closed by |
Gedare Bloom |
Created |
2024-11-20T23:39:41.396Z |
Closed |
2024-11-21T00:11:17.931Z |
Updated |
2024-11-21T00:11:17.917Z |
Milestone |
6.1 |
Labels |
tool::waf |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/44 |
Commit |
Summary¶
Update waf to 2.1.4
This syncs every waf binary together.
Fixes issues with building on Windows see
Updates rtems/rtos/rtems#5153
Author: Amar Takhar
2024-11-20T23:39:41.625Z
assigned to @amar
Author: Gedare Bloom
2024-11-21T00:11:52.779Z
Completed in !43
18 - Add pprinter.py for pretty-printer support (closed)¶
Id |
18 |
State |
closed |
Closed by |
Suraj Kumar |
Created |
2024-07-17T00:51:07.987Z |
Closed |
2024-08-18T07:18:36.227Z |
Updated |
2024-08-18T07:18:36.209Z |
Milestone |
6.1 |
Link |
https://gitlab.rtems.org/rtems/tools/rtems-tools/-/merge_requests/18 |
Commit |
Summary¶
This MR adds a new file, pprinter.py to act as a layer that we can modify to add RTEMS pretty-printers as well as load the existing GCC printers. The stdcxx.py script it references is installed by the RSB at build time.
Author: Suraj Kumar
2024-08-18T06:41:03.587Z
marked this merge request as ready
Author: Chris Johns
2024-08-18T06:42:55.866Z
added 7 commits
56798c1e…478c0727 - 6 commits from branch
rtems/tools:main3228f155 - gdb/python: add new file pprinter.py to support RTEMS pretty-printers and load the GCC printers
Author: Chris Johns
2024-08-18T06:44:33.361Z
Please a license and a comment header about the file and the role it plays in loading GCC pretty printers.