Fixed initialization of default notification center - its streams - Githubissues

pharo-project / pharo-launcher

Lets you manage your pharo images and download new ones

https://pharo-project.github.io/pharo-launcher/

MIT License

108 stars 46 forks source link

Fixed initialization of default notification center - its streams #607

Closed Bajger closed 1 year ago

Bajger commented 1 year ago

removed leftover method on clap context that is defined on sub-class Wrong (missing) intiializaiton of notification center streams caused issue on CI:
```
Updating VM for running sample image.
[31m#nextPutAll: was sent to nil
[0mUndefinedObject(Object)>>doesNotUnderstand: #nextPutAll:
UndefinedObject>>doesNotUnderstand: #nextPutAll:
PharoLauncherCLIConfiguration class>>configFileNotFound:
[ self configFileNotFound: arg1 ] in PharoLauncherCLIConfiguration class>>configurationFromFile:
```
This was caused by a unitialized stream, used in line: PhLNotificationCenter default errorStream in method: PharoLauncherCLIConfiguration class>>configFileNotFound: See: https://ci.inria.fr/pharo-ci-jenkins2/job/PharoLauncher-Pipeline/job/feature%252Fcmd-line/19/console