fix bug

Author: duyanning

src/GccCpp2ObjAction.cpp

@@ -33,6 +33,8 @@ bool GccCpp2ObjAction::execute(const DepInfo& info)
     // birthcert_path += ".birthcert";
 
     string cmd = compile_cpp_cmd;
+
+    cmd += compiler_specific_extra_compile_flags[cpp_path];
 
     cmd += " ";
     cmd += compile_cmd_include_dirs;

src/GccObj2ExeAction.cpp

@@ -51,7 +51,7 @@ bool GccObj2ExeAction::execute(const DepInfo& info)
     cmd += link_cmd_libs;
 
     // -l 选项的位置很重要，必须放在使用它的.o文件之后
-    cmd += extra_link_flags;
+    cmd += compiler_specific_extra_link_flags;
 
     MINILOG(build_exe_summay_logger, "linking " << exe_path.filename().string());
     MINILOG(build_exe_detail_logger, cmd);

src/VcCpp2ObjAction.cpp

@@ -38,6 +38,8 @@ bool VcCpp2ObjAction::execute(const DepInfo& info)
 	string cmd = R"(")"; // cmd的怪癖，传给system函数的东西，前后还得一个引号。 参考： https://stackoverflow.com/questions/9964865/c-system-not-working-when-there-are-spaces-in-two-different-parameters
 	cmd += compile_cpp_cmd;
 
+    cmd += compiler_specific_extra_compile_flags[cpp_path];
+
     cmd += " ";
     cmd += compile_cmd_include_dirs;
 

src/VcObj2ExeAction.cpp

@@ -57,7 +57,7 @@ bool VcObj2ExeAction::execute(const DepInfo& info)
 
     // -l 选项的位置很重要，必须放在使用它的.o文件之后
     cmd += " ";
-    cmd += extra_link_flags;
+    cmd += compiler_specific_extra_link_flags;
 
 
 	cmd += R"(")"; // 传给system函数的东西最后额外的引号

src/global.cpp

@@ -64,5 +64,8 @@ vector<fs::path> headers_to_pc; // 所有需要预编译的头文件的绝对路
 bool vc_use_pch = false; // vc是否使用预编译头文件。(目前只支持一个预编译头文件)
 fs::path vc_h_to_precompile; // vc需要预编译的头文件。等于headers_to_pc[0]
 fs::path vc_cpp_to_generate_pch; // vc用于产生预编译头文件的cpp文件
-string extra_compile_flags; // 源文件中指定的，编译时用的其他选项
-string extra_link_flags; // 源文件中指定的，链接时用的其他选项
+//string compiler_specific_extra_compile_flags; // 源文件中指定的，编译时用的其他选项
+map<fs::path, string> compiler_specific_extra_compile_flags; // 源文件中指定的，编译时用的其他选项。不同文件可能用不同的编译选项
+string compiler_specific_extra_link_flags; // 源文件中指定的，链接时用的其他选项
+
+

src/global.h

@@ -26,8 +26,9 @@ extern map<fs::path, fs::path> source2header_to_pc;
 extern bool vc_use_pch;
 extern fs::path vc_h_to_precompile;
 extern fs::path vc_cpp_to_generate_pch;
-extern string extra_compile_flags;
-extern string extra_link_flags;
+//extern string compiler_specific_extra_compile_flags;
+extern map<fs::path, string> compiler_specific_extra_compile_flags;
+extern string compiler_specific_extra_link_flags;
 extern string compiler_dir;
 extern string compile_cmd_include_dirs;
 extern string link_cmd_lib_dirs;

src/main.cpp

@@ -4,31 +4,30 @@
 int main(int argc, char* argv[])
 try {
 
-	// 解析命令行选项，读取配置文件
-	parse(argc, argv);
+    // 解析命令行选项，读取配置文件
+    parse(argc, argv);
 
     // 扫描源文件，搜集信息
     collect();
     if (collect_only) return 0;
 
-	// todo: 额外的flags，是各个.cpp文件独立设置？还是？独立设置吧
-    compile_cpp_cmd += extra_compile_flags;
-    compile_h_cmd += extra_compile_flags;
-    //gcc_link_cmd += extra_link_flags;
+    // todo: 额外的flags，是各个.cpp文件独立设置？还是？独立设置吧
+    //compile_cpp_cmd += compiler_specific_extra_compile_flags;
+    //compile_h_cmd += compiler_specific_extra_compile_flags;
 
     // 构建
     bool success = build();
-	if (!success) return 0;
+    if (!success) return 0;
     if (build_only) return 0;
 
-	// 运行
+    // 运行
     run();
 
     return 0;
 }
 catch (const po::error& ex) {
-	std::cerr << ex.what() << '\n';
-	return 1;
+    std::cerr << ex.what() << '\n';
+    return 1;
 }
 catch (int exit_code) {
     return exit_code;

src/scan.cpp

@@ -26,22 +26,28 @@ void scan(fs::path src_path)
 	ifstream in(src_path.string());
 
 	string line;
+
 	regex usingcpp_pat{ R"(^\s*#include\s+"([\w\./]+)\.h"\s+//\s+usingcpp)" };
 	regex using_pat{ R"(using\s+([\w\./]+\.(cpp|cxx|c\+\+|cc|c)))" }; // | 或的顺序还挺重要，把长的排前边。免得前缀就匹配。
 	regex linklib_pat{ R"(//\s+linklib\s+([\w\-\.]+))" };
+
 	string compiler_specific_linklib_string = R"(//\s+)" + cc_info[cc].compiler_name;
 	compiler_specific_linklib_string += R"(-linklib\s+([\w\-\.]+))";
 	regex compiler_specific_linklib_pat{ compiler_specific_linklib_string };
 
 	// 下面这行，前后三个*号，不是正则的一部分，而是c++ raw-string的自定义delimiter
 	// https://stackoverflow.com/questions/49416631/escape-r-in-a-raw-string-in-c
 	regex precompile_pat{ R"***(^\s*#include\s+"([\w\./]+\.(h|hpp|H|hh))"\s+//\s+precompile)***" };
-	string extra_compile_flags_string = R"(//\s+)" + cc_info[cc].compiler_name;
-	extra_compile_flags_string += R"(-extra-compile-flags:\s+(.*)$)";
-	regex extra_compile_flags_pat{ extra_compile_flags_string };
-	string extra_link_flags_string = R"(//\s+)" + cc_info[cc].compiler_name;
-	extra_link_flags_string += R"(-extra-link-flags:\s+(.*)$)";
-	regex extra_link_flags_pat{ extra_link_flags_string };
+
+	string compiler_specific_extra_compile_flags_string = R"(//\s+)" + cc_info[cc].compiler_name;
+	compiler_specific_extra_compile_flags_string += R"(-extra-compile-flags:\s+(.*)$)";
+	regex compiler_specific_extra_compile_flags_pat{ compiler_specific_extra_compile_flags_string };
+
+	string compiler_specific_extra_link_flags_string = R"(//\s+)" + cc_info[cc].compiler_name;
+	compiler_specific_extra_link_flags_string += R"(-extra-link-flags:\s+(.*)$)";
+	regex compiler_specific_extra_link_flags_pat{ compiler_specific_extra_link_flags_string };
+
+
 	int n = 0;
 	while (getline(in, line)) {
 		smatch matches;
@@ -90,17 +96,18 @@ void scan(fs::path src_path)
 			libs.push_back(matches[1]);
 		}
 
-		if (regex_search(line, matches, extra_compile_flags_pat)) {
-			MINILOG(collect_info_logger, "found extra compile flags: " << matches[1]);
-			extra_compile_flags += " ";
-			extra_compile_flags += matches[1];
+		if (regex_search(line, matches, compiler_specific_extra_compile_flags_pat)) {
+            MINILOG(collect_info_logger, "found extra link flags: " << matches[1]);
+            string flags = " ";
+            flags += matches[1];
+            compiler_specific_extra_compile_flags[src_path] += flags;
 		}
 
-		if (regex_search(line, matches, extra_link_flags_pat)) {
-			MINILOG(collect_info_logger, "found extra link flags: " << matches[1]);
-			extra_link_flags += " ";
-			extra_link_flags += matches[1];
-		}
+		if (regex_search(line, matches, compiler_specific_extra_link_flags_pat)) {
+            MINILOG(collect_info_logger, "found extra compile flags: " << matches[1]);
+            compiler_specific_extra_link_flags += " ";
+            compiler_specific_extra_link_flags += matches[1];
+        }
 
 		// 搜集需要预编译的头文件
 		if (regex_search(line, matches, precompile_pat)) {