tools should have nonzero error exit codes

diff --git a/tools/compute_image_mean.cpp b/tools/compute_image_mean.cpp
index b803ab2..7cf5fe5 100644 (file)
--- a/tools/compute_image_mean.cpp
+++ b/tools/compute_image_mean.cpp
@@ -18,7 +18,7 @@ int main(int argc, char** argv) {
   ::google::InitGoogleLogging(argv[0]);
   if (argc != 3) {
     LOG(ERROR) << "Usage: compute_image_mean input_leveldb output_file";
-    return(0);
+    return 1;
   }
 
   leveldb::DB* db;

diff --git a/tools/convert_imageset.cpp b/tools/convert_imageset.cpp
index 219c131..82edae4 100644 (file)
--- a/tools/convert_imageset.cpp
+++ b/tools/convert_imageset.cpp
@@ -38,7 +38,7 @@ int main(int argc, char** argv) {
         " RANDOM_SHUFFLE_DATA[0 or 1]\n"
         "The ImageNet dataset for the training demo is at\n"
         "    http://www.image-net.org/download-images\n");
-    return 0;
+    return 1;
   }
   std::ifstream infile(argv[2]);
   std::vector<std::pair<string, int> > lines;

diff --git a/tools/device_query.cpp b/tools/device_query.cpp
index 6b120ba..5040b8e 100644 (file)
--- a/tools/device_query.cpp
+++ b/tools/device_query.cpp
@@ -10,7 +10,7 @@ using namespace caffe;  // NOLINT(build/namespaces)
 int main(int argc, char** argv) {
   if (argc > 2) {
     LOG(ERROR) << "device_query [device_id=0]";
-    return 0;
+    return 1;
   }
   if (argc == 2) {
     LOG(INFO) << "Querying device_id=" << argv[1];

diff --git a/tools/finetune_net.cpp b/tools/finetune_net.cpp
index c7e80c9..db96b02 100644 (file)
--- a/tools/finetune_net.cpp
+++ b/tools/finetune_net.cpp
@@ -16,7 +16,7 @@ int main(int argc, char** argv) {
   ::google::InitGoogleLogging(argv[0]);
   if (argc != 3) {
     LOG(ERROR) << "Usage: finetune_net solver_proto_file pretrained_net";
-    return 0;
+    return 1;
   }
 
   SolverParameter solver_param;

diff --git a/tools/net_speed_benchmark.cpp b/tools/net_speed_benchmark.cpp
index bb19b41..f52aac9 100644 (file)
--- a/tools/net_speed_benchmark.cpp
+++ b/tools/net_speed_benchmark.cpp
@@ -25,7 +25,7 @@ int main(int argc, char** argv) {
   if (argc < 2 || argc > 5) {
     LOG(ERROR) << "net_speed_benchmark net_proto [iterations=50]"
         " [CPU/GPU] [Device_id=0]";
-    return 0;
+    return 1;
   }
 
   if (argc >=3) {

diff --git a/tools/test_net.cpp b/tools/test_net.cpp
index 75d42d6..0abfbf6 100644 (file)
--- a/tools/test_net.cpp
+++ b/tools/test_net.cpp
@@ -20,7 +20,7 @@ int main(int argc, char** argv) {
   if (argc < 4 || argc > 5) {
     LOG(ERROR) << "test_net net_proto pretrained_net_proto iterations "
         << "[CPU/GPU]";
-    return 0;
+    return 1;
   }
 
   cudaSetDevice(0);

diff --git a/tools/train_net.cpp b/tools/train_net.cpp
index bb881d3..751a704 100644 (file)
--- a/tools/train_net.cpp
+++ b/tools/train_net.cpp
@@ -17,7 +17,7 @@ int main(int argc, char** argv) {
   ::google::InitGoogleLogging(argv[0]);
   if (argc < 2 || argc > 3) {
     LOG(ERROR) << "Usage: train_net solver_proto_file [resume_point_file]";
-    return 0;
+    return 1;
   }
 
   SolverParameter solver_param;