Page MenuHomePhabricator

Add a `./bin/aphlict build` workflow.
ClosedPublic

Authored by joshuaspence on May 30 2014, 9:54 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 6:55 PM
Unknown Object (File)
Sat, Dec 14, 4:57 PM
Unknown Object (File)
Sat, Dec 14, 3:26 PM
Unknown Object (File)
Mon, Dec 9, 9:29 AM
Unknown Object (File)
Fri, Dec 6, 3:56 PM
Unknown Object (File)
Thu, Dec 5, 11:55 PM
Unknown Object (File)
Tue, Dec 3, 8:26 PM
Unknown Object (File)
Sat, Nov 30, 11:10 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rPddf5412cbbfd: Add a `./bin/aphlict build` workflow.
Summary

Currently, it is a bit tricky to build the Aphlict client SWF from the ActionScript source. Provide a ./bin/aphlict build workflow that simplifies this process.

Depends on D9226.

Test Plan

Executed the workflow:

> ./bin/aphlict build
Done.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Add a `./bin/aphlict build` workflow..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.

Does this need the --strict flag too now?

One thing I really hate (not quite related to this diff) is that the build process is not idempotent.

epriestley edited edge metadata.

Summarizing from IRC:

  • mxmlc embeds some kind of timestamp or something into the binaries.
  • gzuncompress()'ing the .SWF after stripping the header gives binaries differing by 3 bytes in a nonobvious way (i.e., not an easily identifiable timestamp in an XML section or anything like that).
  • -raw-metadata '' didn't get rid of this either.
  • Least bad way forward for now seems to be just living with it.
This revision is now accepted and ready to land.Jun 7 2014, 6:33 PM
epriestley updated this revision to Diff 22479.

Closed by commit rPddf5412cbbfd (authored by @joshuaspence, committed by @epriestley).