Add explicit return to main
[mspang/inapt.git] / inapt.cc
index f33a5b4..ff41649 100644 (file)
--- a/inapt.cc
+++ b/inapt.cc
@@ -18,7 +18,7 @@
 
 #include "inapt.h"
 #include "util.h"
-#include "acqprogress.h"
+#include "contrib/acqprogress.h"
 
 char *prog = NULL;
 
@@ -67,11 +67,8 @@ bool run_install(pkgCacheFile &cache) {
 
   bool Failed = false;
   for (pkgAcquire::ItemIterator i = Fetcher.ItemsBegin(); i != Fetcher.ItemsEnd(); i++) {
-     if ((*i)->Status != pkgAcquire::Item::StatDone || (*i)->Complete != true) {
-         fprintf(stderr,("Failed to fetch %s  %s\n"),(*i)->DescURI().c_str(),
-                 (*i)->ErrorText.c_str());
+     if ((*i)->Status != pkgAcquire::Item::StatDone || (*i)->Complete != true)
          Failed = true;
-     }
   }
 
   if (Failed)
@@ -110,6 +107,26 @@ static bool test_profile(const char *profile, std::set<std::string> *defines) {
             || (*profile == '!' && defines->find(profile + 1) == defines->end());
 }
 
+static bool test_anyprofile(std::string &profile, std::set<std::string> *defines) {
+    char *s = xstrdup(profile.c_str());
+    const char *c = strtok(s, "/");
+
+    if (test_profile(c, defines)) {
+        free(s);
+        return true;
+    }
+
+    while ((c = strtok(NULL, "/")) != NULL) {
+        if (test_profile(c, defines)) {
+            free(s);
+            return true;
+        }
+    }
+
+    free(s);
+    return false;
+}
+
 static pkgCache::PkgIterator eval_pkg(inapt_package *package, pkgCacheFile &cache) {
     pkgCache::PkgIterator pkg;
 
@@ -152,8 +169,7 @@ static pkgCache::PkgIterator eval_pkg(inapt_package *package, pkgCacheFile &cach
             std::vector<std::string>::iterator i = package->alternates.begin();
             std::string message = *(i++);
             while (i != package->alternates.end()) {
-                message.append(", ");
-                message.append(*(i++));
+                message.append(", ").append(*(i++));
             }
             _error->Error("%s:%d: No alternative available: %s", package->filename, package->linenum, message.c_str());
         }
@@ -165,7 +181,7 @@ static pkgCache::PkgIterator eval_pkg(inapt_package *package, pkgCacheFile &cach
 static bool test_profiles(vector<std::string> *profiles, std::set<std::string> *defines) {
     bool ok = true;
     for (vector<std::string>::iterator j = profiles->begin(); j < profiles->end(); j++) {
-        if (!test_profile((*j).c_str(), defines)) {
+        if (!test_anyprofile(*j, defines)) {
             ok = false;
             break;
         }
@@ -263,13 +279,12 @@ static bool sanity_check(std::vector<inapt_package *> *final_actions, pkgCacheFi
 }
 
 static void show_breakage(pkgCacheFile &cache) {
-    fprintf(stderr, "fatal: Unable to solve dependencies\n");
-    fprintf(stderr, "The following packages are broken:");
+    std::string broken;
     for (pkgCache::PkgIterator i = cache->PkgBegin(); !i.end(); i++)
         if (cache[i].NowBroken() || cache[i].InstBroken())
-            fprintf(stderr, " %s", i.Name());
-    fprintf(stderr, "\n");
-    exit(1);
+            broken.append(" ").append(i.Name());
+
+    _error->Error("Broken packages:%s", broken.c_str());
 }
 
 static void exec_actions(std::vector<inapt_package *> *final_actions) {
@@ -349,8 +364,10 @@ static void exec_actions(std::vector<inapt_package *> *final_actions) {
             fix.Protect((*i)->pkg);
         fix.Resolve();
 
-        if (cache->BrokenCount())
+        if (cache->BrokenCount()) {
             show_breakage(cache);
+            return;
+        }
     }
 
     cache->MarkAndSweep();
@@ -429,7 +446,7 @@ int main(int argc, char *argv[]) {
                 _config->Set("Inapt::Purge", true);
                 break;
             case 'd':
-                debug_enabled = true;
+                debug_level++;
                 break;
             case 'o':
                 set_option(optarg);
@@ -460,4 +477,6 @@ int main(int argc, char *argv[]) {
        _error->DumpErrors();
         exit(1);
     }
+
+    return 0;
 }