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

"Basic inspect it" causes system error #5579

Closed
marsell opened this issue Jan 26, 2020 · 7 comments · Fixed by #11899
Closed

"Basic inspect it" causes system error #5579

marsell opened this issue Jan 26, 2020 · 7 comments · Fixed by #11899

Comments

@marsell
Copy link

marsell commented Jan 26, 2020

exception

Replication steps:

  1. open Playground
  2. type "1"
  3. Right-click on 1 and select "Basic Inspect it"

This affects Pharo-8.0.0+build.1122.sha.bbcdf9770748d12e2c1b84f168f0232e4f918d20 (64 Bit), running on Windows.

@welcome
Copy link

welcome bot commented Jan 26, 2020

Thanks for opening your first issue! Please check the CONTRIBUTING documents for some tips about which information should be provided. You can find information of how to do a Pull Request here: https://github.com/pharo-project/pharo/wiki/Contribute-a-fix-to-Pharo

GitHub
Pharo is a dynamic reflective pure object-oriented language supporting live programming inspired by Smalltalk. - pharo-project/pharo

@JanBliznicenko
Copy link
Contributor

I confirm the issue. The main problem is the deprecation warning which should not happen and should be fixed in Pharo 8.
Because of bug fixed since build 1124 (current main image in Pharo Launcher), emergency evaluator opens instead of warning window (see closed issue #5545 ).

This is the real, still existing, problem that needs to be fixed:
2020-01-26

@estebanlm
Copy link
Member

this should be fixed now (since it was the same problem as the Warning stuff)

@JanBliznicenko
Copy link
Contributor

JanBliznicenko commented Jan 27, 2020

@estebanlm Well, the warning no longer opens emergency evaluator instead of debugger window, but the deprecation warning should not happen at all when user tries to use "Basic inspect it". That is the problem that still needs to be fixed (is not fixed in build 1124).

@MarcusDenker
Copy link
Member

I tried to fix this, but after doing it it turns out there are many many more things not working
(e.g #doItReceiver: is depurated, too).

I suggest we should remove the menu entry as this tool is completely broken and not used...

@JanBliznicenko
Copy link
Contributor

I agree - we don't need the tool to work, just remove it so that nobody would accidentaly try to use it and get these errors as a result.

@MarcusDenker
Copy link
Member

fixed as pharo-spec/NewTools#431

@MarcusDenker MarcusDenker linked a pull request Nov 16, 2022 that will close this issue
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 a pull request may close this issue.

4 participants