I can queue it...

On Fri, Jun 21, 2013 at 11:06 AM, jannik laval <jannik.la...@gmail.com>wrote:

> Ok, so, can you commit them ?
>
> Jannik
>
>
> 2013/6/21 Guillermo Polito <guillermopol...@gmail.com>
>
>> To me they look ok... :)
>>
>> as I see here [1], you have no commit grants in the VMMaker repo and you
>> have no user either in source.squeak.org.
>>
>> [1] http://source.squeak.org/@f1QSsdar8e_PgYbD/R72is4gY
>>
>>
>> On Fri, Jun 21, 2013 at 10:32 AM, jannik laval <jannik.la...@gmail.com>wrote:
>>
>>> Nop, I just send the mail.
>>> I prefer you review the proposed changes :)
>>>
>>> Jannik
>>>
>>>
>>> 2013/6/21 Guillermo Polito <guillermopol...@gmail.com>
>>>
>>>> Jannik, did you commit them? :)
>>>>
>>>>
>>>> On Thu, Jun 20, 2013 at 11:23 AM, jannik laval 
>>>> <jannik.la...@gmail.com>wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> Making a try with "PharoIPhoneBuilder buildIPhoneSimulator."
>>>>> I had to change some things:
>>>>>
>>>>> - The method String>>getEnclosedExpressionFrom: aStream does not
>>>>> exists in Pharo2.0
>>>>> I took it from Pharo1.4, but probably, you have a better solution.
>>>>> - There are some problems in the method
>>>>> "PharoIPhoneSimulator>>setExtraTargetProperties: maker" due to
>>>>> FileDirectory>>FileSystem changes. Here is the version of the method I 
>>>>> did,
>>>>> probably you want to review it (in bold is what I changed):
>>>>>
>>>>> ---
>>>>> PharoIPhoneSimulator>>setExtraTargetProperties: maker
>>>>> "super setExtraTargetProperties: maker."
>>>>>
>>>>> | precompiledHeaders plist |
>>>>>  precompiledHeaders := (self executableName, '_Prefix.pch').
>>>>>  *(maker buildDir / precompiledHeaders) writeStream *
>>>>> nextPutAll: self precompiledHeaders;
>>>>> close.
>>>>>  maker addXCodeProperty: 'GCC_PREFIX_HEADER' value: '${buildDir}/',
>>>>> precompiledHeaders.
>>>>>  self setCommonProperties: maker.
>>>>>
>>>>> " generated and add Info.plist file "
>>>>>  plist := self plistFile.
>>>>> *(maker buildDir / 'Info.plist') writeStream*
>>>>> nextPutAll: plist;
>>>>>  close.
>>>>>
>>>>> maker addProperty: 'MACOSX_BUNDLE_INFO_PLIST' value:
>>>>> '${buildDir}/Info.plist'.
>>>>>  maker
>>>>> cmd: 'set_target_properties'
>>>>> params: self executableName, ' PROPERTIES COMPILE_FLAGS "-include \"',
>>>>> (self buildDir / self prefixHeaderFileName) fullName, '\""'.
>>>>>
>>>>> maker set: 'CMAKE_XCODE_ATTRIBUTE_CODE_SIGN_IDENTITY'
>>>>> toString: self codeSignIdentity.
>>>>>  maker set: 'CMAKE_XCODE_ATTRIBUTE_TARGETED_DEVICE_FAMILY'
>>>>> toString: self targetedDeviceFamilyAsString.
>>>>>  maker set: 'CMAKE_XCODE_ATTRIBUTE_IPHONEOS_DEPLOYMENT_TARGET'
>>>>> to: self deploymentTargetVersion.
>>>>>  maker set: 'CMAKE_XCODE_ATTRIBUTE_CONFIGURATION_BUILD_DIR'
>>>>> toString: '$(BUILD_DIR)/$(CONFIGURATION)'.
>>>>>  maker set: 'CMAKE_XCODE_ATTRIBUTE_CONFIGURATION_TEMP_DIR'
>>>>> toString: '$(BUILD_DIR)/$(CONFIGURATION)'.
>>>>>
>>>>> maker set: 'EXECUTABLE_OUTPUT_PATH' toString:* self outputDir fullName
>>>>> *.
>>>>>  maker puts:
>>>>> 'set_source_files_properties(${srcVMDir}/gcc3x-cointerp.c PROPERTIES
>>>>> COMPILE_FLAGS -O0)'
>>>>> ---
>>>>>
>>>>> After these two changes, all seems working.
>>>>> Thank you for the work.
>>>>>
>>>>> Jannik
>>>>>
>>>>>
>>>>> 2013/6/19 Igor Stasenko <siguc...@gmail.com>
>>>>>
>>>>>> On 19 June 2013 16:34, Guillermo Polito <guillermopol...@gmail.com>
>>>>>> wrote:
>>>>>> > Yes, but right now moving from FileDirectory to FileSystem will
>>>>>> make harder
>>>>>> > the merge between VMMaker branches.
>>>>>> > And cleaning and removing stuff will make it worse...
>>>>>> >
>>>>>> > So either we keep the mess or we lose the ability to easily merge :)
>>>>>> >
>>>>>>
>>>>>> i don't think Cog will ever run on riscos in future.. (while on ARM,
>>>>>> in general, it will).
>>>>>> but it definitely won't use RiscOSVMMaker.
>>>>>> I see no reason changing that, especially that Eliot made an effort to
>>>>>> use single VMMaker class suitable to be used for building Cog on all
>>>>>> platforms.
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Best regards,
>>>>>> Igor Stasenko.
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>>
>>>>> ~~Dr. Jannik Laval~~
>>>>> École des Mines de Douai
>>>>> Enseignant-chercheur
>>>>> http://www.jannik-laval.eu
>>>>>
>>>>
>>>>
>>>
>>>
>>> --
>>>
>>> ~~Dr. Jannik Laval~~
>>> École des Mines de Douai
>>> Enseignant-chercheur
>>> http://www.jannik-laval.eu
>>>
>>
>>
>
>
> --
>
> ~~Dr. Jannik Laval~~
> École des Mines de Douai
> Enseignant-chercheur
> http://www.jannik-laval.eu
>

Reply via email to