Needs fixing. Running "make test-prod" produces 60 failures here, which is admittedly a nice improvement down from 79, but still not enough. :-)
You most likely don't see these failures: I'd like to understand why I do. There's one more thing I'd like to clarify, see comment inline. https://codereview.appspot.com/6947057/diff/1/Makefile File Makefile (right): https://codereview.appspot.com/6947057/diff/1/Makefile#newcode293 Makefile:293: cp -r --parents */assets "$(PWD)/build-$(1)/juju-ui/assets/") There's been some back and forth on this. You originally put in the "cp" instruction; then I wanted to have symlinks in the build directories rather than copies, so I spent some effort to come up with the "find" command that created the right links in the right places, and did not see requests for them return 404. Your experience has apparently been different. I'd like to understand: 1) why; 2) if there's value enough, in having symlinks instead of copies, to worry about this. https://codereview.appspot.com/6947057/ -- https://code.launchpad.net/~benji/juju-gui/bug-1088507/+merge/140020 Your team Juju GUI Hackers is requested to review the proposed merge of lp:~benji/juju-gui/bug-1088507 into lp:juju-gui. -- Mailing list: https://launchpad.net/~yellow Post to : [email protected] Unsubscribe : https://launchpad.net/~yellow More help : https://help.launchpad.net/ListHelp
