FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > ArchLinux > ArchLinux Pacman Development

 
 
LinkBack Thread Tools
 
Old 07-19-2008, 02:40 PM
Nagy Gabor
 
Default Rephrase some debug messages in alpm/sync.c

>From 2433ae595f715847c38d6df312a4941d73ea39f2 Mon Sep 17 00:00:00 2001
From: Nagy Gabor <ngaba@bibl.u-szeged.hu>
Date: Sat, 19 Jul 2008 16:34:02 +0200
Subject: [PATCH] Rephrase some debug messages in alpm/sync.c

Debug messages were removed from _alpm_sync_find, because it is a general
purpose function; debug messages should be placed in the caller function.

I inserted "adding package foo-1.0-1 to the transaction targets" debug
message to find_replacements and sync_sysupgrade.

Signed-off-by: Nagy Gabor <ngaba@bibl.u-szeged.hu>
---
lib/libalpm/sync.c | 11 ++++++-----
1 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c
index ee260f6..7a577a1 100644
--- a/lib/libalpm/sync.c
+++ b/lib/libalpm/sync.c
@@ -149,6 +149,8 @@ static int find_replacements(pmtrans_t *trans, pmdb_t *db_local,
return(-1);
}
sync->removes = alpm_list_add(NULL, lpkg);
+ _alpm_log(PM_LOG_DEBUG, "adding package %s-%s to the transaction targets
",
+ alpm_pkg_get_name(spkg), alpm_pkg_get_version(spkg));
*syncpkgs = alpm_list_add(*syncpkgs, sync);
}
_alpm_log(PM_LOG_DEBUG, "%s-%s elected for removal (to be replaced by %s-%s)
",
@@ -259,6 +261,8 @@ int _alpm_sync_sysupgrade(pmtrans_t *trans,
alpm_list_free(replaced);
return(-1);
}
+ _alpm_log(PM_LOG_DEBUG, "adding package %s-%s to the transaction targets
",
+ alpm_pkg_get_name(spkg), alpm_pkg_get_version(spkg));
*syncpkgs = alpm_list_add(*syncpkgs, sync);
}
}
@@ -342,8 +346,8 @@ int _alpm_sync_addtarget(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t *dbs_sy
if(sync == NULL) {
return(-1);
}
- _alpm_log(PM_LOG_DEBUG, "adding target '%s' to the transaction set
",
- alpm_pkg_get_name(spkg));
+ _alpm_log(PM_LOG_DEBUG, "adding package %s-%s to the transaction targets
",
+ alpm_pkg_get_name(spkg), alpm_pkg_get_version(spkg));
trans->packages = alpm_list_add(trans->packages, sync);

return(0);
@@ -1042,13 +1046,10 @@ pmsyncpkg_t *_alpm_sync_find(alpm_list_t *syncpkgs, const char* pkgname)

pmpkg_t *pkg = alpm_sync_get_pkg(syncpkg);
if(strcmp(alpm_pkg_get_name(pkg), pkgname) == 0) {
- _alpm_log(PM_LOG_DEBUG, "found package '%s-%s' in sync
",
- alpm_pkg_get_name(pkg), alpm_pkg_get_version(pkg));
return(syncpkg);
}
}

- _alpm_log(PM_LOG_DEBUG, "package '%s' not found in sync
", pkgname);
return(NULL); /* not found */
}

--
1.5.6.3


_______________________________________________
pacman-dev mailing list
pacman-dev@archlinux.org
http://archlinux.org/mailman/listinfo/pacman-dev
 

Thread Tools




All times are GMT. The time now is 08:52 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org