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

Fix flash target in contrib/Makefile #262

Merged
merged 1 commit into from
Oct 25, 2024
Merged

Fix flash target in contrib/Makefile #262

merged 1 commit into from
Oct 25, 2024

Conversation

agren
Copy link
Contributor

@agren agren commented Sep 6, 2024

Description

Fixes an issue with running make -C contrib flash (Command found in the dev guide). The command exits with the following error:
Error: workdir "/build/hw/application_fpga" does not exist on container

Fixed by setting the working directory to /build.

Previously the working directory was set to /build/hw/application_fpga. But that path does not exist since hw/application_fpga is already mounted to /build. Thus resulting in an error.

Type of change

  • Bugfix (non breaking change which resolve an issue)

Submission checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my changes
  • I have tested and verified my changes on target
  • My changes are well written and CI is passing
  • I have squashed my work to relevant commits and rebased on main for linear history

@agren agren requested a review from mchack-work September 6, 2024 08:20
@agren agren self-assigned this Sep 6, 2024
@dehanj dehanj requested a review from jthornblad October 9, 2024 11:43
Setting the working directory to a directory that exist.

Previously the working directory was set to /build/hw/application_fpga.
But that path does not exist since hw/application_fpga was mounted to
/build. Thus resulting in an error:
'Error: workdir "/build/hw/application_fpga" does not exist on
container'
@agren agren force-pushed the fix_flash_make_target branch from b7d1c75 to a32ecad Compare October 25, 2024 11:00
@agren agren merged commit a32ecad into main Oct 25, 2024
5 checks passed
@agren agren deleted the fix_flash_make_target branch October 25, 2024 11:08
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

Successfully merging this pull request may close these issues.

2 participants