I have made changes locally which I believe address the things I was concerned about. Can someone please advise : should these changes be pushed to the 0.9.8 release branch or to develop? I think it is supposed to be the release branch, and they will later be merged back to develop, but I just want to verify that before I do anything. I should be able to push these tonight once I can be sure which branch I should commit them to.
The changes do the following: -For 3 zero-byte font (.ser files) and 1 binary vector image file (.afdesign in frameworks\themes\JewelTheme\src\main\resources\afdesign) - exclude these from RAT reporting in the ApproveRoyale script - remove unnecessary noise -Include compiler's RELEASE_NOTES.md in release staging -update top level RELEASE_NOTES to include a 0.9.8 section (summary of asjs and compiler changes) -update royale-typedefs RELEASE_NOTES to include a 0.9.8 section ('no major changes') -minor changes to asjs RELEASE_NOTES and compiler RELEASE_NOTES (one addition to compiler, mostly fixing spelling errors/typos for the remainder) Thanks, Greg On Thu, Aug 19, 2021 at 12:21 AM Piotr Zarzycki <piotrzarzyck...@gmail.com> wrote: > Hi Josh, > > Thanks for explanation. My expectation was just that if I run approval > script with switch for JS only tests - I would get JS only binaries after > final build. - This is not a blocker but everyone need to be aware that > build producing JS-SWF version. > > wt., 17 sie 2021 o 18:46 Josh Tynjala <joshtynj...@bowlerhat.dev> > napisał(a): > > > In your screenshot, the left side looks like a JS-only distribution > (except > > for the extra player and air directories that are there, for some reason) > > because frameworks/libs contains KeepsFBFromHanging.swc, while the right > > side looks like a JS+SWF distribution with all of the framework SWCs. > > > > It's worth mentioning that a freshly downloaded JS+SWF binary > distribution > > is not currently supposed to contain playerglobal.swc, and you are > expected > > to add it manually (or another tool like Moonshine would do that). It's > > quite possible that the ApproveRoyale script doesn't do that for you, and > > you still need to do it manually. > > > > Technically, we are able to build a playerglobal.swc/airglobal.swc from > the > > asdoc XML using the playerglobalc tool I created, but that is not yet > > included in our distribution. > > > > -- > > Josh Tynjala > > Bowler Hat LLC <https://bowlerhat.dev> > > > > > > On Tue, Aug 17, 2021 at 7:59 AM Piotr Zarzycki < > piotrzarzyck...@gmail.com> > > wrote: > > > > > I went trough Approval script and everything goes fine. I have even > > tested > > > Maven artifacts to build our Apache Royale application - this one also > > went > > > fine. I have faced some wall when I wanted to try binary distribution > > which > > > was produced after build in Moonshine. When I try build Royale > > application > > > I'm getting following error: > > > > > > "This SDK does not contains playerglobal.swc in > > > frameworks/libs/player/11.7/playerglobal.swc." > > > > > > Folder frameworks/libs - contains bunch of swcs "Collections.swc", > > > "Basic.swc" etc. > > > > > > When I download nightly produced by Maven [1] I see in folder > > > frameworks/libs/: > > > > > > frameworks/libs/air > > > frameworks/libs/player/20.0/playerglobal.swc > > > > > > Screenshot [2] > > > > > > Why they are so big differences ? I used following command to run > > approval > > > script: ant -e -f ApproveRoyale.xml -Drelease.version=0.9.8 -Drc=6 > > > > > > Any thoughts on this ? > > > > > > [1] https://nightlies.apache.org/Royale/Royale-asjs/ > > > [2] https://paste.pics/b034e5b11b9342184899e8f29b9887d8 > > > > > > Thanks, > > > Piotr > > > > > > pon., 16 sie 2021 o 23:47 Andrew Wetmore <cottag...@gmail.com> > > napisał(a): > > > > > > > This is great news > > > > > > > > On Mon., Aug. 16, 2021, 3:22 p.m. Josh Tynjala, < > > > joshtynj...@bowlerhat.dev > > > > > > > > > wrote: > > > > > > > > > Thanks again for the hard work, Yishay! > > > > > > > > > > I ran the approval script successfully, I confirmed that the > > > royale-asjs > > > > > works correctly for powering code intelligence in vscode-as3mxml, > > and I > > > > can > > > > > compile and run asconfigc for Node.js. Looks good to me. > > > > > > > > > > -- > > > > > Josh Tynjala > > > > > Bowler Hat LLC <https://bowlerhat.dev> > > > > > > > > > > > > > > > On Mon, Aug 16, 2021 at 1:15 AM <apacheroyal...@gmail.com> wrote: > > > > > > > > > > > This is the discussion thread. > > > > > > > > > > > > Thanks, > > > > > > Yishay Weiss > > > > > > > > > > > > > > > > > > -- > > > > > > Piotr Zarzycki > > > > > > > > -- > > Piotr Zarzycki >