yellow team mailing list archive

[Matthew Scott <matthew.scott@xxxxxxxxxxxxx>]

Matthew Scott has proposed merging lp:~makyo/juju-gui/make-clean-1090086 into lp:juju-gui.

Requested reviews:
  Juju GUI Hackers (juju-gui)
Related bugs:
  Bug #1090086 in juju-gui: "make clean should get rid of links in app/assets/javascripts"
  https://bugs.launchpad.net/juju-gui/+bug/1090086

For more details, see:
https://code.launchpad.net/~makyo/juju-gui/make-clean-1090086/+merge/139967

make clean to remove links

The clean target now removes symlinks in app/assets/javascripts.

https://codereview.appspot.com/6945057/

-- 
https://code.launchpad.net/~makyo/juju-gui/make-clean-1090086/+merge/139967
Your team Juju GUI Hackers is requested to review the proposed merge of lp:~makyo/juju-gui/make-clean-1090086 into lp:juju-gui.

=== modified file 'Makefile'
--- Makefile	2012-12-13 18:42:03 +0000
+++ Makefile	2012-12-14 17:05:29 +0000
@@ -343,6 +343,7 @@
 
 clean:
 	rm -rf build build-debug build-prod
+	find app/assets/javascripts/ -type l | xargs rm -rf
 
 clean-deps:
 	rm -rf node_modules virtualenv