The following commit has been merged in the master branch:
commit 1a34570e755a93ecc7b9bf87b9f5e99c187b6dda
Author: Guillem Jover <guil...@debian.org>
Date:   Wed Mar 2 09:12:11 2011 +0100

    dpkg: Use package specifiers in statusfd and log messages
    
    The colon is often used as a separator but it is always preceded and/or
    followed by a space so that it's possible to distinguish between the
    colon used as a separator and the colon embedded in a package name
    (like in "libc6:amd64").
    
    Based-on-patch-by: Raphaƫl Hertzog <hert...@debian.org>
    Patch-sponsored-by: Linaro Limited
    
    Signed-off-by: Guillem Jover <guil...@debian.org>

diff --git a/lib/dpkg/dbmodify.c b/lib/dpkg/dbmodify.c
index 6f74406..540d481 100644
--- a/lib/dpkg/dbmodify.c
+++ b/lib/dpkg/dbmodify.c
@@ -407,9 +407,11 @@ void modstatdb_note(struct pkginfo *pkg) {
     pkg->trigaw.head = pkg->trigaw.tail = NULL;
   }
 
-  log_message("status %s %s %s", statusinfos[pkg->status].name, pkg->set->name,
+  log_message("status %s %s %s", statusinfos[pkg->status].name,
+              pkg_name(pkg, pnaw_always),
              versiondescribe(&pkg->installed.version, vdew_nonambig));
-  statusfd_send("status: %s: %s", pkg->set->name, 
statusinfos[pkg->status].name);
+  statusfd_send("status: %s: %s", pkg_name(pkg, pnaw_nonambig),
+                statusinfos[pkg->status].name);
 
   if (cstatus >= msdbrw_write)
     modstatdb_note_core(pkg);
diff --git a/src/help.c b/src/help.c
index ae7e352..872c07d 100644
--- a/src/help.c
+++ b/src/help.c
@@ -653,8 +653,9 @@ void ensure_pathname_nonexisting(const char *pathname) {
 void
 log_action(const char *action, struct pkginfo *pkg, struct pkgbin *pkgbin)
 {
-  log_message("%s %s %s %s", action, pkg->set->name,
+  log_message("%s %s %s %s", action, pkgbin_name(pkg, pkgbin, pnaw_always),
              versiondescribe(&pkg->installed.version, vdew_nonambig),
              versiondescribe(&pkg->available.version, vdew_nonambig));
-  statusfd_send("processing: %s: %s", action, pkg->set->name);
+  statusfd_send("processing: %s: %s", action,
+                pkgbin_name(pkg, pkgbin, pnaw_nonambig));
 }
diff --git a/src/packages.c b/src/packages.c
index 8764829..4c1a44f 100644
--- a/src/packages.c
+++ b/src/packages.c
@@ -232,7 +232,8 @@ void process_queue(void) {
         return;
       continue;
     }
-    push_error_context_jump(&ejbuf, print_error_perpackage, pkg->set->name);
+    push_error_context_jump(&ejbuf, print_error_perpackage,
+                            pkg_name(pkg, pnaw_nonambig));
 
     switch (action_todo) {
     case act_triggers:
diff --git a/src/trigproc.c b/src/trigproc.c
index 5f8fa63..073a18c 100644
--- a/src/trigproc.c
+++ b/src/trigproc.c
@@ -120,7 +120,7 @@ trigproc_run_deferred(void)
                        continue;
                }
                push_error_context_jump(&ejbuf, print_error_perpackage,
-                                       pkg->set->name);
+                                       pkg_name(pkg, pnaw_nonambig));
 
                pkg->clientdata->trigprocdeferred = NULL;
                trigproc(pkg);
@@ -292,7 +292,7 @@ check_trigger_cycle(struct pkginfo *processing_now)
        pkg_set_status(giveup, stat_halfconfigured);
        modstatdb_note(giveup);
        print_error_perpackage(_("triggers looping, abandoned"),
-                              giveup->set->name);
+                              pkg_name(giveup, pnaw_nonambig));
 
        return giveup;
 }

-- 
dpkg's main repository


-- 
To UNSUBSCRIBE, email to debian-dpkg-cvs-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to