Skip to content

The debugger now delegate UI decisions to the backend provided by its application#667

Merged
jecisc merged 7 commits intopharo-spec:Pharo13from
StevenCostiou:15454-removing-ui-manager-dependencies
Jun 20, 2024
Merged

The debugger now delegate UI decisions to the backend provided by its application#667
jecisc merged 7 commits intopharo-spec:Pharo13from
StevenCostiou:15454-removing-ui-manager-dependencies

Conversation

@StevenCostiou
Copy link
Member

It requires the spec gtk backend to reimplement spawnNewDebugSessionForSpecBackend: and returns its own debug session.
The spec gtk debug session must implement at least deferDebuggerOpeningToBackend: that can just do aStDebugger openWithFullView.

Requires pharo-project/pharo#16015

… application.

It requires the spec gtk backend to reimplement spawnNewDebugSessionForSpecBackend: and returns its own debug session.
The spec gtk debug session must implement at least deferDebuggerOpeningToBackend: that can just do aStDebugger openWithFullView
@StevenCostiou
Copy link
Member Author

This should now be in conflict with latest merges in the debugger.
I will take care of that but not immediately (there is a PR waiting list...)

@Ducasse Ducasse changed the base branch from Pharo12 to Pharo13 June 13, 2024 07:45
@Ducasse
Copy link
Contributor

Ducasse commented Jun 13, 2024

@StevenCostiou I moved the PR to Pharo 13 and will see if I can find a way to relaunch the tests

@Ducasse Ducasse closed this Jun 13, 2024
@Ducasse Ducasse reopened this Jun 13, 2024
@StevenCostiou
Copy link
Member Author

Conflict solved.
Waiting for tests to run.

@jecisc jecisc merged commit 2306aa8 into pharo-spec:Pharo13 Jun 20, 2024
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.

4 participants