'make server' should not always run yuidoc

'make server' should not always run yuidoc I discovered that 'make server' always ran yuidoc. This was because, in part, the automatically generated FILES list included inappropriate files. There is another factor that affects this that I did not identify, but fixing the first problem is sufficient. Fixing the problem specified the JS files more tightly, which revealed a few more lint changes. I did a fly-by in the README. https://code.launchpad.net/~gary/juju-gui/makefile/+merge/132302 (do not edit description out of merge proposal)

[gary.poster, 2012-10-31 11:25:15]

Please take a look.

[gary.poster, 2012-10-31 11:31:02]

*** Submitted: 'make server' should not always run yuidoc I discovered that 'make server' always ran yuidoc. This was because, in part, the automatically generated FILES list included inappropriate files. There is another factor that affects this that I did not identify, but fixing the first problem is sufficient. Fixing the problem specified the JS files more tightly, which revealed a few more lint changes. I did a fly-by in the README. R= CC= https://codereview.appspot.com/6823057

[benji, 2012-10-31 12:43:02]

Looks good. I had one small question. https://codereview.appspot.com/6823057/diff/3001/Makefile File Makefile (right): https://codereview.appspot.com/6823057/diff/3001/Makefile#newcode1 Makefile:1: JSFILES=$(shell bzr ls -RV -k file | grep -E -e '.+\.js(on)?$$|generateTemplates$$' | grep -Ev -e '^manifest\.json$$' -e '^test/assets/' -e '^app/assets/javascripts/reconnecting-websocket.js$$' -e '^server.js$$') Do we want to include any assets? It seems to me that we want to exclude '/assets/' in all its forms.

[teknico, 2012-10-31 14:27:28]

Looks good to me. Thanks for the single quotes. :-)