Refactor logger to do message formatting internally

[builder.git] / source / packagemanager.cpp

diff --git a/source/packagemanager.cpp b/source/packagemanager.cpp
index af672d9710fd175db235bf9be11d2ba730ef20b8..31b24613b6f4c468957208ae412c68fa9775d656 100644 (file)
--- a/source/packagemanager.cpp
+++ b/source/packagemanager.cpp
@@ -4,7 +4,6 @@
 #include <msp/fs/stat.h>
 #include <msp/fs/utils.h>
 #include <msp/io/print.h>
-#include <msp/strings/format.h>
 #include <msp/strings/utils.h>
 #include <msp/time/timedelta.h>
 #include <msp/time/utils.h>
@@ -162,7 +161,7 @@ string PackageManager::run_pkgconfig(const string &pkg, const string &what)
                argv.push_back("--variable="+what);
        argv.push_back(pkg);
 
-       builder.get_logger().log("auxcommands", format("Running %s", join(argv.begin(), argv.end())));
+       builder.get_logger().log("auxcommands", "Running %s", join(argv.begin(), argv.end()));
 
        return ExternalTask::run_and_capture_output(argv);
 #else
@@ -174,7 +173,7 @@ string PackageManager::run_pkgconfig(const string &pkg, const string &what)
 
 FS::Path PackageManager::get_package_location(const string &name)
 {
-       builder.get_logger().log("packagemgr", format("Looking for source package %s", name));
+       builder.get_logger().log("packagemgr", "Looking for source package %s", name);
 
        try
        {
@@ -190,7 +189,7 @@ FS::Path PackageManager::get_package_location(const string &name)
        {
                for(const FS::Path &p: pkg_path)
                {
-                       builder.get_logger().log("files", format("Traversing %s", p));
+                       builder.get_logger().log("files", "Traversing %s", p);
                        unsigned count = 0;
                        for(const string &f: list_files(p))
                        {
@@ -202,10 +201,10 @@ FS::Path PackageManager::get_package_location(const string &name)
                                }
                        }
 
-                       builder.get_logger().log("packagemgr", format("%d source packages found in %s", count, p));
+                       builder.get_logger().log("packagemgr", "%d source packages found in %s", count, p);
                }
 
-               builder.get_logger().log("packagemgr", format("%d source packages found", pkg_dirs.size()));
+               builder.get_logger().log("packagemgr", "%d source packages found", pkg_dirs.size());
        }
 
        bool msp = !name.compare(0, 3, "msp");
@@ -225,20 +224,20 @@ FS::Path PackageManager::get_package_location(const string &name)
 
 FS::Path PackageManager::get_binary_package_file(const string &name)
 {
-       builder.get_logger().log("packagemgr", format("Looking for binary package %s", name));
+       builder.get_logger().log("packagemgr", "Looking for binary package %s", name);
 
        if(binpkg_files.empty())
        {
                for(const FS::Path &p: binpkg_path)
                {
-                       builder.get_logger().log("files", format("Traversing %s", p));
+                       builder.get_logger().log("files", "Traversing %s", p);
                        vector<string> files = list_filtered(p, "\\.bpk$");
                        for(const string &f: files)
                                binpkg_files.push_back(p/f);
-                       builder.get_logger().log("packagemgr", format("%d binary packages found in %s", files.size(), p));
+                       builder.get_logger().log("packagemgr", "%d binary packages found in %s", files.size(), p);
                }
 
-               builder.get_logger().log("packagemgr", format("%d binary packages found", binpkg_files.size()));
+               builder.get_logger().log("packagemgr", "%d binary packages found", binpkg_files.size());
        }
 
        auto i = find_if(binpkg_files, [&name](const FS::Path &p){ return FS::basepart(FS::basename(p))==name; });