The Inbox: CommandLine-ct.19.mcz

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

The Inbox: CommandLine-ct.19.mcz

commits-2
Christoph Thiede uploaded a new version of CommandLine to project The Inbox:
http://source.squeak.org/inbox/CommandLine-ct.19.mcz

==================== Summary ====================

Name: CommandLine-ct.19
Author: ct
Time: 24 January 2021, 9:24:27.349384 pm
UUID: 14c388f7-bb9a-874f-9759-e2e6e8b27957
Ancestors: CommandLine-TheresaHMartenK.17

Don't quit the VM when a warning was raised in a command-line context.

The intention of a Warning is to "be brought to the attention of the user", but is only a kind of Notification ("if it is not handled, it will pass by without effect"), so in contrast to an error ("the current computation is terminated"), there is no need to interrupt the entire execution.
Au contraire, this has often been a very tedious behavior; for example, if you use a deprecated selector in smalltalkCI, it is a good thing to be notified about this incident, but it is a bad thing to interrupt the entire execution just because such an incident has been reported. Note that smalltalkCI is only one example of many; any headless server image should not crash upon a simple deprecation warning for the same reason.

=============== Diff against CommandLine-TheresaHMartenK.17 ===============

Item was changed:
  ----- Method: CommandLineToolSet class>>handleWarning: (in category 'debugging - handlers') -----
  handleWarning: aWarning
 
+ aWarning printVerboseOn: FileStream stderr.
+ FileStream stderr flush.!
- self handleError: aWarning.!