discovery icon indicating copy to clipboard operation
discovery copied to clipboard

Added Note on loading files from /target

Open mike2ykme opened this issue 3 years ago • 1 comments

Using Windows the relative path from the cmd prompt was not working:

C:\Program Files (x86)\GNU Arm Embedded Toolchain\9 2020-q2-update\bin\arm-none-eabi-gdb.exe: warning: Couldn't determine a path for the index cache directory.
target/thumbv7em-none-eabihf/debug/led-roulette: No such file or directory. (gdb) target remote :3333
Remote debugging using :3333 warning: No executable has been specified and target does not support determining executable automatically. Try using the "file" command. 0x08000c78 in ?? () (gdb) quit A debugging session is active.

    Inferior 1 [Remote target] will be detached.

Quit anyway? (y or n) n Not confirmed. (gdb) quit A debugging session is active.

    Inferior 1 [Remote target] will be detached.

Quit anyway? (y or n) y Detaching from program: , Remote target Ending remote debugging. [Inferior 1 (Remote target) detached]


After going up a couple directories I was able to have it read the target files.


C:\Users\mike2\Documents\proging\Rust\embedded\discovery\src\05-led-roulette>cd ../..

C:\Users\mike2\Documents\proging\Rust\embedded\discovery>arm-none-eabi-gdb.exe -q target/thumbv7em-none-eabihf/debug/led-roulette C:\Program Files (x86)\GNU Arm Embedded Toolchain\9 2020-q2-update\bin\arm-none-eabi-gdb.exe: warning: Couldn't determine a path for the index cache directory. Reading symbols from target/thumbv7em-none-eabihf/debug/led-roulette...

mike2ykme avatar Sep 17 '20 05:09 mike2ykme

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @adamgreig (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

rust-highfive avatar Sep 17 '20 05:09 rust-highfive

I’ve hit this issue as well, and it appears others have too: https://github.com/rust-embedded/discovery/issues/495

I’m not sure what’s the process is for re-queueing for review, especially since there are conflicts now, but it’d be nice to see this note added.

arr-ee avatar Mar 12 '23 07:03 arr-ee

  • https://github.com/rust-embedded/discovery/issues/511

arr-ee avatar Mar 12 '23 07:03 arr-ee

If @mike2ykme is not around, @arr-ee could you open a new PR starting with the current state? A note would need to be added to both the f3 and the microbit books.

eldruin avatar Mar 27 '23 10:03 eldruin

@eldruin opened #519 that re-uses existing (and imo, better) solution. Let me know if you'd like to see a note added about why symlinks are there.

arr-ee avatar Mar 27 '23 23:03 arr-ee

@arr-ee Does the symlinks still pose a problem for windows users like OP?

eldruin avatar Mar 28 '23 06:03 eldruin

Good call, I was not paying attention to the OP’s platform.

It does not, in fact, work on Windows due to symlinks being a breaking new feature there apparently. I’ll see if we can solve this somehow without user intervention, or open a PR with a note otherwise.

arr-ee avatar Mar 28 '23 18:03 arr-ee

@eldruin opened #520 with a note, should be easier than trying to be clever about it.

arr-ee avatar Mar 28 '23 19:03 arr-ee