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

Startup Problem on Windows 10 64bits #572

Closed yann-gael closed 1 year ago

yann-gael commented 2 years ago

Describe the bug I just downloaded pharo-launcher for Windows and installed it. When starting the launcher, there's an exception.

To Reproduce Steps to reproduce the behavior:

  1. Download pharo-launcher
  2. Install
  3. Run
  4. See error

Version information:

Here is the error in "stderr_51830228148831905"

 Process>>doTerminationFromAnotherProcess Process>>terminate PhLImageDescriptionUpdater>>terminateProcess PhLDeploymentScript class>>resetImageDescriptionUpdater PhLDeploymentScript class>>doAll PhLDeploymentScript class>>developmentEnvironmentEnabled: SettingDeclaration>>realValue: SettingNode>>realValue: [ aSettingNode realValue: value ] in StoredSetting(AbstractStoredSetting)>>updateSettingNode: in Block: [ aSettingNode realValue: value ] FullBlockClosure(BlockClosure)>>on:do: StoredSetting(AbstractStoredSetting)>>updateSettingNode: [ :storedSetting | storedSetting updateSettingNode: eachSettingNode ] in [ :eachSettingNode | storedSettings detect: [ :eachStoredSetting | eachStoredSetting isForSettingNode: eachSettingNode ] ifFound: [ :storedSetting | storedSetting updateSettingNode: eachSettingNode ] ifNone: [ "ignore it" ] ] in SystemSettingsPersistence>>updateSettingNodes: in Block: [ :storedSetting | storedSetting updateSettingNode...etc... FullBlockClosure(BlockClosure)>>cull: [ :each | (aBlock value: each) ifTrue: [ ^ foundBlock cull: each ] ] in Array(Collection)>>detect:ifFound:ifNone: in Block: [ :each | ... Array(SequenceableCollection)>>do: Array(Collection)>>detect:ifFound:ifNone: [ :eachSettingNode | storedSettings detect: [ :eachStoredSetting | eachStoredSetting isForSettingNode: eachSettingNode ] ifFound: [ :storedSetting | storedSetting updateSettingNode: eachSettingNode ] ifNone: [ "ignore it" ] ] in SystemSettingsPersistence>>updateSettingNodes: in Block: [ :eachSettingNode |... OrderedCollection>>do: SystemSettingsPersistence>>updateSettingNodes: SystemSettingsPersistence>>updateSettingNodes PhLSettingTree(SettingTree)>>updateSettingNodes PhLSettingBrowser>>loadSettings PhLSettingBrowser class>>launcherStartUp [ :c | c ifNotNil: [ c launcherStartUp ] ] in PhLStartupManager class>>startUp: in Block: [ :c | c ifNotNil: [ c launcherStartUp ] ] SortedCollection(OrderedCollection)>>do: PhLStartupManager class>>startUp: ClassSessionHandler>>startup: [ :each | each startup: isImageStarting ] in WorkingSession>>runStartup: in Block: [ :each | each startup: isImageStarting ] [aBlock value: each] in [ :each | [aBlock value: each] on: Exception do: [ :error | self errorHandler handleError: error] ] in WorkingSession>>runList:do: in Block: [aBlock value: each] FullBlockClosure(BlockClosure)>>on:do: 

yann-gael commented 2 years ago

Okay, it seems that the problem is that in C:\Users\\AppData\Roaming, there is a folder called pharo-launcher that contained "old" files that broke the launcher.

yann-gael commented 2 years ago

Here is the folder that prevented the launcher to start, it used to be called "pharo-launcher": OLD pharo-launcher.zip

yann-gael commented 2 years ago

It seems that PharoLauncher insists in installing things in C:\Users\\Documents\Pharo and it happens that my user name has spaces and accents. I cannot change the location from Settings > Pharo Launcher, when I try, it throws an exception.

demarey commented 2 years ago

Hi Yann-Gaël,

I have a fix for the change of image directory location (pb when moving to spec2). I cannot commit it yet (some pb in iceberg). I was able to change the image directory with one containing non-ascii characters.

For the image launch problem, I was able to reproduce the pb: the current VM does not allow to run images with non-ascii characters from the command line. we look to fix this.

yann-gael commented 2 years ago
Hi Christophe!

Thanks for the help, let me know when the fixes are integrated, I could test them :-)

Best,
Yann

On 2022-06-15 05:52, demarey wrote:

Attention This email originates from outside the concordia.ca domain. // Ce courriel provient de l'exterieur du domaine de concordia.ca

Hi Yann-Gaël,

I have a fix for the change of image directory location (pb when moving to spec2). I cannot commit it yet (some pb in iceberg). I was able to change the image directory with one containing non-ascii characters.

For the image launch problem, I was able to reproduce the pb: the current VM does not allow to run images with non-ascii characters from the command line. we look to fix this.

— Reply to this email directly, view it on GitHub <https://github.com/pharo-project/pharo-launcher/issues/572#issuecomment-1156254596 or unsubscribe https://github.com/notifications/unsubscribe-auth/ACLN726QMT6KSGIY4YEBGLDVPGRUDANCNFSM5YX26L5Q> You are receiving this because you authored the thread.Message ID: @.***>

-- Yann-Gaël Guéhéneuc Ph.D. and eng. / Ph.D. et ing. Full Professor / Professeur titulaire Chaire de recherche tier 1 sur le Génie logiciel empirique pour l'IdO Research Chair Tier 1 on Empirical Software Engineering for the IoT CSSE, Concordia University 1-514-848-2424 #7110 (Phone / Tél.) 1455 De Maisonneuve Blvd West 1-514-848-2830 (Fax / Téléc.) Montréal, QC, H3G 1M8, Canada www.ptidej.net

demarey commented 1 year ago

@tesonep