I didn't have anything to do with the creation of royale-sdk-description.
It looks like someone copied the flex-sdk-description generation script.
Is there any reason for that?

IMO, parsing a string in an XML file to understand the SDK contents
doesn’t make sense to me.  If the only current consumer of this file is VS
Code and it is only using the presence of the file, now would be a good
time to set the contents of royale-sdk-description in a way that is more
future-proof.  Plus, the VS Code extension can probably be adjusted if
needed.

I'd suggest something like:

<!-- Description of Apache Royale SDK -->
<royale-sdk-description>
  <product>Apache Royale</product>
  <version>0.8.0</version>
  <build>20171230</build>
  <output-targets>
    <output-target name="js" version="Ecma5Strict" />
    <output-target name="swf" version="AS3" AIR="25.0" Flash="25.0" />
  </output-targets>
</royale-sdk-description>

That way everything is an XML tag or attribute and no parsing is required.
 I'm regretting calling things "JSOnly".  I'd rather focus on what you
have instead of what you don't have.

Thoughts?
-Alex

On 12



On 12/30/17, 3:33 AM, "Piotr Zarzycki" <[email protected]> wrote:

>Here is my thoughts about that, cause I don't want to break anything. In
>ANT build royale-sdk-description is being created in target "prebuild" ->
>"
>create-description". I see following option:
>
>- Check whether we have environment variables set up:
>env.PLAYERGLOBAL_HOME
>OR env.AIR_HOME  if not make short description with "Apache Royale 0.9.0
>en_US (JS)"
>
>What do you think ? Is it sufficient ?
>
>Thanks, Piotr
>
>
>2017-12-30 12:04 GMT+01:00 Piotr Zarzycki <[email protected]>:
>
>> I think I have found where those files are generated. I will think how
>>to
>> change them, but if someone have other thoughts let me know! :)
>>
>> Thanks, Piotr
>>
>> 2017-12-30 11:59 GMT+01:00 piotrz <[email protected]>:
>>
>>> Alex,
>>>
>>> Currently in Moonshine we are using flex-sdk-description, but probably
>>> after
>>> first release of Royale we will use royale-sdk-description. Information
>>> from
>>> that file is being used to display name of the SDK.
>>>
>>> 
>>><https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fapache-
>>>royale-development.20373.n8.nabble.com%2Ffile%2F&data=02%7C01%7Caharui%4
>>>0adobe.com%7C2a19eab2ec1f4efeea9b08d54f7923c0%7Cfa7b1b5a7b34438794aed2c1
>>>78decee1%7C0%7C0%7C636502304073200067&sdata=OrjKd30bpS9CjjBH0xl9t8VlHcN6
>>>gaQQvhBwCdD%2Bjug%3D&reserved=0
>>> t1/royale_sdk_description.png>
>>>
>>> Currently I have implemented logic which showing there (JSOnly), but I
>>> thought that we can have such information in those files.
>>>
>>> May proposition having your thoughts in mind are:
>>>
>>> Royale JS: "Apache Royale 0.9.0 en_US (JS)"
>>> Royale ASJS: "Apache Royale 0.9.0 FP11.7 AIR14.0 en_US (JS, SWF)"
>>>
>>> The question is where actually this file is created ? I will try to
>>>find
>>> code, but any tip would be great.
>>>
>>> Thanks, Piotr
>>>
>>>
>>>
>>>
>>> --
>>> Sent from: 
>>>https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fapache-r
>>>oyale-development.20373.n8.nabble.com%2F&data=02%7C01%7Caharui%40adobe.c
>>>om%7C2a19eab2ec1f4efeea9b08d54f7923c0%7Cfa7b1b5a7b34438794aed2c178decee1
>>>%7C0%7C0%7C636502304073200067&sdata=cWk4J6XPAOiogm4%2BnrSw7AEYnIgtP7c4ZF
>>>U2Z184T3A%3D&reserved=0
>>>
>>
>>
>>
>> --
>>
>> Piotr Zarzycki
>>
>> Patreon: 
>>*https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.pat
>>reon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7C2a19eab2ec1f
>>4efeea9b08d54f7923c0%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C6365023
>>04073200067&sdata=trB5QDMp7NylryBjWLUQjFzP2grJoOjEk7yv3aoQKbg%3D&reserved
>>=0
>> 
>><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.pat
>>reon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7C2a19eab2ec1f
>>4efeea9b08d54f7923c0%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C6365023
>>04073200067&sdata=trB5QDMp7NylryBjWLUQjFzP2grJoOjEk7yv3aoQKbg%3D&reserved
>>=0>*
>>
>
>
>
>-- 
>
>Piotr Zarzycki
>
>Patreon: 
>*https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patr
>eon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7C2a19eab2ec1f4e
>feea9b08d54f7923c0%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C6365023040
>73200067&sdata=trB5QDMp7NylryBjWLUQjFzP2grJoOjEk7yv3aoQKbg%3D&reserved=0
><https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patr
>eon.com%2Fpiotrzarzycki&data=02%7C01%7Caharui%40adobe.com%7C2a19eab2ec1f4e
>feea9b08d54f7923c0%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C6365023040
>73200067&sdata=trB5QDMp7NylryBjWLUQjFzP2grJoOjEk7yv3aoQKbg%3D&reserved=0>*

Reply via email to