[solc] Exit code 2 for exceptions.

Author: aarlt

Changelog.md

@@ -4,6 +4,7 @@ Language Features:
 
 
 Compiler Features:
+* Commandline Interface: Return exit code `2` on uncaught exceptions.
 * Commandline Interface: Add `--no-cbor-metadata` that skips CBOR metadata from getting appended at the end of the bytecode.
 * Standard JSON: Add a boolean field `settings.metadata.appendCBOR` that skips CBOR metadata from getting appended at the end of the bytecode.
 * Yul Optimizer: Allow replacing the previously hard-coded cleanup sequence by specifying custom steps after a colon delimiter (``:``) in the sequence string.

solc/main.cpp

@@ -44,36 +44,36 @@ int main(int argc, char** argv)
 	{
 		cerr << "SMT logic error:" << endl;
 		cerr << boost::diagnostic_information(_exception);
-		return 1;
+		return 2;
 	}
 	catch (langutil::UnimplementedFeatureError const& _exception)
 	{
 		cerr << "Unimplemented feature:" << endl;
 		cerr << boost::diagnostic_information(_exception);
-		return 1;
+		return 2;
 	}
 	catch (langutil::InternalCompilerError const& _exception)
 	{
 		cerr << "Internal compiler error:" << endl;
 		cerr << boost::diagnostic_information(_exception);
-		return 1;
+		return 2;
 	}
 	catch (boost::exception const& _exception)
 	{
 		cerr << "Uncaught exception:" << endl;
 		cerr << boost::diagnostic_information(_exception) << endl;
-		return 1;
+		return 2;
 	}
 	catch (std::exception const& _exception)
 	{
 		cerr << "Uncaught exception:" << endl;
 		cerr << boost::diagnostic_information(_exception) << endl;
-		return 1;
+		return 2;
 	}
 	catch (...)
 	{
 		cerr << "Uncaught exception" << endl;
 		cerr << boost::current_exception_diagnostic_information() << endl;
-		return 1;
+		return 2;
 	}
 }

test/tools/isoltest.cpp

@@ -502,22 +502,22 @@ int main(int argc, char const *argv[])
 	catch (boost::program_options::error const& exception)
 	{
 		cerr << exception.what() << endl;
-		return EXIT_FAILURE;
+		return 2;
 	}
 	catch (std::runtime_error const& exception)
 	{
 		cerr << exception.what() << endl;
-		return EXIT_FAILURE;
+		return 2;
 	}
 	catch (solidity::test::ConfigException const& exception)
 	{
 		cerr << exception.what() << endl;
-		return EXIT_FAILURE;
+		return 2;
 	}
 	catch (...)
 	{
 		cerr << "Unhandled exception caught." << endl;
 		cerr << boost::current_exception_diagnostic_information() << endl;
-		return EXIT_FAILURE;
+		return 2;
 	}
 }

tools/solidityUpgrade/main.cpp

@@ -85,10 +85,12 @@ int main(int argc, char** argv)
 	catch (boost::exception const& _exception)
 	{
 		cerr << "Exception while processing input: " << boost::diagnostic_information(_exception) << endl;
+		return 2;
 	}
 	catch (...)
 	{
 		cerr << "Unknown exception while processing input: " << boost::current_exception_diagnostic_information() << endl;
+		return 2;
 	}
 
 	return 0;

tools/yulPhaser/main.cpp

@@ -36,7 +36,7 @@ int main(int argc, char** argv)
 
 		std::cerr << std::endl;
 		std::cerr << "ERROR: " << exception.what() << std::endl;
-		return 1;
+		return 2;
 	}
 	catch (solidity::phaser::BadInput const& exception)
 	{
@@ -45,7 +45,7 @@ int main(int argc, char** argv)
 
 		std::cerr << std::endl;
 		std::cerr << "ERROR: " << exception.what() << std::endl;
-		return 1;
+		return 2;
 	}
 	catch (solidity::util::Exception const& exception)
 	{
@@ -65,26 +65,25 @@ int main(int argc, char** argv)
 		char const* const* function = boost::get_error_info<boost::throw_function>(exception);
 		if (function != nullptr)
 			std::cerr << "Function: " << *function << std::endl;
-
-		// Let it crash. The terminate() will print some more stuff useful for debugging like
-		// what() and the actual exception type.
-		throw;
+		std::cerr << "ERROR: " << exception.what() << std::endl;
+		return 2;
 	}
-	catch (std::exception const&)
+	catch (std::exception const& exception)
 	{
 		// Again, probably a bug but this time it's just plain std::exception so there's no point
 		// in doing anything special. terminate() will do an adequate job.
 		std::cerr << std::endl;
 		std::cerr << "UNCAUGHT EXCEPTION!" << std::endl;
-		throw;
+		std::cerr << "ERROR: " << exception.what() << std::endl;
+		return 2;
 	}
 	catch (...)
 	{
 		// Some people don't believe these exist.
-		// I have no idea what this is and it's flying towards me so technically speaking it's an
+		// I have no idea what this is, and it's flying towards me so technically speaking it's an
 		// unidentified flying object.
 		std::cerr << std::endl;
 		std::cerr << "UFO SPOTTED!" << std::endl;
-		throw;
+		return 2;
 	}
 }