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

Version solving errors are hard to parse #3363

Open
gaaclarke opened this issue Mar 24, 2022 · 3 comments
Open

Version solving errors are hard to parse #3363

gaaclarke opened this issue Mar 24, 2022 · 3 comments
Labels
type-enhancement A request for a change that isn't a bug

Comments

@gaaclarke
Copy link

The error is very difficult to parse because proper nouns and code aren't delineated somehow.

Seen

$ ~/dev/dart-sdk/sdk/xcodebuild/DebugX64/dart run test
Resolving dependencies in /Users/aaclarke/dev/dart-sdk/sdk/pkg/kernel... (1.0s)
Because every version of expect from path depends on smith any which is forbidden, expect from path is forbidden.
So, because kernel depends on expect from path, version solving failed.

Expected

$ ~/dev/dart-sdk/sdk/xcodebuild/DebugX64/dart run test
Resolving dependencies in /Users/aaclarke/dev/dart-sdk/sdk/pkg/kernel... (1.0s)
Because every version of `expect` from `path` depends on `smith` any which is forbidden, `expect` from `path` is forbidden.
So, because `kernel` depends on `expect` from `path`, version solving failed.

That isn't quite right since I don't understand "any which is forbidden".

@vsmenon vsmenon transferred this issue from dart-lang/sdk Mar 25, 2022
@vsmenon
Copy link
Member

vsmenon commented Mar 25, 2022

Actually, not sure I triaged this correctly. I missed these are SDK packages. Are those handled differently?

@jonasfj jonasfj added the needs-info Additional information needed from the issue author label Mar 31, 2022
@jonasfj
Copy link
Member

jonasfj commented Mar 31, 2022

Can you share the pubspec.yaml you're trying to resolve, including the pubspec.yaml inside your path dependency?

I wonder if this would be fixed by #3196, in particular I'm curious if it just failed to tell you that smith does not exist.

@gaaclarke
Copy link
Author

The pubspec.yaml is in the dart sdk itself: https://github.com/dart-lang/sdk/blob/main/pkg/kernel/pubspec.yaml

The big problem is that it hard to read because there is no delineation between package names and pubspec concepts and English. For example, I didn't author the pubspec so the idea that expect is a package name wasn't even on my radar. I still have a hard time understanding what it says.

@sigurdm sigurdm added type-enhancement A request for a change that isn't a bug and removed needs-info Additional information needed from the issue author labels Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

4 participants