Added optional runtime path for buildController macro.

[tiagolr]

Because in openfl its usual to rename the assets paths, my app was freezing when using @:buildMacro("assets/xml/ui/MyPopup.xml")

now it works if i add the runtime (renamed) path as second argument:

@:buildMacro("assets/xml/MyPopup.xml", "ui/Popup.xml")

[ianharrigan]

Im not really sure why you would do this... Why wouldnt you just use the path that is correct?

[tiagolr]

Cause openfl renames the path, i have:

<assets path="assets/xml/ui" rename="ui" />

If i use the renamed path, the macro won't find the file, if i use the original path, the runtime will crash when doing : super("assets/xml/ui") inside constructor.

Meanwhile i started extending Popup class instead so i no longer use this method, this is used by a very particular case: Using BuildComponent macro where openfl renames the assets path.

[ianharrigan]

Hmmm... right, i see... there must be a better way of doing this... I dont like having to alias it... You maybe right and its the only way, but ill investigate if there is another way to resolve the .xml file at build time... I mean im guessing all the <asset /> entries must be available somehow? Ill have a think about it. If i cant get that info then it will have to be this method, but initially im not that keep on the second parameter.

Cheers,
Ian

[tiagolr]

Cool, I'm fine with that since i don't need it atm, its a very specific case but may happen again with someone in the future, there may be alternatives but i don't see an easy one:

• embedding the xml file on the constructor, the drawback with this one is that live reloading won't be possible if someone needs it.
• looking at compile arguments for renamed paths (don't know how this can be done)
• looking for the same asset on a different folder (a lot of work, checking all files etc... forget it :p)

[tiagolr]

Added a new commit, feel free to discard any additions, this last one to Popup its pretty useful to quickly customize / have more control over popup buttons.