Changes

this fixes #7153 by using the cached pragmas in #allSystemPragmas

This PR just does the trivial change, we can later clean up more (e.g. allSystemPragmas senders can use Pragma allInstalled directly)

PragmaType is from Spotter but hat no users --> remove

refactor some of the usres of PragmaCollector to use Pragma allNamed: instead

There is more to be done, I would have fixed all but some of the rest are in "externally managed packages"...

add two more forwarding method... they are used when scanning the whole system for self and thisContext (which is a bit slow thus I am hesitent to add tests)

Another small cleanup related to RemoteString:

ClassOrganization
- simplify #comment
- #commentStamp - use commentSourcePointer, not commentRemoteString
- #hasComment -> use commentSourcePointer, , not commentRemoteString
- remove #commentRemoteString

Ring: #asHistorical can be much simpler accessing sourcePointer instead of remote string

PharoChangesCondenser
- implement #commentRemoteStringFor: so we do not need to expose commentRemoteString in the API of the class organizer

Fix restart. With tests.
Restarting a test re-executes the setup to refresh the execution context.
However, this is pushing old values in the operand stack before re-executing the setup, mixing old and new values and causing strange errors.
Re-executing the setup before restarting fixes it.