Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tools/build_all_headers.py exception - broken by rp2040 and rp2350 duplicate file names #2220

Open
d-c-d opened this issue Jan 30, 2025 · 2 comments
Milestone

Comments

@d-c-d
Copy link

d-c-d commented Jan 30, 2025

in the top level pico-sdk folder

$ python3 tools/build_all_headers.py  $PWD /tmp/build_all_headers.out
Traceback (most recent call last):
  File "/cygdrive/c/git/pico-sdk/tools/build_all_headers.py", line 66, in <module>
    raise Exception("Duplicate include file '{}' (found in both {} and {})".format(include_file, include_locations[include_file], include_path))
Exception: Duplicate include file 'boot_stage2/config.h' (found in both rp2040/boot_stage2/include and rp2350/boot_stage2/include)

workaround - add IGNORE_DIRS.add('rp2040') or IGNORE_DIRS.add('rp2350')

While useful, there were no examples of this tool being used.

@lurch
Copy link
Contributor

lurch commented Jan 30, 2025

It's something I added a long time ago in #319 - it's not really something that we expected anybody to actually use for anything 😂

@d-c-d
Copy link
Author

d-c-d commented Jan 30, 2025

For context, I only ran into the file tools/build_all_headers.py because I was trying to see if semihosting would be valuable in my debug context, so I was looking at all ( of the few places semihosting was mentioned :-) )

@kilograham kilograham added this to the none milestone Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants