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

README update #1176

Closed
wants to merge 1 commit into from
Closed

README update #1176

wants to merge 1 commit into from

Conversation

omarzl
Copy link

@omarzl omarzl commented Jul 17, 2024

Hello!

I followed the instructions in the README, but I got this error:

ERROR: no such package '@@[unknown repo 'swift_deps_info' requested from @@]//': The repository '@@[unknown repo 'swift_deps_info' requested from @@]' could not be resolved: No repository visible as '@swift_deps_info' from main repository

After looking at the Example files, I found out that I was missing the line "swift_deps_info" in use_repo. bazel mod tidy was not generating it.

After adding the line declare_swift_deps_info = True it was generated and I was able to compile correctly, that is why I am updating the README file

@cgrindel
Copy link
Owner

Thanks for the submission! You are correct that the README.md is not clear on when to enable to declare_swift_deps_info. I just put up #1185 providing more context. Can you take a look and provide feedback on that PR?

@cgrindel cgrindel closed this Jul 29, 2024
cgrindel added a commit that referenced this pull request Jul 30, 2024
Clarify when the `declare_swift_deps_info` attribute should be enabled
for workspaces that use `rules_swift_package_manager`.

Closes #1176.
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.

None yet

2 participants