On 20 Jan 2014, at 16:04, Tudor Girba <tudor@tudorgirba.com> wrote:
I think we should use Metacello versions for this, no?
Yes, that is one option, that is what I meant by 'having 2 code versions'. But it is a lot of hassle for one little platform difference. Maybe the project already has some platform specific code, then it would be easier to play along.
Doru
Sent from my iPhone
On Jan 20, 2014, at 15:39, Sven Van Caekenberghe <sven@stfx.eu> wrote:
On 20 Jan 2014, at 15:30, Tudor Girba <tudor@tudorgirba.com> wrote:
Ohh, I misread the mail. Sorry.
Sebastian: I just checked, and you have commit rights to PharoExtras, so you could just commit your fix to the RFB project.
But if he does that, RFB will no longer work on 2.0
And for such a small difference, having two code versions is silly, so the only solution is a dynamic hack (try one, if failed, try the other).
Doru
On Mon, Jan 20, 2014 at 3:27 PM, Sven Van Caekenberghe <sven@stfx.eu> wrote:
On 20 Jan 2014, at 15:26, Tudor Girba <tudor@tudorgirba.com> wrote:
Thank you.
Could you please open an issue and propose a slice for this?
RFB is an external package, right ?
Doru
On Mon, Jan 20, 2014 at 1:57 PM, Sebastian Sastre <sebastian@flowingconcept.com> wrote: Hi there,
this is causing a debugger to be open in the deprecated method
RFBSession>>desktopName "Answer the name of the desktop, constructed from the image, host and platform names." "RFBSession new desktopName"
^String streamContents: [:stream | stream nextPutAll: SmalltalkImage current imageName asFileReference basename; nextPut: $@; nextPutAll: server localHostName; nextPutAll: ' ['; nextPutAll: OSPlatform platformName; space; nextPutAll: OSPlatform osVersion; nextPut: $]. interactive ifFalse: [stream nextPutAll: ' - view only']]
fix:
RFBSession>>desktopName "Answer the name of the desktop, constructed from the image, host and platform names." "RFBSession new desktopName"
^String streamContents: [:stream | stream nextPutAll: SmalltalkImage current imageName asFileReference basename; nextPut: $@; nextPutAll: server localHostName; nextPutAll: ' ['; nextPutAll: Smalltalk os name; space; nextPutAll: Smalltalk os version; nextPut: $]. interactive ifFalse: [stream nextPutAll: ' - view only']]
-- www.tudorgirba.com
"Every thing has its own flow"
-- www.tudorgirba.com
"Every thing has its own flow"