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.

Sign up for GitHub

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

Jump to bottom

crochet.core cannot be debugged with user-level tools #16

Closed
robotlolita opened this issue Aug 15, 2021 · 1 comment
Closed

crochet.core cannot be debugged with user-level tools #16

robotlolita opened this issue Aug 15, 2021 · 1 comment
Labels
error Something is confusing, misbehaving, or harmful. s:0 minor It's annoying that this is happening, but we can live with it for now.
Milestone
First stable release

Comments

@robotlolita
Copy link
Contributor

robotlolita commented Aug 15, 2021 •
edited
Loading

This is a really unfortunate consequence of packages not supporting mutual dependencies. Everything depends on crochet.core, including crochet.debug, consequently crochet.core cannot have any dependencies. Due to the strict capabilities on open coupled with dependencies, it's not possible for core to open the debugging package.

This will require some more thinking around both debugging tools and the roles of dependencies, capabilities, and provided services.

Also marginally related to #10.
@robotlolita robotlolita added the error Something is confusing, misbehaving, or harmful. label Aug 15, 2021
@robotlolita robotlolita added this to the An Experimental Crochet milestone Aug 15, 2021
robotlolita added a commit that referenced this issue Aug 15, 2021
@robotlolita
(temp) Discloses values from arbitrary instance for now
e7f4280 
This is done to temporarily mitigate the impact of #16 while a better solution is found. Although different execution modes *should* be more lenient on disclosing values anyway
@robotlolita robotlolita added the s:0 minor It's annoying that this is happening, but we can live with it for now. label Oct 3, 2021
@robotlolita
Copy link
Contributor Author

robotlolita commented Jan 8, 2022 •
edited
Loading

This is addressed by the tracing package (#43). However, for debugging tests, it's still necessary to allow them to be ran from the playground (which is in the roadmap anyway).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
error Something is confusing, misbehaving, or harmful. s:0 minor It's annoying that this is happening, but we can live with it for now.
Projects
None yet
Milestone
First stable release
Development

No branches or pull requests
1 participant
@robotlolita