Is fileout on the change sorter working in Pharo 70?

It looks like the problem is here but I cannot find it.

CodeExporter>> writeSourceCodeFrom: aStream baseName: baseName isSt: stOrCsFlag

| extension fileName  outputStream |
self halt.
extension := stOrCsFlag ifTrue: ['.st']  ifFalse: ['.cs'].
fileName := baseName, extension.
fileName := FileSystem disk checkName: fileName fixErrors: true.
outputStream := (File named: fileName) writeStream.
(ZnCrPortableWriteStream on: (ZnCharacterWriteStream
on: outputStream
encoding: 'utf8')) nextPutAll: aStream contents.

outputStream close.

self inform: 'Filed out to: ', String cr, fileName.

On Wed, Nov 8, 2017 at 9:38 PM, Stephane Ducasse
<[hidden email]> wrote:
> Ah thanks we should fix this.
>
> On Wed, Nov 8, 2017 at 9:31 PM, Christian Kellermann
> <[hidden email]> wrote:
>> * Stephane Ducasse <[hidden email]> [171108 21:29]:
>>> Hi
>>>
>>> I'm trying to rescue some of my work (that I cannot save in MC nor in
>>> Iceberg because the system
>>> does not see it - super funny).
>>>
>>> I can browse the change using the changesorter when I do fileout
>>> the system tells me that the file it saved but I cannot find it on my disc.
>>> Apparently none of the fileout is working in the changesorter. :(
>>
>> I have successfully filed out code with epicea but not changesorter.
>>
>> HTH,
>>
>> Christian
>>
>> --
>> May you be peaceful, may you live in safety, may you be free from
>> suffering, and may you live with ease.
>>

On 08-11-17 22:03, Stephane Ducasse wrote:

File @ /Users/ducasse/Documents/Pharo/vms/70-x86/Pharo.app/Contents/MacOS

Yes, this is looks like one of the problems I have with PharoLauncher directories. It is unclear to me what is kept with the vm and what with the image. I asked about it in users

Stephan

On Thu, Nov 9, 2017 at 10:35 AM, Denis Kudriashov <[hidden email]> wrote:

Hi

2017-11-08 21:53 GMT+01:00 Stephane Ducasse <[hidden email]>:

It looks like the problem is here but I cannot find it.

CodeExporter>> writeSourceCodeFrom: aStream baseName: baseName isSt: stOrCsFlag

| extension fileName  outputStream |
self halt.
extension := stOrCsFlag ifTrue: ['.st']  ifFalse: ['.cs'].
fileName := baseName, extension.
fileName := FileSystem disk checkName: fileName fixErrors: true.
outputStream := (File named: fileName) writeStream.
(ZnCrPortableWriteStream on: (ZnCharacterWriteStream
on: outputStream
encoding: 'utf8')) nextPutAll: aStream contents.

outputStream close.

self inform: 'Filed out to: ', String cr, fileName.

Try rewrite code to FileSystem. I think we should never use File directly:

extension := stOrCsFlag ifTrue: ['st']  ifFalse: ['cs'].

FileLocator home / baseName , extension writeStreamDo: [:s |

(ZnCrPortableWriteStream on: (ZnCharacterWriteStream

on: s

encoding: 'utf8')) nextPutAll: aStream contents].

We should choose what to use instead of #home in that case. 

I just discussed a point with Denis, just replicating it here.

We should not **blindly** rewrite everything to use FileSystem. Because that may break dependencies in the system.

For example, CodeExporter is used by Monticello.

Should monticello depend on FileSystem or not?

I'm not saying monticello should or should not depend on it. Just saying that we should ask ourselves the question :)

 

I wondering does File is actually used by FileSystem?

It will ;)

 

Should not we remove it from system?

No! File is exactly the low level File representation, that FileSystem should use.

 

 

On Wed, Nov 8, 2017 at 9:38 PM, Stephane Ducasse
<[hidden email]> wrote:
> Ah thanks we should fix this.
>
> On Wed, Nov 8, 2017 at 9:31 PM, Christian Kellermann
> <[hidden email]> wrote:
>> * Stephane Ducasse <[hidden email]> [171108 21:29]:
>>> Hi
>>>
>>> I'm trying to rescue some of my work (that I cannot save in MC nor in
>>> Iceberg because the system
>>> does not see it - super funny).
>>>
>>> I can browse the change using the changesorter when I do fileout
>>> the system tells me that the file it saved but I cannot find it on my disc.
>>> Apparently none of the fileout is working in the changesorter. :(
>>
>> I have successfully filed out code with epicea but not changesorter.
>>
>> HTH,
>>
>> Christian
>>
>> --
>> May you be peaceful, may you live in safety, may you be free from
>> suffering, and may you live with ease.
>>

--

Guille Polito Research Engineer Centre de Recherche en Informatique, Signal et Automatique de Lille CRIStAL - UMR 9189 French National Center for Scientific Research - http://www.cnrs.fr Web: http://guillep.github.io Phone: <a href="tel:+33%206%2052%2070%2066%2013" value="+33652706613" target="_blank">+33 06 52 70 66 13