Skip to content

treewide: remove deprecated BOARDSDIR variable - #22723

Merged
crasbe merged 1 commit into
RIOT-OS:masterfrom
crasbe:pr/remove_BOARDSDIR
Sep 17, 2026
Merged

crasbe merged 1 commit into
RIOT-OS:masterfrom
crasbe:pr/remove_BOARDSDIR

Conversation

@crasbe

@crasbe crasbe commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Contribution description

BOARDSDIR has been deprecated in favor of EXTERNAL_BOARD_DIRS since 2021. Even the last application should've migrated by now.

Testing procedure

No references to BOARDSDIR left (except for the one covered in #22722 ).

buechse@skyleaf:~/RIOTstuff/riot-upstream/RIOT$ grep -RnwI BOARDSDIR
release-notes.txt:15598:    * info-global.inc.mk: reset BOARDSDIR
release-notes.txt:16345:        + introduce 'BOARDSDIR' to support external boards using common code in
dist/gdbinit-docker:21:    path = os.environ['BOARDSDIR']

Issues/PRs references

Follow up for #16104 which deprecated BOARDSDIR.

Depends on #22722.

Declaration of AI-Tools / LLMs usage:

AI-Tools / LLMs that were used are:

  • none

@crasbe
crasbe requested a review from maribu September 16, 2026 16:45
@crasbe crasbe added Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation State: waiting for other PR State: The PR requires another PR to be merged first CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: removal Integration Process: The PR is removing a deprecated feature or API AI: Not Used AI was stated to not be used in this PR/Issue labels Sep 16, 2026
@github-actions github-actions Bot added Area: doc Area: Documentation Area: tests Area: tests and testing framework Area: build system Area: Build system Area: tools Area: Supplementary tools labels Sep 16, 2026
@riot-ci

riot-ci commented Sep 16, 2026

Copy link
Copy Markdown

Murdock results

✔️ PASSED

7d56fd0 treewide: remove deprecated BOARDSDIR variable

Success Failures Total Runtime
11286 0 11286 16m:47s

Artifacts

BOARDSDIR has been deprecated in favor of
EXTERNAL_BOARD_DIRS since 2021. Even the last
application should've migrated by now.
@crasbe
crasbe force-pushed the pr/remove_BOARDSDIR branch from 4e57859 to 7d56fd0 Compare September 17, 2026 09:24
@crasbe crasbe removed the State: waiting for other PR State: The PR requires another PR to be merged first label Sep 17, 2026
@crasbe

crasbe commented Sep 17, 2026

Copy link
Copy Markdown
Contributor Author

Since it's not clear yet where to go with #22722 yet, I modified the gdbinit-docker script to use RIOTBOARD instead of BOARDSDIR, which is also what makefiles/docker.inc.mk sets the volume too.

I was so free to squash and force-push the change directly.

@crasbe
crasbe enabled auto-merge September 17, 2026 09:27
@crasbe
crasbe added this pull request to the merge queue Sep 17, 2026
Merged via the queue into RIOT-OS:master with commit 761cb74 Sep 17, 2026
28 checks passed
@crasbe
crasbe deleted the pr/remove_BOARDSDIR branch September 18, 2026 10:51
@crasbe

crasbe commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

Thank you for the review :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI: Not Used AI was stated to not be used in this PR/Issue Area: build system Area: Build system Area: doc Area: Documentation Area: tests Area: tests and testing framework Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: removal Integration Process: The PR is removing a deprecated feature or API Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants