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

Changes to make the examples run on Fresh Install #3

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Willmac16
Copy link

I found a couple of things I had to change to get the repo to run on my fresh install on my machine (also an intel machine, not an M series).

  • thread_execution_width is now deprecated and causes compiler errors. All references to it have been removed, and the store array has been reindexed.
  • AFAIK standard brew install paths have changed from /opt/homebrew/opt/ to /usr/local/opt/

Improvements/non-required changes I made

  • I added Launch Targets that point at all of the top level tasks
  • I edited the tasks.json file to be json compliant so I could jq it while trying to work out a bug that was actually just invoking a target that relied on relative paths from the wrong directory.

Let me know if my changes sound reasonable, or if you would like to only let through some/a modification to them. Also, I would recommend adding a note about the launch targets displaying program output to the Debug Console (at least on my config) because that was very non-obvious to me.

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.

1 participant