[Refactoring] Code cleanup and remove duplicate methods
[platform/core/dotnet/launcher.git] / NativeLauncher / tool / nitool.cc
index aac3d0e..943852e 100644 (file)
@@ -14,6 +14,7 @@
  * limitations under the License.
  */
 
+#include "utils.h"
 #include "ni_common.h"
 
 #include <cstdio>
@@ -32,7 +33,7 @@ std::vector<std::string> getCmdArgs(char** begin, char** end)
        return list;
 }
 
-bool cmdOptionExists(char** begin, char** end, const std::string& option)
+static bool cmdOptionExists(char** begin, char** end, const std::string& option)
 {
        return std::find(begin, end, option) != end;
 }
@@ -40,71 +41,52 @@ bool cmdOptionExists(char** begin, char** end, const std::string& option)
 static void help(const char *argv0)
 {
        const char* helpDesc =
-               "Usage: %s [args] <root paths or pkg name>\n"
-               "       --help              - Display this screen\n"
-               "       --system            - Create NI under System DLLs\n"
-               "       --dll               - Create NI for DLL\n"
-               "       --pkg               - Create NI for package\n"
-               "       --dir               - Create NI for directory\n"
-               "       --r2r               - Use ready-to-run option (default: FNV)\n"
-               "                             (This option should be used with other options)\n"
-               "       --reset-system      - Remove System NI files\n"
-               "       --reset-pkg         - Remove App NI files\n"
-               "       --regen-all-app     - Re-generate All App NI files\n"
-               "\n"
+               "%s is deprecated. Please use dotnettool instead.\n"
                "Example:\n"
-               "1. Create native image for dlls and exes under platform directories\n"
-               "   # %s --system\n"
-               "2. Create native image for dll\n"
-               "   # %s --dll /usr/bin/Tizen.Runtime.dll\n"
-               "3. Create native image under the package's bin and lib directory\n"
-               "   # %s --pkg org.tizen.FormsGallery\n"
-               "4. Regenerate native images for all installed .net packages with ready-to-run option\n"
-               "   # %s --r2r --regen-all-app\n\n";
-       printf(helpDesc, argv0, argv0, argv0, argv0, argv0);
+               "   # dotnettool --help\n\n";
+       printf(helpDesc, argv0);
 }
 
 int main(int argc, char* argv[])
 {
        bool pkgMode = false;
        bool dllMode = false;
-       bool dirMode = false;
        bool rmPkgMode = false;
-       bool enableR2R = false;
-       bool pkgDllMode = false;
 
-       NiCommonOption option = {std::string(), std::string(), std::string()};
-       if (initNICommon(&option) < 0) {
-               fprintf(stderr, "Fail to initialize NI Common\n");
+       if (initNICommon() != NI_ERROR_NONE) {
                return -1;
        }
 
-       if (cmdOptionExists(argv, argv+argc, "--r2r")) {
-               enableR2R = true;
+       NIOption* opt = getNIOption();
+       if (opt == nullptr) {
+               _SERR("Fail to create option structure.");
+               return -1;
+       }
+
+       // Parse optional switches first.
+       if (cmdOptionExists(argv, argv+argc, "--verbose")) {
+               opt->flags |= NI_FLAGS_VERBOSE;
        }
 
+       // The following commands are mutually exclusive.
        if (cmdOptionExists(argv, argv+argc, "--help")) {
                help(argv[0]);
                return 0;
        } else if (cmdOptionExists(argv, argv+argc, "--system")) {
-               createNiPlatform(enableR2R);
+               createNIPlatform("", opt->flags);
                return 0;
        } else if (cmdOptionExists(argv, argv+argc, "--dll")) {
                dllMode = true;
        } else if (cmdOptionExists(argv, argv+argc, "--pkg")) {
                pkgMode = true;
-       } else if (cmdOptionExists(argv, argv+argc, "--dir")) {
-               dirMode = true;
        } else if (cmdOptionExists(argv, argv+argc, "--reset-system")) {
-               removeNiPlatform();
+               removeNIPlatform();
                return 0;
        } else if (cmdOptionExists(argv, argv+argc, "--reset-pkg")) {
                rmPkgMode = true;
        } else if (cmdOptionExists(argv, argv+argc, "--regen-all-app")) {
-               regenerateAppNI(enableR2R);
+               regenerateAppNI(opt->flags);
                return 0;
-       } else if (cmdOptionExists(argv, argv+argc, "--pkg-dll")) {
-               pkgDllMode = true;
        } else {
                help(argv[0]);
                return 0;
@@ -114,40 +96,47 @@ int main(int argc, char* argv[])
 
        if (args.size() < 1) {
                if (pkgMode)
-                       fprintf(stderr, "Package name is missed\n");
+                       _SERR("Package name is missing.");
                else if (dllMode)
-                       fprintf(stderr, "DLL path is missed\n");
+                       _SERR("DLL path is missing.");
                help(argv[0]);
-               return 1;
+               return -1;
        }
 
        if (pkgMode) {
                for (const std::string pkg : args) {
-                       if (createNiUnderPkgRoot(pkg, enableR2R) != 0) {
-                               fprintf(stderr, "Failed to get root path from [%s]\n", pkg.c_str());
+                       // if there is AOTed dlls under package root, that is skiped.
+                       int ret = createNIUnderPkgRoot(pkg, opt->flags);
+                       if (ret == NI_ERROR_INVALID_PACKAGE) {
+                               _SERR("Failed to get root path from [%s]", pkg.c_str());
+                               return -1;
+                       } else if (ret != NI_ERROR_NONE) {
+                               _SERR("Failed to generate NI file [%s]", pkg.c_str());
                                return -1;
                        }
                }
-       } else if (pkgDllMode) {
-               if (createNiDllUnderPkgRoot(args[0], args[1], enableR2R) != 0) {
-                       fprintf(stderr, "Failed to get root path from [%s]\n", args[0].c_str());
-                       return -1;
-               }
        } else if (rmPkgMode) {
                for (const std::string pkg : args) {
-                       if (removeNiUnderPkgRoot(pkg) != 0) {
-                               fprintf(stderr, "Failed to get root path from [%s]\n", pkg.c_str());
+                       int ret = removeNIUnderPkgRoot(pkg);
+                       if (ret == NI_ERROR_INVALID_PACKAGE) {
+                               _SERR("Failed to get root path from [%s]", pkg.c_str());
+                               return -1;
+                       } else if (ret != NI_ERROR_NONE) {
+                               _SERR("Failed to remove dlls for given package [%s]", pkg.c_str());
                                return -1;
                        }
                }
        } else if (dllMode) {
+               // donot return error code for generation failure.
+               // we have to run crossgen for all input dlls.
                for (const std::string dll : args) {
-                       if (createNiDll(dll, enableR2R) != 0) {
-                               fprintf(stderr, "Failed to generate NI file [%s]\n", dll.c_str());
+                       int ret = createNIDll(dll, opt->flags);
+                       if (ret == NI_ERROR_ALREADY_EXIST) {
+                               // skip for already exist case
+                       } else if (ret != NI_ERROR_NONE) {
+                               _SERR("Failed to generate NI file [%s]", dll.c_str());
                        }
                }
-       } else if (dirMode) {
-               createNiUnderDirs(args.data(), args.size(), enableR2R);
        }
 
        return 0;