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 ASAP7 for reworked upstream repo structure. #211

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

mithro
Copy link
Member

@mithro mithro commented Nov 7, 2023

No description provided.

@mithro mithro requested a review from QuantamHD November 7, 2023 18:56
@mithro mithro merged commit 2d6c21f into hdl:main Nov 7, 2023
4 checks passed
@mithro mithro deleted the temp-fix-asap7 branch November 7, 2023 21:40
sha256 = "fd9e99f6ccb9e946755f9bc444abefbdd1eedb32c372c56dcacc7eb486aed178",
)

load("@rules_7zip//:setup.bzl", "setup_7zip")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe this should go into dependency support instead? so that downstream of this package don't have to do the same thing in their workspace?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand bazel enough to know when something should go into dependency_support verse WORKSPACE. @QuantamHD - Do you have any guidance?

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.

3 participants