[caffe-contrib] 59/362: rules: add comment

[Zhou Mo]

This is an automated email from the git hooks/post-receive script.

cdluminate-guest pushed a commit to branch master
in repository caffe-contrib.

commit e9d09a64b0a0f78cd72f370cb683de9bbed36d3b
Author: Zhou Mo <cdluminate at gmail.com>
Date:   Fri Aug 7 08:02:50 2015 +0000

    rules: add comment
---
 debian/rules | 31 +++++++++++++++++++------------
 1 file changed, 19 insertions(+), 12 deletions(-)

diff --git a/debian/rules b/debian/rules
index 3c79d80..6f125a5 100755
--- a/debian/rules
+++ b/debian/rules
@@ -7,35 +7,42 @@ export DEB_LDFLAGS_MAINT_APPEND = -Wl,--as-needed
 RULES  := debian/rules
 CONFIG := debian/config/Makefile.config
 
+# TODO: bump build system from Make to Cmake
+
 %:
 	dh $@ --parallel
 
-.PHONY: config rmconfig
-config:
-	cp $(CONFIG) ./
-rmconfig:
-	-rm Makefile.config
+# this part if for the Make build system.   
+# The Makefile of upstream checks the file  
+# Makefile.config before every action,      
+# even the 'clean'                          
+.PHONY: config rmconfig                     
+config:                                     
+	cp $(CONFIG) ./                         
+rmconfig:                                   
+	-rm Makefile.config                     
 
 override_dh_auto_clean: config
+	# we need a config file before `make clean`
 	dh_auto_clean
+	# debhelper will complain if there is a Makefile.config which
+	# is considered "dirty", so delete it
 	$(RULES) rmconfig
 
 override_dh_auto_build: config
 	dh_auto_build -- all pycaffe distribute
 	$(RULES) rmconfig
 
+# In the test phase, we need to set LD_LIBRARY_PATH properly
+# for those test ELFs linked with libcaffe.so.X
 override_dh_auto_test: config
-	# if LD_LIBRARY_PATH was not set, the test programs will not work.
-#$(MAKE) test # should run with only 1 job ?
-	dh_auto_test -- test
-	dh_auto_test -- runtest LD_LIBRARY_PATH=.build_release/lib/
-
-#binary-arch:
-#binary-indep:
+	dh_auto_test -- test # compile test excutables 
+	dh_auto_test -- runtest LD_LIBRARY_PATH=.build_release/lib/ 
 
 # Since the Distributed Binary Caffe can not play the best performance
 # of user's machines, it's good to provide a custom build target,
 # allowing users to build their local performance-optimized packages.
+# TODO: let the custom target use CMake?
 .PHONY: custom
 custom:
 	# this custom target is inspired by openblas's rules.

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/debian-science/packages/caffe-contrib.git