-
Notifications
You must be signed in to change notification settings - Fork 229
Adding a dependency override to a package with a binary borks pub run #8
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
Comments
<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3 The title says "dev dependency" but step 4 says "dependency_override". Which is it? |
<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3 Changed the title to: "Adding a dependency override to a package with a binary borks pub run". |
<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3 I don't think this is actually specific to dependency overrides; I can reproduce it with a normal dependency, as long as the second dependency is a path dependency. |
<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3 Fixed by dart-lang/sdk@60d2af7. Added Fixed label. |
<img src="https://avatars.githubusercontent.com/u/17034?v=3" align="left" width="96" height="96"hspace="10"> Issue by kevmoo
Originally opened as dart-lang/sdk#23113
Expected:
run 'test' from the override dir
Actual: 'test' still runs from the created cache
work-around: delete '.pub' from the project directory
The text was updated successfully, but these errors were encountered: