WIP: Add pico_clib_interface_test #3113

2026-08-05T11:47:31-05:00

peterharperuk commented

2026-08-05 11:47:31 -05:00

(Migrated from github.com)

This is a very simple test that references errno.
A TLS reference from a translation unit compiled by the SDK. With ATfE-22.1.0-Linux-x86_64 it generates a linker error

unable to move location counter (0x20000318) backward to 0x20000314 for section '.tbss_space'

See https://github.com/raspberrypi/pico-sdk/issues/3112

This is a very simple test that references errno. A TLS reference from a translation unit compiled by the SDK. With ATfE-22.1.0-Linux-x86_64 it generates a linker error unable to move location counter (0x20000318) backward to 0x20000314 for section '.tbss_space' See https://github.com/raspberrypi/pico-sdk/issues/3112

kilograham (Migrated from github.com) reviewed

2026-08-05 12:20:06 -05:00

test/CMakeLists.txt

					
				@ -22,6 +22,7 @@ if (PICO_ON_DEVICE)

				    add_subdirectory(pico_low_power_test)

one of these directories is not like the other ones; one of these directories is not the same!

lurch (Migrated from github.com) reviewed

2026-08-05 12:22:23 -05:00

test/CMakeLists.txt

					
				@ -22,6 +22,7 @@ if (PICO_ON_DEVICE)

				    add_subdirectory(pico_low_power_test)

Are you suggesting that pico_clib_interface ought to have _test at the end?

Are you suggesting that `pico_clib_interface` ought to have `_test` at the end?

kilograham (Migrated from github.com) reviewed

2026-08-05 12:28:29 -05:00

test/CMakeLists.txt

					
				@ -22,6 +22,7 @@ if (PICO_ON_DEVICE)

				    add_subdirectory(pico_low_power_test)

yes

peterharperuk (Migrated from github.com) reviewed

2026-08-05 12:57:57 -05:00

test/CMakeLists.txt

					
				@ -22,6 +22,7 @@ if (PICO_ON_DEVICE)

				    add_subdirectory(pico_low_power_test)

Fixed. pico_clib_interface_test is probably a bad name

This pull request has changes conflicting with the target branch.

test/CMakeLists.txt

View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Merge commit title

Merge pull request 'WIP: Add pico_clib_interface_test' (#3113) from peterharperuk/tls_link_error into develop

Merge commit body

Reviewed-on: https://git.richey.dev/gh-raspberrypi/pico-sdk/pulls/3113

Merge pull request 'WIP: Add pico_clib_interface_test' (#3113) from peterharperuk/tls_link_error into develop

Reviewed-on: https://git.richey.dev/gh-raspberrypi/pico-sdk/pulls/3113

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin 'peterharperuk/tls_link_error:peterharperuk/tls_link_error'

git switch 'peterharperuk/tls_link_error'

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch 'develop'

git merge --no-ff 'peterharperuk/tls_link_error'

git switch 'peterharperuk/tls_link_error'

git rebase 'develop'

git switch 'develop'

git merge --ff-only 'peterharperuk/tls_link_error'

git switch 'peterharperuk/tls_link_error'

git rebase 'develop'

git switch 'develop'

git merge --no-ff 'peterharperuk/tls_link_error'

git switch 'develop'

git merge --squash 'peterharperuk/tls_link_error'