- Pacman-dev - lists.archlinux.org

[pacman-dev] CVS update of pacman-lib/lib/libalpm (package.c)
by Aaron Griffin 01 Feb '07

01 Feb '07

Date: Wednesday, January 31, 2007 @ 20:51:12 Author: aaron Path: /home/cvs-pacman/pacman-lib/lib/libalpm Modified: package.c (1.53 -> 1.54) * !strcmp reads funny (as "not string compare" - switch those to == 0, which has a positive connotation. * Added a nice TODO -----------+ package.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) Index: pacman-lib/lib/libalpm/package.c diff -u pacman-lib/lib/libalpm/package.c:1.53 pacman-lib/lib/libalpm/package.c:1.54 --- pacman-lib/lib/libalpm/package.c:1.53 Tue Jan 30 03:14:11 2007 +++ pacman-lib/lib/libalpm/package.c Wed Jan 31 20:51:12 2007 @@ -268,12 +268,15 @@ RET_ERR(PM_ERR_MEMORY, NULL); } + /* TODO there is no reason to make temp files to read + * from a libarchive archive, it can be done by reading + * directly from the archive */ for(i = 0; archive_read_next_header (archive, &entry) == ARCHIVE_OK; i++) { if(config && filelist && scriptcheck) { /* we have everything we need */ break; } - if(!strcmp(archive_entry_pathname (entry), ".PKGINFO")) { + if(strcmp(archive_entry_pathname (entry), ".PKGINFO") == 0) { /* extract this file into /tmp. it has info for us */ descfile = strdup("/tmp/alpm_XXXXXX"); fd = mkstemp(descfile); @@ -296,10 +299,10 @@ FREE(descfile); close(fd); continue; - } else if(!strcmp(archive_entry_pathname (entry), "._install") || !strcmp(archive_entry_pathname (entry), ".INSTALL")) { + } else if(strcmp(archive_entry_pathname (entry), ".INSTALL") == 0) { info->scriptlet = 1; scriptcheck = 1; - } else if(!strcmp(archive_entry_pathname (entry), ".FILELIST")) { + } else if(strcmp(archive_entry_pathname (entry), ".FILELIST") == 0) { /* Build info->files from the filelist */ FILE *fp; char *fn;

1 0

[pacman-dev] CVS update of pacman-lib/scripts (makepkg)
by dan＠archlinux.org 01 Feb '07

01 Feb '07

Date: Wednesday, January 31, 2007 @ 20:18:22 Author: dan Path: /home/cvs-pacman/pacman-lib/scripts Modified: makepkg (1.35 -> 1.36) removal of unnecessary '\' in integrity check generation ---------+ makepkg | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) Index: pacman-lib/scripts/makepkg diff -u pacman-lib/scripts/makepkg:1.35 pacman-lib/scripts/makepkg:1.36 --- pacman-lib/scripts/makepkg:1.35 Thu Jan 25 20:26:08 2007 +++ pacman-lib/scripts/makepkg Wed Jan 31 20:18:22 2007 @@ -626,7 +626,6 @@ msg "Generating ${integrity_name}s for source files" plain "" ct=0 - newline=0 numsrc=${#source[@]} for netfile in "${source[@]}"; do file=$(strip_url "$netfile") @@ -641,8 +640,7 @@ if [ $ct -eq $numsrc ]; then echo ')' else - echo ' \' - newline=0 + echo fi done plain ""

1 0

[pacman-dev] makepkg.conf - trouble 'upgrading'
by Dan McGee 01 Feb '07

01 Feb '07

When we do roll all this out, everyone is going to get a new makepkg.conf.pacnew file. Our format has changed a bit so we should probably mention this in the install message (e.g. no longer putting export in front of variables, new variables, etc.). One of the new additions is the INTEGRITY_CHECK variable. If this is not present in makepkg,conf, makepkg fails silently as I discovered tonight- when generating it does not generate, and the integrity checks never happen when building. I bring this one up more than anything because isn't this fairly standard and unlikely to be changed by the user? Should this just be at the top of makepkg.conf? It seems like a distribution-wide thing that is standard, and it should join BUILDSCRIPT and PKGEXT at the top of makepkg. -Dan

1 0

[pacman-dev] Removal of bindings
by Jürgen Hötzel 31 Jan '07

31 Jan '07

Hi, I don't know why language bindings generation is part of the pacman-lib CVS repository. Any swig-supported (not only perl/python/java) language binding can be generated from alpm.h and a swig interface file. Real language bindings will also ship higher level function/classes. So i would prefer to remove automated bindings generation. Jürgen

3 3

[pacman-dev] Fun supplementary files
by Aaron Griffin 31 Jan '07

31 Jan '07

I wanted to put this out there. There's a handful of supplementary files that would be nice to have in pacman cvs. One that I can think of off the top of my head, is an 'indent.pro' (see The Fine Manual for details) file for GNU indent (based on the pacman coding guidelines). In addition, it'd be nice to have a guide for translators, so they would know how to send translations (I don't feel qualified to write something like that). I had some more, I'll tack them on with other emails later. This is mainly here for people with free time who want to help out, so feel free to submit anything you feel would be worthwhile. Thanks, aaron

2 1

[pacman-dev] New RC up, getting close!
by Aaron Griffin 31 Jan '07

31 Jan '07

Ok, with Dan formally committing to the repo, we seem to be able to get rid of alot of these bugs fairly quickly. There is very little left in the bug tracker, so as long as we all feel comfortable, we should be able to officially release pacman 3 soon. The newest RC (8) is up at: http://archlinux.org/~aaron/pacman/ Please try it out and report _anything_ and _everything_ you can think of, no matter how small it may seem. Tomorrow I'll setup a semi-formal release schedule, and hopefully we can get this out the door and can move on to pumping out features (3.1). So anyway, it's time we all got excited!

4 6

[pacman-dev] Output questions
by Aaron Griffin 31 Jan '07

31 Jan '07

I have a few pending questions about the current output that I'd like to get some opinions on. Firstly, the spacing in -Ss output - there is an extra newline after each package. This was added a while back upon request, and i personally like it, but it makes the output real huge. Looks like the options are: * Keep the line (-Ss output) * Remove the line (-Qs output) * Add the line between repositories only. Secondly, we have an interesting -Ss "issue" here: $ pacman -Ss ^kernel26$ This should only output (duh) kernel26. However, it also displays kernel26beyond. This is because -Ss ALSO searches the provides=() entries for each package... this is a bit misleading. The options are as follows: * Keep this search and indicate, somewhere, that the package was matched due to a provides=() entry * Don't search provides=() at all So, I'd like some opinions on these two topics... there's probably more somewhere, but I can't think of any.

5 5

[pacman-dev] CVS update of pacman-lib/src/pacman (sync.c)
by Aaron Griffin 31 Jan '07

31 Jan '07

Date: Wednesday, January 31, 2007 @ 03:30:16 Author: aaron Path: /home/cvs-pacman/pacman-lib/src/pacman Modified: sync.c (1.104 -> 1.105) Removed the extra newline based on general opinion --------+ sync.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: pacman-lib/src/pacman/sync.c diff -u pacman-lib/src/pacman/sync.c:1.104 pacman-lib/src/pacman/sync.c:1.105 --- pacman-lib/src/pacman/sync.c:1.104 Wed Jan 31 01:10:22 2007 +++ pacman-lib/src/pacman/sync.c Wed Jan 31 03:30:15 2007 @@ -263,7 +263,7 @@ } indentprint(alpm_pkg_get_desc(pkg), 4); - printf("\n\n"); + printf("\n"); } alpm_list_free(ret); } else {

1 0

[pacman-dev] CVS update of pacman-lib (TODO.aaron lib/libalpm/deps.h)
by Aaron Griffin 31 Jan '07

31 Jan '07

Date: Wednesday, January 31, 2007 @ 03:10:03 Author: aaron Path: /home/cvs-pacman/pacman-lib Modified: TODO.aaron (1.13 -> 1.14) lib/libalpm/deps.h (1.17 -> 1.18) * Forgot the int->enum header checkin * Removed the dep-sorting TODO item - switch the sorting to a topological sort later (I'll let Dan do the CS stuff, heh) --------------------+ TODO.aaron | 6 ------ lib/libalpm/deps.h | 2 +- 2 files changed, 1 insertion(+), 7 deletions(-) Index: pacman-lib/TODO.aaron diff -u pacman-lib/TODO.aaron:1.13 pacman-lib/TODO.aaron:1.14 --- pacman-lib/TODO.aaron:1.13 Wed Jan 31 01:54:28 2007 +++ pacman-lib/TODO.aaron Wed Jan 31 03:10:01 2007 @@ -43,12 +43,6 @@ **** BUGS **** * Removal of conflicts on -A and -U (INCOMPLETE) -* ordering of packages to be installed based on dependencies - i.e. pkga depends=(pkgb pkgc), pkgb depends=(pkgc) - pacman -S pkga pkgb pkgc - > a depends on b and c, move them: pkgb pkgc pkga - > b depends on c, move it: pkgc pkgb pkga - > pacman -S pkgc pkgb pkga * gensync: get all info from packages instead of PKGBUILDs for now, we only need to check the PKGBUILD in case of a 'force' option Index: pacman-lib/lib/libalpm/deps.h diff -u pacman-lib/lib/libalpm/deps.h:1.17 pacman-lib/lib/libalpm/deps.h:1.18 --- pacman-lib/lib/libalpm/deps.h:1.17 Tue Jan 23 22:02:54 2007 +++ pacman-lib/lib/libalpm/deps.h Wed Jan 31 03:10:03 2007 @@ -46,7 +46,7 @@ pmdepmod_t depmod, const char *depname, const char *depversion); int _alpm_depmiss_isin(pmdepmissing_t *needle, alpm_list_t *haystack); -alpm_list_t *_alpm_sortbydeps(alpm_list_t *targets, int mode); +alpm_list_t *_alpm_sortbydeps(alpm_list_t *targets, pmtranstype_t mode); alpm_list_t *_alpm_checkdeps(pmtrans_t *trans, pmdb_t *db, pmtranstype_t op, alpm_list_t *packages); int _alpm_splitdep(char *depstr, pmdepend_t *depend);

1 0

[pacman-dev] CVS update of pacman-lib/src/pacman (query.c)
by Aaron Griffin 31 Jan '07

31 Jan '07

Date: Wednesday, January 31, 2007 @ 03:00:50 Author: aaron Path: /home/cvs-pacman/pacman-lib/src/pacman Modified: query.c (1.20 -> 1.21) * Make -Qo work on multiple files again * no real sense in query_fileowner returning an int ---------+ query.c | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) Index: pacman-lib/src/pacman/query.c diff -u pacman-lib/src/pacman/query.c:1.20 pacman-lib/src/pacman/query.c:1.21 --- pacman-lib/src/pacman/query.c:1.20 Wed Jan 31 01:10:22 2007 +++ pacman-lib/src/pacman/query.c Wed Jan 31 03:00:50 2007 @@ -39,7 +39,7 @@ extern config_t *config; extern pmdb_t *db_local; -static int query_fileowner(pmdb_t *db, char *filename) +static void query_fileowner(pmdb_t *db, char *filename) { struct stat buf; int gotcha = 0; @@ -47,16 +47,16 @@ alpm_list_t *i, *j; if(db == NULL) { - return(0); + return; } if(filename == NULL || strlen(filename) == 0) { ERR(NL, _("no file was specified for --owns\n")); - return(1); + return; } if(stat(filename, &buf) == -1 || S_ISDIR(buf.st_mode) || realpath(filename, rpath) == NULL) { - ERR(NL, _("%s is not a file.\n"), filename); - return(1); + /* fail silently if we're a directory */ + return; } for(i = alpm_db_getpkgcache(db); i && !gotcha; i = alpm_list_next(i)) { @@ -76,10 +76,7 @@ } if(!gotcha) { ERR(NL, _("No package owns %s\n"), filename); - return(1); } - - return(0); } int pacman_query(alpm_list_t *targets) @@ -195,7 +192,8 @@ /* determine the owner of a file */ if(config->op_q_owns) { - return(query_fileowner(db_local, package)); + query_fileowner(db_local, package); + continue; } /* find packages in the db */

1 0

[pacman-dev] CVS update of pacman-lib/lib/libalpm (deps.c)
by Aaron Griffin 31 Jan '07

31 Jan '07

Date: Wednesday, January 31, 2007 @ 02:14:50 Author: aaron Path: /home/cvs-pacman/pacman-lib/lib/libalpm Modified: deps.c (1.64 -> 1.65) Switched an int to an enum --------+ deps.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: pacman-lib/lib/libalpm/deps.c diff -u pacman-lib/lib/libalpm/deps.c:1.64 pacman-lib/lib/libalpm/deps.c:1.65 --- pacman-lib/lib/libalpm/deps.c:1.64 Wed Jan 31 01:10:21 2007 +++ pacman-lib/lib/libalpm/deps.c Wed Jan 31 02:14:50 2007 @@ -103,7 +103,7 @@ * This function returns the new alpm_list_t* target list. * */ -alpm_list_t *_alpm_sortbydeps(alpm_list_t *targets, int mode) +alpm_list_t *_alpm_sortbydeps(alpm_list_t *targets, pmtranstype_t mode) { alpm_list_t *newtargs = NULL; alpm_list_t *i, *j, *k, *l;

1 0

[pacman-dev] CVS update of pacman-lib (TODO.dan)
by dan＠archlinux.org 31 Jan '07

31 Jan '07

Date: Wednesday, January 31, 2007 @ 01:55:40 Author: dan Path: /home/cvs-pacman/pacman-lib Modified: TODO.dan (1.1 -> 1.2) just an update ----------+ TODO.dan | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) Index: pacman-lib/TODO.dan diff -u pacman-lib/TODO.dan:1.1 pacman-lib/TODO.dan:1.2 --- pacman-lib/TODO.dan:1.1 Tue Jan 30 02:35:51 2007 +++ pacman-lib/TODO.dan Wed Jan 31 01:55:40 2007 @@ -27,7 +27,7 @@ * New cache cleaning functionality in pacman - keep installed version, choose number to keep, etc. Good python script in the forums that we should try to emulate, perhaps. -* pacman - downgrade feature - allow users to see cached packages andi +* pacman - downgrade feature - allow users to see cached packages and downgrade to (previous or any?) available options. * alpm_log_action & install script logging interaction - producing duplicate @@ -60,5 +60,3 @@ * 'ldd' & 'ldd -u' - many linkages * possibly split utilities/extras from pacman package * makepkg3 produces files with arch appended- gensync can't deal with it -* Fix repo-add script so it replaces packages instead of simply adding new onesi - (so old version is properly removed).

1 0

[pacman-dev] CVS update of pacman-lib (TODO.aaron)
by Aaron Griffin 31 Jan '07

31 Jan '07

Date: Wednesday, January 31, 2007 @ 01:54:28 Author: aaron Path: /home/cvs-pacman/pacman-lib Modified: TODO.aaron (1.12 -> 1.13) TODO changes 8) ------------+ TODO.aaron | 18 +----------------- 1 file changed, 1 insertion(+), 17 deletions(-) Index: pacman-lib/TODO.aaron diff -u pacman-lib/TODO.aaron:1.12 pacman-lib/TODO.aaron:1.13 --- pacman-lib/TODO.aaron:1.12 Tue Jan 30 03:20:15 2007 +++ pacman-lib/TODO.aaron Wed Jan 31 01:54:28 2007 @@ -2,31 +2,15 @@ * Make sure all the alpm_list_t changes are valid (especially alpm_list_free) -* The --debug flag should't need a silly mask. -* PM_LOG_FLOW1 and PM_LOG_FLOW2 seem extraneous at this point. These should both - be moved to PM_LOG_DEBUG messages, as it's going to end up being impossible to - keep track of what is "flow1" and what is "flow2" - * src/pacman: - There's quite a few single function headers which contain the pacman_* functions. We should move these to a single header (pacman.h) to clean up the source a bit. - - config_t duplicates much of what can be accessed from the alpm_option_* - interface. Determine what can / can't be removed and do so. - - initialize alpm BEFORE parsing args, so we can get rid of a handful of - local lists and things in pacman.c (see below) - - fix the "--debug" parameter. requiring a bitmask from the user is silly. - we can parse this can send a real bitmask to alpm. * lib/libalpm: - - Remove `root` param from alpm_initialize. We can simply initialize to / - and call set_root at a later time. This allows us to initiliaze earlier. - move logging (alpm_logaction) out of the library. log files should be written by the app, not the library. Adding a PM_LOG_* constant that frontends can recognize and send to a file is probably a good idea. - - maybe it's my fault, but I see little difference between PM_LOG_FLOW1, - PM_LOG_FLOW2, and PM_LOG_DEBUG. We might want to just do away with flow1 - and flow2 and use debug across the board. - clear up list allocation/deallocation - some lists need to be free'd, some do not and there is no clear indication WHEN this should happen. - remove DB entries (directories) on a read error? @@ -59,7 +43,7 @@ **** BUGS **** * Removal of conflicts on -A and -U (INCOMPLETE) -* ordering of installed packages based on dependencies +* ordering of packages to be installed based on dependencies i.e. pkga depends=(pkgb pkgc), pkgb depends=(pkgc) pacman -S pkga pkgb pkgc > a depends on b and c, move them: pkgb pkgc pkga

1 0

[pacman-dev] CVS update of pacman-lib/lib/libalpm (be_files.c cache.c)
by Aaron Griffin 31 Jan '07

31 Jan '07

Date: Wednesday, January 31, 2007 @ 01:48:06 Author: aaron Path: /home/cvs-pacman/pacman-lib/lib/libalpm Modified: be_files.c (1.24 -> 1.25) cache.c (1.29 -> 1.30) Fix corrupt DB entry handling when loading package entries. * scan loops no longer abort on corrupt entries * reloading a cache package (and discovering it corrupt) no longer prints 2 messages and uses the pre-build "remove from package cache" function NOTE: The TODOs in there are important for later w.r.t. cleaning up corrupt DB entries but there are some logical complexities with doing so, so I'm holding off for now. ------------+ be_files.c | 111 +++++++++++++++++++++++++++++++---------------------------- cache.c | 5 +- 2 files changed, 61 insertions(+), 55 deletions(-) Index: pacman-lib/lib/libalpm/be_files.c diff -u pacman-lib/lib/libalpm/be_files.c:1.24 pacman-lib/lib/libalpm/be_files.c:1.25 --- pacman-lib/lib/libalpm/be_files.c:1.24 Wed Jan 31 01:10:21 2007 +++ pacman-lib/lib/libalpm/be_files.c Wed Jan 31 01:48:06 2007 @@ -114,7 +114,7 @@ char name[PKG_FULLNAME_LEN]; char *ptr = NULL; int found = 0; - pmpkg_t *pkg; + pmpkg_t *pkg = NULL; ALPM_LOG_FUNC; @@ -122,64 +122,71 @@ RET_ERR(PM_ERR_DB_NULL, NULL); } - if(target != NULL) { - /* search for a specific package (by name only) */ - rewinddir(db->handle); - while(!found && (ent = readdir(db->handle)) != NULL) { - if(!strcmp(ent->d_name, ".") || !strcmp(ent->d_name, "..")) { - continue; - } - /* stat the entry, make sure it's a directory */ - snprintf(path, PATH_MAX, "%s/%s", db->path, ent->d_name); - if(stat(path, &sbuf) || !S_ISDIR(sbuf.st_mode)) { - continue; - } - STRNCPY(name, ent->d_name, PKG_FULLNAME_LEN); - /* truncate the string at the second-to-last hyphen, */ - /* which will give us the package name */ - if((ptr = rindex(name, '-'))) { - *ptr = '\0'; + /* We loop here until we read a valid package. When an iteration of this loop + * fails, it means alpm_db_read failed to read a valid package, so we'll read + * the next so as not to abort whole-db operations early + */ + while(!pkg) { + if(target != NULL) { + /* search for a specific package (by name only) */ + rewinddir(db->handle); + while(!found && (ent = readdir(db->handle)) != NULL) { + if(!strcmp(ent->d_name, ".") || !strcmp(ent->d_name, "..")) { + continue; + } + /* stat the entry, make sure it's a directory */ + snprintf(path, PATH_MAX, "%s/%s", db->path, ent->d_name); + if(stat(path, &sbuf) || !S_ISDIR(sbuf.st_mode)) { + continue; + } + STRNCPY(name, ent->d_name, PKG_FULLNAME_LEN); + /* truncate the string at the second-to-last hyphen, */ + /* which will give us the package name */ + if((ptr = rindex(name, '-'))) { + *ptr = '\0'; + } + if((ptr = rindex(name, '-'))) { + *ptr = '\0'; + } + if(!strcmp(name, target)) { + found = 1; + } } - if((ptr = rindex(name, '-'))) { - *ptr = '\0'; + if(!found) { + return(NULL); } - if(!strcmp(name, target)) { - found = 1; + } else { + /* normal iteration */ + int isdir = 0; + while(!isdir) { + ent = readdir(db->handle); + if(ent == NULL) { + return(NULL); + } + if(!strcmp(ent->d_name, ".") || !strcmp(ent->d_name, "..")) { + isdir = 0; + continue; + } + /* stat the entry, make sure it's a directory */ + snprintf(path, PATH_MAX, "%s/%s", db->path, ent->d_name); + if(!stat(path, &sbuf) && S_ISDIR(sbuf.st_mode)) { + isdir = 1; + } } } - if(!found) { + + pkg = _alpm_pkg_new(NULL, NULL); + if(pkg == NULL) { return(NULL); } - } else { - /* normal iteration */ - int isdir = 0; - while(!isdir) { - ent = readdir(db->handle); - if(ent == NULL) { - return(NULL); - } - if(!strcmp(ent->d_name, ".") || !strcmp(ent->d_name, "..")) { - isdir = 0; - continue; - } - /* stat the entry, make sure it's a directory */ - snprintf(path, PATH_MAX, "%s/%s", db->path, ent->d_name); - if(!stat(path, &sbuf) && S_ISDIR(sbuf.st_mode)) { - isdir = 1; - } + if(_alpm_pkg_splitname(ent->d_name, pkg->name, pkg->version, 0) == -1) { + _alpm_log(PM_LOG_ERROR, _("invalid name for dabatase entry '%s'"), ent->d_name); + return(NULL); + } + if(_alpm_db_read(db, inforeq, pkg) == -1) { + /* TODO removed corrupt entry from the FS here */ + FREEPKG(pkg); } - } - - pkg = _alpm_pkg_new(NULL, NULL); - if(pkg == NULL) { - return(NULL); - } - if(_alpm_pkg_splitname(ent->d_name, pkg->name, pkg->version, 0) == -1) { - _alpm_log(PM_LOG_ERROR, _("invalid name for dabatase entry '%s'"), ent->d_name); - return(NULL); - } - if(_alpm_db_read(db, inforeq, pkg) == -1) { - FREEPKG(pkg); } return(pkg); Index: pacman-lib/lib/libalpm/cache.c diff -u pacman-lib/lib/libalpm/cache.c:1.29 pacman-lib/lib/libalpm/cache.c:1.30 --- pacman-lib/lib/libalpm/cache.c:1.29 Tue Jan 30 03:14:10 2007 +++ pacman-lib/lib/libalpm/cache.c Wed Jan 31 01:48:06 2007 @@ -121,9 +121,8 @@ pmpkg_t *pkg = (pmpkg_t *)p->data; if(infolevel != INFRQ_NONE && !(pkg->infolevel & infolevel)) { if(_alpm_db_read(db, infolevel, pkg) == -1) { - _alpm_log(PM_LOG_ERROR, _("failed to read package '%s-%s', removing from package cache"), - pkg->name, pkg->version); - p = alpm_list_remove_node(p); + /* TODO should we actually remove from the filesystem here as well? */ + _alpm_db_remove_pkgfromcache(db, pkg); } else { reloaded = 1; }

1 0

[pacman-dev] CVS update of pacman-lib (19 files)
by Aaron Griffin 31 Jan '07

31 Jan '07

Date: Wednesday, January 31, 2007 @ 01:10:22 Author: aaron Path: /home/cvs-pacman/pacman-lib Modified: lib/libalpm/add.c (1.108 -> 1.109) lib/libalpm/alpm.c (1.109 -> 1.110) lib/libalpm/alpm.h (1.69 -> 1.70) lib/libalpm/be_files.c (1.23 -> 1.24) lib/libalpm/db.c (1.58 -> 1.59) lib/libalpm/deps.c (1.63 -> 1.64) lib/libalpm/handle.c (1.30 -> 1.31) lib/libalpm/remove.c (1.59 -> 1.60) lib/libalpm/sync.c (1.95 -> 1.96) lib/libalpm/util.c (1.43 -> 1.44) src/pacman/add.c (1.26 -> 1.27) src/pacman/conf.c (1.21 -> 1.22) src/pacman/conf.h (1.13 -> 1.14) src/pacman/deptest.c (1.12 -> 1.13) src/pacman/log.c (1.25 -> 1.26) src/pacman/pacman.c (1.84 -> 1.85) src/pacman/query.c (1.19 -> 1.20) src/pacman/remove.c (1.26 -> 1.27) src/pacman/sync.c (1.103 -> 1.104) Debug logging changes: * The --debug params were goofy. New setup allows --debug without params, --debug=<level> where level 1=debug output, 2=debug and download output, 3=debug, download, and function tracing output. This seems more sane to me. * Removed PM_LOG_FLOW1 and PM_LOG_FLOW2. They were just confusing. When adding new functions, it is near impossible to determin if your output should be "flow1" or "flow2" without tracking all the way up the call chain. Rarely would one ever say "ok, lets just show "flow2" output. These have both been replaced with PM_LOG_DEBUG * Removed the need for the root parameter on alpm_initialize. it is now defaulted to PM_ROOT just like dbpath and cachedir. This allows alpm to be initialized BEFORE option parsing in the front end, saving us some duplicate variables in the frontend. * Cleaned up front end variables due to early alpm_initialize call. ------------------------+ lib/libalpm/add.c | 38 ++++++++++++------------ lib/libalpm/alpm.c | 24 ++++----------- lib/libalpm/alpm.h | 14 +++------ lib/libalpm/be_files.c | 2 - lib/libalpm/db.c | 4 +- lib/libalpm/deps.c | 4 +- lib/libalpm/handle.c | 2 + lib/libalpm/remove.c | 26 ++++++++-------- lib/libalpm/sync.c | 44 ++++++++++++++-------------- lib/libalpm/util.c | 2 - src/pacman/add.c | 6 +-- src/pacman/conf.c | 3 - src/pacman/conf.h | 5 --- src/pacman/deptest.c | 2 - src/pacman/log.c | 11 ++----- src/pacman/pacman.c | 72 ++++++++++++++++++++++++++--------------------- src/pacman/query.c | 2 - src/pacman/remove.c | 2 - src/pacman/sync.c | 4 +- 19 files changed, 127 insertions(+), 140 deletions(-) Index: pacman-lib/lib/libalpm/add.c diff -u pacman-lib/lib/libalpm/add.c:1.108 pacman-lib/lib/libalpm/add.c:1.109 --- pacman-lib/lib/libalpm/add.c:1.108 Tue Jan 30 21:58:12 2007 +++ pacman-lib/lib/libalpm/add.c Wed Jan 31 01:10:21 2007 @@ -126,7 +126,7 @@ return(add_faketarget(trans, name)); } - _alpm_log(PM_LOG_FLOW2, _("loading target '%s'"), name); + _alpm_log(PM_LOG_DEBUG, _("loading target '%s'"), name); /* TODO FS#5120 we need a better way to check if a package is a valid package, * and read the metadata instead of relying on the filename for package name @@ -188,7 +188,7 @@ } } - _alpm_log(PM_LOG_FLOW2, _("reading '%s' metadata"), pkgname); + _alpm_log(PM_LOG_DEBUG, _("reading '%s' metadata"), pkgname); info = _alpm_pkg_load(name); if(info == NULL) { /* pm_errno is already set by pkg_load() */ @@ -241,7 +241,7 @@ EVENT(trans, PM_TRANS_EVT_CHECKDEPS_START, NULL, NULL); /* look for unsatisfied dependencies */ - _alpm_log(PM_LOG_FLOW1, _("looking for unsatisfied dependencies")); + _alpm_log(PM_LOG_DEBUG, _("looking for unsatisfied dependencies")); lp = _alpm_checkdeps(trans, db, trans->type, trans->packages); if(lp != NULL) { if(data) { @@ -253,7 +253,7 @@ } /* no unsatisfied deps, so look for conflicts */ - _alpm_log(PM_LOG_FLOW1, _("looking for conflicts")); + _alpm_log(PM_LOG_DEBUG, _("looking for conflicts")); lp = _alpm_checkconflicts(db, trans->packages); for(i = lp; i; i = i->next) { int skip_this = 0; @@ -292,7 +292,7 @@ } /* re-order w.r.t. dependencies */ - _alpm_log(PM_LOG_FLOW1, _("sorting by dependencies")); + _alpm_log(PM_LOG_DEBUG, _("sorting by dependencies")); lp = _alpm_sortbydeps(trans->packages, PM_TRANS_TYPE_ADD); /* free the old alltargs */ FREELISTPTR(trans->packages); @@ -304,7 +304,7 @@ /* Cleaning up */ EVENT(trans, PM_TRANS_EVT_CLEANUP_START, NULL, NULL); - _alpm_log(PM_LOG_FLOW1, _("cleaning up")); + _alpm_log(PM_LOG_DEBUG, _("cleaning up")); for (lp=trans->packages; lp!=NULL; lp=lp->next) { info=(pmpkg_t *)lp->data; for (rmlist=info->removes; rmlist!=NULL; rmlist=rmlist->next) { @@ -321,7 +321,7 @@ EVENT(trans, PM_TRANS_EVT_FILECONFLICTS_START, NULL, NULL); - _alpm_log(PM_LOG_FLOW1, _("looking for file conflicts")); + _alpm_log(PM_LOG_DEBUG, _("looking for file conflicts")); lp = _alpm_db_find_conflicts(db, trans, handle->root, &skiplist); if(lp != NULL) { if(data) { @@ -388,7 +388,7 @@ if(local) { EVENT(trans, PM_TRANS_EVT_UPGRADE_START, info, NULL); cb_state = PM_TRANS_PROGRESS_UPGRADE_START; - _alpm_log(PM_LOG_FLOW1, _("upgrading package %s-%s"), info->name, info->version); + _alpm_log(PM_LOG_DEBUG, _("upgrading package %s-%s"), info->name, info->version); if((what = (char *)malloc(strlen(info->name)+1)) == NULL) { RET_ERR(PM_ERR_MEMORY, -1); } @@ -420,7 +420,7 @@ if(oldpkg) { pmtrans_t *tr; - _alpm_log(PM_LOG_FLOW1, _("removing old package first (%s-%s)"), oldpkg->name, oldpkg->version); + _alpm_log(PM_LOG_DEBUG, _("removing old package first (%s-%s)"), oldpkg->name, oldpkg->version); tr = _alpm_trans_new(); if(tr == NULL) { RET_ERR(PM_ERR_TRANS_ABORT, -1); @@ -450,7 +450,7 @@ if(!pmo_upgrade) { EVENT(trans, PM_TRANS_EVT_ADD_START, info, NULL); cb_state = PM_TRANS_PROGRESS_ADD_START; - _alpm_log(PM_LOG_FLOW1, _("adding package %s-%s"), info->name, info->version); + _alpm_log(PM_LOG_DEBUG, _("adding package %s-%s"), info->name, info->version); if((what = (char *)malloc(strlen(info->name)+1)) == NULL) { RET_ERR(PM_ERR_MEMORY, -1); } @@ -461,11 +461,11 @@ _alpm_runscriptlet(handle->root, info->data, "pre_install", info->version, NULL, trans); } } else { - _alpm_log(PM_LOG_FLOW1, _("adding new package %s-%s"), info->name, info->version); + _alpm_log(PM_LOG_DEBUG, _("adding new package %s-%s"), info->name, info->version); } if(!(trans->flags & PM_TRANS_FLAG_DBONLY)) { - _alpm_log(PM_LOG_FLOW1, _("extracting files")); + _alpm_log(PM_LOG_DEBUG, _("extracting files")); /* Extract the package */ if ((archive = archive_read_new ()) == NULL) @@ -685,7 +685,7 @@ } if(installnew) { - _alpm_log(PM_LOG_FLOW2, _("extracting %s"), pathname); + _alpm_log(PM_LOG_DEBUG, _("extracting %s"), pathname); if(_alpm_copyfile(temp, expath)) { _alpm_log(PM_LOG_ERROR, _("could not copy %s to %s (%s)"), temp, pathname, strerror(errno)); errors++; @@ -705,9 +705,9 @@ close(fd); } else { if(!notouch) { - _alpm_log(PM_LOG_FLOW2, _("extracting %s"), pathname); + _alpm_log(PM_LOG_DEBUG, _("extracting %s"), pathname); } else { - _alpm_log(PM_LOG_FLOW2, _("%s is in NoUpgrade -- skipping"), pathname); + _alpm_log(PM_LOG_DEBUG, _("%s is in NoUpgrade -- skipping"), pathname); strncat(expath, ".pacnew", PATH_MAX); _alpm_log(PM_LOG_WARNING, _("extracting %s as %s.pacnew"), pathname, pathname); alpm_logaction(_("warning: extracting %s%s as %s"), handle->root, pathname, expath); @@ -800,8 +800,8 @@ /* remove the extra line feed appended by asctime() */ info->installdate[strlen(info->installdate)-1] = 0; - _alpm_log(PM_LOG_FLOW1, _("updating database")); - _alpm_log(PM_LOG_FLOW2, _("adding database entry '%s'"), info->name); + _alpm_log(PM_LOG_DEBUG, _("updating database")); + _alpm_log(PM_LOG_DEBUG, _("adding database entry '%s'"), info->name); if(_alpm_db_write(db, info, INFRQ_ALL)) { _alpm_log(PM_LOG_ERROR, _("could not update database entry %s-%s"), info->name, info->version); @@ -815,7 +815,7 @@ /* XXX: This is copy-pasta from remove.c - refactor */ /* update dependency packages' REQUIREDBY fields */ if(info->depends) { - _alpm_log(PM_LOG_FLOW2, _("updating dependency packages 'requiredby' fields")); + _alpm_log(PM_LOG_DEBUG, _("updating dependency packages 'requiredby' fields")); } for(lp = info->depends; lp; lp = lp->next) { pmpkg_t *depinfo; @@ -875,7 +875,7 @@ /* run ldconfig if it exists */ if(handle->trans->state != STATE_INTERRUPTED) { - _alpm_log(PM_LOG_FLOW1, _("running \"ldconfig -r %s\""), handle->root); + _alpm_log(PM_LOG_DEBUG, _("running \"ldconfig -r %s\""), handle->root); _alpm_ldconfig(handle->root); } Index: pacman-lib/lib/libalpm/alpm.c diff -u pacman-lib/lib/libalpm/alpm.c:1.109 pacman-lib/lib/libalpm/alpm.c:1.110 --- pacman-lib/lib/libalpm/alpm.c:1.109 Tue Jan 30 03:14:10 2007 +++ pacman-lib/lib/libalpm/alpm.c Wed Jan 31 01:10:21 2007 @@ -72,13 +72,10 @@ /** Initializes the library. This must be called before any other * functions are called. - * @param root the full path of the root we'll be installing to (usually /) * @return 0 on success, -1 on error (pm_errno is set accordingly) */ -int SYMEXPORT alpm_initialize(const char *root) +int SYMEXPORT alpm_initialize() { - char str[PATH_MAX]; - ASSERT(handle == NULL, RET_ERR(PM_ERR_HANDLE_NOT_NULL, -1)); handle = _alpm_handle_new(); @@ -86,13 +83,6 @@ RET_ERR(PM_ERR_MEMORY, -1); } - STRNCPY(str, (root) ? root : PM_ROOT, PATH_MAX); - /* add a trailing '/' if there isn't one */ - if(str[strlen(str)-1] != '/') { - strcat(str, "/"); - } - handle->root = strdup(str); - return(0); } @@ -185,7 +175,7 @@ RET_ERR(PM_ERR_DB_NOT_FOUND, -1); } - _alpm_log(PM_LOG_FLOW1, _("unregistering database '%s'"), db->treename); + _alpm_log(PM_LOG_DEBUG, _("unregistering database '%s'"), db->treename); /* Cleanup */ _alpm_db_free_pkgcache(db); @@ -236,11 +226,11 @@ return(-1); } db->servers = alpm_list_add(db->servers, server); - _alpm_log(PM_LOG_FLOW2, _("adding new server to database '%s': protocol '%s', server '%s', path '%s'"), + _alpm_log(PM_LOG_DEBUG, _("adding new server to database '%s': protocol '%s', server '%s', path '%s'"), db->treename, server->s_url->scheme, server->s_url->host, server->s_url->doc); } else { FREELIST(db->servers); - _alpm_log(PM_LOG_FLOW2, _("serverlist flushed for '%s'"), db->treename); + _alpm_log(PM_LOG_DEBUG, _("serverlist flushed for '%s'"), db->treename); } return(0); @@ -312,7 +302,7 @@ snprintf(path, PATH_MAX, "%s%s/%s" PM_EXT_DB, handle->root, handle->dbpath, db->treename); /* remove the old dir */ - _alpm_log(PM_LOG_FLOW2, _("flushing database %s/%s"), handle->dbpath, db->treename); + _alpm_log(PM_LOG_DEBUG, _("flushing database %s/%s"), handle->dbpath, db->treename); for(lp = _alpm_db_get_pkgcache(db, INFRQ_NONE); lp; lp = lp->next) { if(_alpm_db_remove(db, lp->data) == -1) { if(lp->data) { @@ -497,7 +487,7 @@ } if(strcmp(sha1sum, pkg->sha1sum) == 0) { - _alpm_log(PM_LOG_FLOW1, _("checksums for package %s-%s are matching"), + _alpm_log(PM_LOG_DEBUG, _("checksums for package %s-%s are matching"), pkg->name, pkg->version); } else { _alpm_log(PM_LOG_ERROR, _("sha1sums do not match for package %s-%s"), @@ -546,7 +536,7 @@ } if(strcmp(md5sum, pkg->md5sum) == 0) { - _alpm_log(PM_LOG_FLOW1, _("checksums for package %s-%s are matching"), + _alpm_log(PM_LOG_DEBUG, _("checksums for package %s-%s are matching"), pkg->name, pkg->version); } else { _alpm_log(PM_LOG_ERROR, _("md5sums do not match for package %s-%s"), Index: pacman-lib/lib/libalpm/alpm.h diff -u pacman-lib/lib/libalpm/alpm.h:1.69 pacman-lib/lib/libalpm/alpm.h:1.70 --- pacman-lib/lib/libalpm/alpm.h:1.69 Tue Jan 30 21:58:12 2007 +++ pacman-lib/lib/libalpm/alpm.h Wed Jan 31 01:10:21 2007 @@ -63,7 +63,7 @@ * Library */ -int alpm_initialize(const char *root); +int alpm_initialize(); int alpm_release(void); /* @@ -72,13 +72,11 @@ /* Levels */ typedef enum _pmloglevel_t { - PM_LOG_DEBUG = 0x01, - PM_LOG_ERROR = 0x02, - PM_LOG_WARNING = 0x04, - PM_LOG_FLOW1 = 0x08, - PM_LOG_FLOW2 = 0x10, - PM_LOG_FUNCTION = 0x20, - PM_LOG_DOWNLOAD = 0x40 + PM_LOG_ERROR = 0x01, + PM_LOG_WARNING = 0x02, + PM_LOG_DEBUG = 0x04, + PM_LOG_DOWNLOAD = 0x08, + PM_LOG_FUNCTION = 0x10 } pmloglevel_t; typedef void (*alpm_cb_log)(unsigned short, char *); Index: pacman-lib/lib/libalpm/be_files.c diff -u pacman-lib/lib/libalpm/be_files.c:1.23 pacman-lib/lib/libalpm/be_files.c:1.24 --- pacman-lib/lib/libalpm/be_files.c:1.23 Tue Jan 30 03:14:10 2007 +++ pacman-lib/lib/libalpm/be_files.c Wed Jan 31 01:10:21 2007 @@ -56,7 +56,7 @@ /* ORE we should not simply unpack the archive, but better parse it and db_write each entry (see sync_load_dbarchive to get archive content) */ - _alpm_log(PM_LOG_FLOW2, _("unpacking database '%s'"), dbfile); + _alpm_log(PM_LOG_DEBUG, _("unpacking database '%s'"), dbfile); if(_alpm_unpack(dbfile, db->path, NULL)) { RET_ERR(PM_ERR_SYSTEM, -1); Index: pacman-lib/lib/libalpm/db.c diff -u pacman-lib/lib/libalpm/db.c:1.58 pacman-lib/lib/libalpm/db.c:1.59 --- pacman-lib/lib/libalpm/db.c:1.58 Tue Jan 30 03:14:10 2007 +++ pacman-lib/lib/libalpm/db.c Wed Jan 31 01:10:21 2007 @@ -174,12 +174,12 @@ } } - _alpm_log(PM_LOG_FLOW1, _("registering database '%s'"), treename); + _alpm_log(PM_LOG_DEBUG, _("registering database '%s'"), treename); /* make sure the database directory exists */ snprintf(path, PATH_MAX, "%s%s/%s", handle->root, handle->dbpath, treename); if(stat(path, &buf) != 0 || !S_ISDIR(buf.st_mode)) { - _alpm_log(PM_LOG_FLOW1, _("database directory '%s' does not exist -- try creating it"), path); + _alpm_log(PM_LOG_DEBUG, _("database directory '%s' does not exist -- try creating it"), path); if(_alpm_makepath(path) != 0) { RET_ERR(PM_ERR_SYSTEM, NULL); } Index: pacman-lib/lib/libalpm/deps.c diff -u pacman-lib/lib/libalpm/deps.c:1.63 pacman-lib/lib/libalpm/deps.c:1.64 --- pacman-lib/lib/libalpm/deps.c:1.63 Tue Jan 30 03:14:11 2007 +++ pacman-lib/lib/libalpm/deps.c Wed Jan 31 01:10:21 2007 @@ -495,7 +495,7 @@ /* see if it was explicitly installed */ if(dep->reason == PM_PKG_REASON_EXPLICIT) { - _alpm_log(PM_LOG_FLOW2, _("excluding %s -- explicitly installed"), dep->name); + _alpm_log(PM_LOG_DEBUG, _("excluding %s -- explicitly installed"), dep->name); needed = 1; } @@ -515,7 +515,7 @@ _alpm_log(PM_LOG_DEBUG, _("loading ALL info for '%s'"), pkg->name); _alpm_db_read(db, INFRQ_ALL, pkg); newtargs = alpm_list_add(newtargs, pkg); - _alpm_log(PM_LOG_FLOW2, _("adding '%s' to the targets"), pkg->name); + _alpm_log(PM_LOG_DEBUG, _("adding '%s' to the targets"), pkg->name); newtargs = _alpm_removedeps(db, newtargs); } } Index: pacman-lib/lib/libalpm/handle.c diff -u pacman-lib/lib/libalpm/handle.c:1.30 pacman-lib/lib/libalpm/handle.c:1.31 --- pacman-lib/lib/libalpm/handle.c:1.30 Tue Jan 30 03:14:11 2007 +++ pacman-lib/lib/libalpm/handle.c Wed Jan 31 01:10:21 2007 @@ -77,8 +77,10 @@ handle->access = PM_ACCESS_RW; #endif + handle->root = strdup(PM_ROOT); handle->dbpath = strdup(PM_DBPATH); handle->cachedir = strdup(PM_CACHEDIR); + handle->logmask = PM_LOG_ERROR | PM_LOG_WARNING; return(handle); } Index: pacman-lib/lib/libalpm/remove.c diff -u pacman-lib/lib/libalpm/remove.c:1.59 pacman-lib/lib/libalpm/remove.c:1.60 --- pacman-lib/lib/libalpm/remove.c:1.59 Tue Jan 30 03:14:11 2007 +++ pacman-lib/lib/libalpm/remove.c Wed Jan 31 01:10:21 2007 @@ -89,7 +89,7 @@ } } - _alpm_log(PM_LOG_FLOW2, _("adding %s in the targets list"), info->name); + _alpm_log(PM_LOG_DEBUG, _("adding %s in the targets list"), info->name); trans->packages = alpm_list_add(trans->packages, info); return(0); @@ -107,7 +107,7 @@ if(!(trans->flags & (PM_TRANS_FLAG_NODEPS)) && (trans->type != PM_TRANS_TYPE_UPGRADE)) { EVENT(trans, PM_TRANS_EVT_CHECKDEPS_START, NULL, NULL); - _alpm_log(PM_LOG_FLOW1, _("looking for unsatisfied dependencies")); + _alpm_log(PM_LOG_DEBUG, _("looking for unsatisfied dependencies")); lp = _alpm_checkdeps(trans, db, trans->type, trans->packages); if(lp != NULL) { if(trans->flags & PM_TRANS_FLAG_CASCADE) { @@ -117,7 +117,7 @@ pmdepmissing_t *miss = (pmdepmissing_t *)i->data; pmpkg_t *info = _alpm_db_scan(db, miss->depend.name, INFRQ_ALL); if(info) { - _alpm_log(PM_LOG_FLOW2, _("pulling %s in the targets list"), info->name); + _alpm_log(PM_LOG_DEBUG, _("pulling %s in the targets list"), info->name); trans->packages = alpm_list_add(trans->packages, info); } else { _alpm_log(PM_LOG_ERROR, _("could not find %s in database -- skipping"), @@ -138,12 +138,12 @@ } if(trans->flags & PM_TRANS_FLAG_RECURSE) { - _alpm_log(PM_LOG_FLOW1, _("finding removable dependencies")); + _alpm_log(PM_LOG_DEBUG, _("finding removable dependencies")); trans->packages = _alpm_removedeps(db, trans->packages); } /* re-order w.r.t. dependencies */ - _alpm_log(PM_LOG_FLOW1, _("sorting by dependencies")); + _alpm_log(PM_LOG_DEBUG, _("sorting by dependencies")); lp = _alpm_sortbydeps(trans->packages, PM_TRANS_TYPE_REMOVE); /* free the old alltargs */ FREELISTPTR(trans->packages); @@ -212,7 +212,7 @@ } } if ( skipit ) { - _alpm_log(PM_LOG_FLOW2, _("skipping removal of %s as it has moved to another package"), + _alpm_log(PM_LOG_DEBUG, _("skipping removal of %s as it has moved to another package"), line); } else { /* if the file is flagged, back it up to .pacsave */ @@ -228,7 +228,7 @@ } } } else { - _alpm_log(PM_LOG_FLOW2, _("unlinking %s"), line); + _alpm_log(PM_LOG_DEBUG, _("unlinking %s"), line); int list_count = alpm_list_count(trans->packages); /* this way we don't have to call alpm_list_count twice during PROGRESS */ PROGRESS(trans, PM_TRANS_PROGRESS_REMOVE_START, info->name, (double)(percent * 100), list_count, (list_count - alpm_list_count(targ) + 1)); ++(*position); @@ -262,7 +262,7 @@ if(trans->type != PM_TRANS_TYPE_UPGRADE) { EVENT(trans, PM_TRANS_EVT_REMOVE_START, info, NULL); - _alpm_log(PM_LOG_FLOW1, _("removing package %s-%s"), info->name, info->version); + _alpm_log(PM_LOG_DEBUG, _("removing package %s-%s"), info->name, info->version); /* run the pre-remove scriptlet if it exists */ if(info->scriptlet && !(trans->flags & PM_TRANS_FLAG_NOSCRIPTLET)) { @@ -273,7 +273,7 @@ if(!(trans->flags & PM_TRANS_FLAG_DBONLY)) { int filenum = alpm_list_count(info->files); - _alpm_log(PM_LOG_FLOW1, _("removing files")); + _alpm_log(PM_LOG_DEBUG, _("removing files")); /* iterate through the list backwards, unlinking files */ for(lp = alpm_list_last(info->files); lp; lp = lp->prev) { @@ -290,8 +290,8 @@ } /* remove the package from the database */ - _alpm_log(PM_LOG_FLOW1, _("updating database")); - _alpm_log(PM_LOG_FLOW2, _("removing database entry '%s'"), info->name); + _alpm_log(PM_LOG_DEBUG, _("updating database")); + _alpm_log(PM_LOG_DEBUG, _("removing database entry '%s'"), info->name); if(_alpm_db_remove(db, info) == -1) { _alpm_log(PM_LOG_ERROR, _("could not remove database entry %s-%s"), info->name, info->version); } @@ -300,7 +300,7 @@ } /* update dependency packages' REQUIREDBY fields */ - _alpm_log(PM_LOG_FLOW2, _("updating dependency packages 'requiredby' fields")); + _alpm_log(PM_LOG_DEBUG, _("updating dependency packages 'requiredby' fields")); for(lp = info->depends; lp; lp = lp->next) { pmpkg_t *depinfo = NULL; pmdepend_t depend; @@ -355,7 +355,7 @@ /* run ldconfig if it exists */ if((trans->type != PM_TRANS_TYPE_UPGRADE) && (handle->trans->state != STATE_INTERRUPTED)) { - _alpm_log(PM_LOG_FLOW1, _("running \"ldconfig -r %s\""), handle->root); + _alpm_log(PM_LOG_DEBUG, _("running \"ldconfig -r %s\""), handle->root); _alpm_ldconfig(handle->root); } Index: pacman-lib/lib/libalpm/sync.c diff -u pacman-lib/lib/libalpm/sync.c:1.95 pacman-lib/lib/libalpm/sync.c:1.96 --- pacman-lib/lib/libalpm/sync.c:1.95 Tue Jan 30 03:14:11 2007 +++ pacman-lib/lib/libalpm/sync.c Wed Jan 31 01:10:21 2007 @@ -138,7 +138,7 @@ ALPM_LOG_FUNC; /* check for "recommended" package replacements */ - _alpm_log(PM_LOG_FLOW1, _("checking for package replacements")); + _alpm_log(PM_LOG_DEBUG, _("checking for package replacements")); for(i = dbs_sync; i; i = i->next) { for(j = _alpm_db_get_pkgcache(i->data, INFRQ_DESC); j; j = j->next) { pmpkg_t *spkg = j->data; @@ -183,7 +183,7 @@ sync->data = alpm_list_add(NULL, dummy); trans->packages = alpm_list_add(trans->packages, sync); } - _alpm_log(PM_LOG_FLOW2, _("%s-%s elected for upgrade (to be replaced by %s-%s)"), + _alpm_log(PM_LOG_DEBUG, _("%s-%s elected for upgrade (to be replaced by %s-%s)"), lpkg->name, lpkg->version, spkg->name, spkg->version); } } @@ -205,10 +205,10 @@ ALPM_LOG_FUNC; /* check for "recommended" package replacements */ - _alpm_log(PM_LOG_FLOW1, _("checking for package replacements")); + _alpm_log(PM_LOG_DEBUG, _("checking for package replacements")); if( find_replacements(trans, db_local, dbs_sync) == 0 ) { /* match installed packages with the sync dbs and compare versions */ - _alpm_log(PM_LOG_FLOW1, _("checking for package upgrades")); + _alpm_log(PM_LOG_DEBUG, _("checking for package upgrades")); for(i = _alpm_db_get_pkgcache(db_local, INFRQ_NONE); i; i = i->next) { int cmp; int replace=0; @@ -254,11 +254,11 @@ local->name, local->version, spkg->version); } else if(istoonew(spkg)) { /* package too new (UpgradeDelay) */ - _alpm_log(PM_LOG_FLOW1, _("%s-%s: delaying upgrade of package (%s)"), + _alpm_log(PM_LOG_DEBUG, _("%s-%s: delaying upgrade of package (%s)"), local->name, local->version, spkg->version); /* check if spkg->name is already in the packages list. */ } else { - _alpm_log(PM_LOG_FLOW2, _("%s-%s elected for upgrade (%s => %s)"), + _alpm_log(PM_LOG_DEBUG, _("%s-%s elected for upgrade (%s => %s)"), local->name, local->version, local->version, spkg->version); if(!find_pkginsync(spkg->name, trans->packages)) { pmpkg_t *dummy = _alpm_pkg_new(local->name, local->version); @@ -314,7 +314,7 @@ if(spkg == NULL) { /* Search provides */ alpm_list_t *p; - _alpm_log(PM_LOG_FLOW2, _("target '%s' not found -- looking for provisions"), targ); + _alpm_log(PM_LOG_DEBUG, _("target '%s' not found -- looking for provisions"), targ); p = _alpm_db_whatprovides(dbs, targ); if(p == NULL) { RET_ERR(PM_ERR_PKG_NOT_FOUND, -1); @@ -337,7 +337,7 @@ } if(spkg == NULL) { /* Search provides */ - _alpm_log(PM_LOG_FLOW2, _("target '%s' not found -- looking for provisions"), targ); + _alpm_log(PM_LOG_DEBUG, _("target '%s' not found -- looking for provisions"), targ); for(j = dbs_sync; j && !spkg; j = j->next) { pmdb_t *dbs = j->data; alpm_list_t *p = _alpm_db_whatprovides(dbs, targ); @@ -389,7 +389,7 @@ FREEPKG(dummy); RET_ERR(PM_ERR_MEMORY, -1); } - _alpm_log(PM_LOG_FLOW2, _("adding target '%s' to the transaction set"), spkg->name); + _alpm_log(PM_LOG_DEBUG, _("adding target '%s' to the transaction set"), spkg->name); trans->packages = alpm_list_add(trans->packages, sync); } @@ -437,7 +437,7 @@ /* Resolve targets dependencies */ EVENT(trans, PM_TRANS_EVT_RESOLVEDEPS_START, NULL, NULL); - _alpm_log(PM_LOG_FLOW1, _("resolving target's dependencies")); + _alpm_log(PM_LOG_DEBUG, _("resolving target's dependencies")); for(i = trans->packages; i; i = i->next) { pmpkg_t *spkg = ((pmsyncpkg_t *)i->data)->pkg; if(_alpm_resolvedeps(db_local, dbs_sync, spkg, list, trail, trans, data) == -1) { @@ -457,7 +457,7 @@ goto cleanup; } trans->packages = alpm_list_add(trans->packages, sync); - _alpm_log(PM_LOG_FLOW2, _("adding package %s-%s to the transaction targets"), + _alpm_log(PM_LOG_DEBUG, _("adding package %s-%s to the transaction targets"), spkg->name, spkg->version); } else { /* remove the original targets from the list if requested */ @@ -492,7 +492,7 @@ EVENT(trans, PM_TRANS_EVT_RESOLVEDEPS_DONE, NULL, NULL); - _alpm_log(PM_LOG_FLOW1, _("looking for unresolvable dependencies")); + _alpm_log(PM_LOG_DEBUG, _("looking for unresolvable dependencies")); deps = _alpm_checkdeps(trans, db_local, PM_TRANS_TYPE_UPGRADE, list); if(deps) { if(data) { @@ -512,7 +512,7 @@ /* check for inter-conflicts and whatnot */ EVENT(trans, PM_TRANS_EVT_INTERCONFLICTS_START, NULL, NULL); - _alpm_log(PM_LOG_FLOW1, _("looking for conflicts")); + _alpm_log(PM_LOG_DEBUG, _("looking for conflicts")); deps = _alpm_checkconflicts(db_local, list); if(deps) { int errorout = 0; @@ -523,7 +523,7 @@ pmsyncpkg_t *sync; pmpkg_t *local; - _alpm_log(PM_LOG_FLOW2, _("package '%s' is conflicting with '%s'"), + _alpm_log(PM_LOG_DEBUG, _("package '%s' is conflicting with '%s'"), miss->target, miss->depend.name); /* check if the conflicting package is one that's about to be removed/replaced. @@ -596,7 +596,7 @@ if(rmpkg) { pmsyncpkg_t *rsync = find_pkginsync(rmpkg, trans->packages); void *vpkg; - _alpm_log(PM_LOG_FLOW2, _("removing '%s' from target list"), rmpkg); + _alpm_log(PM_LOG_DEBUG, _("removing '%s' from target list"), rmpkg); trans->packages = alpm_list_remove(trans->packages, rsync, pkg_cmp, &vpkg); FREESYNC(vpkg); continue; @@ -628,12 +628,12 @@ FREEPKG(sync->data); } /* append to the replaces list */ - _alpm_log(PM_LOG_FLOW2, _("electing '%s' for removal"), miss->depend.name); + _alpm_log(PM_LOG_DEBUG, _("electing '%s' for removal"), miss->depend.name); sync->data = alpm_list_add(sync->data, q); if(rsync) { /* remove it from the target list */ void *vpkg; - _alpm_log(PM_LOG_FLOW2, _("removing '%s' from target list"), miss->depend.name); + _alpm_log(PM_LOG_DEBUG, _("removing '%s' from target list"), miss->depend.name); trans->packages = alpm_list_remove(trans->packages, rsync, pkg_cmp, &vpkg); FREESYNC(vpkg); } @@ -707,7 +707,7 @@ } } if(list) { - _alpm_log(PM_LOG_FLOW1, _("checking dependencies of packages designated for removal")); + _alpm_log(PM_LOG_DEBUG, _("checking dependencies of packages designated for removal")); deps = _alpm_checkdeps(trans, db_local, PM_TRANS_TYPE_REMOVE, list); if(deps) { int errorout = 0; @@ -740,7 +740,7 @@ for(o = sp->pkg->provides; o && !pfound; o = o->next) { if(!strcmp(m->data, o->data)) { /* found matching provisio -- we're good to go */ - _alpm_log(PM_LOG_FLOW2, _("found '%s' as a provision for '%s' -- conflict aborted"), + _alpm_log(PM_LOG_DEBUG, _("found '%s' as a provision for '%s' -- conflict aborted"), sp->pkg->name, (char *)o->data); pfound = 1; } @@ -970,7 +970,7 @@ } } if(replaces) { - _alpm_log(PM_LOG_FLOW1, _("removing conflicting and to-be-replaced packages")); + _alpm_log(PM_LOG_DEBUG, _("removing conflicting and to-be-replaced packages")); if(_alpm_trans_prepare(tr, data) == -1) { _alpm_log(PM_LOG_ERROR, _("could not prepare removal transaction")); goto error; @@ -985,7 +985,7 @@ FREETRANS(tr); /* install targets */ - _alpm_log(PM_LOG_FLOW1, _("installing packages")); + _alpm_log(PM_LOG_DEBUG, _("installing packages")); tr = _alpm_trans_new(); if(tr == NULL) { _alpm_log(PM_LOG_ERROR, _("could not create transaction")); @@ -1028,7 +1028,7 @@ /* propagate replaced packages' requiredby fields to their new owners */ if(replaces) { - _alpm_log(PM_LOG_FLOW1, _("updating database for replaced packages' dependencies")); + _alpm_log(PM_LOG_DEBUG, _("updating database for replaced packages' dependencies")); for(i = trans->packages; i; i = i->next) { pmsyncpkg_t *sync = i->data; if(sync->type == PM_SYNC_TYPE_REPLACE) { Index: pacman-lib/lib/libalpm/util.c diff -u pacman-lib/lib/libalpm/util.c:1.43 pacman-lib/lib/libalpm/util.c:1.44 --- pacman-lib/lib/libalpm/util.c:1.43 Tue Jan 30 03:14:11 2007 +++ pacman-lib/lib/libalpm/util.c Wed Jan 31 01:10:22 2007 @@ -468,7 +468,7 @@ _alpm_log(PM_LOG_ERROR, _("could not change directory to %s (%s)"), root, strerror(errno)); } - _alpm_log(PM_LOG_FLOW2, _("executing %s script..."), script); + _alpm_log(PM_LOG_DEBUG, _("executing %s script..."), script); if(oldver) { snprintf(cmdline, PATH_MAX, "source %s %s %s %s", Index: pacman-lib/src/pacman/add.c diff -u pacman-lib/src/pacman/add.c:1.26 pacman-lib/src/pacman/add.c:1.27 --- pacman-lib/src/pacman/add.c:1.26 Tue Jan 30 23:42:39 2007 +++ pacman-lib/src/pacman/add.c Wed Jan 31 01:10:22 2007 @@ -67,7 +67,7 @@ ERR(NL, "%s\n", alpm_strerror(pm_errno)); if(pm_errno == PM_ERR_HANDLE_LOCK) { MSG(NL, _(" if you're sure a package manager is not already running,\n" - " you can remove %s%s\n"), config->root, PM_LOCK); + " you can remove %s%s\n"), alpm_option_get_root(), PM_LOCK); } return(1); } @@ -124,7 +124,7 @@ switch(alpm_conflict_get_type(conflict)) { case PM_CONFLICT_TYPE_TARGET: MSG(NL, _("%s%s exists in \"%s\" (target) and \"%s\" (target)"), - config->root, + alpm_option_get_root(), alpm_conflict_get_file(conflict), alpm_conflict_get_target(conflict), alpm_conflict_get_ctarget(conflict)); @@ -132,7 +132,7 @@ case PM_CONFLICT_TYPE_FILE: MSG(NL, _("%s: %s%s exists in filesystem"), alpm_conflict_get_target(conflict), - config->root, + alpm_option_get_root(), alpm_conflict_get_file(conflict)); break; } Index: pacman-lib/src/pacman/conf.c diff -u pacman-lib/src/pacman/conf.c:1.21 pacman-lib/src/pacman/conf.c:1.22 --- pacman-lib/src/pacman/conf.c:1.21 Fri Jan 19 04:28:46 2007 +++ pacman-lib/src/pacman/conf.c Wed Jan 31 01:10:22 2007 @@ -26,7 +26,6 @@ #include <libintl.h> #include <alpm.h> -#include <alpm_list.h> /* pacman */ #include "util.h" #include "log.h" @@ -51,9 +50,7 @@ return(-1); } - FREE(config->root); FREE(config->configfile); - FREELIST(config->op_s_ignore); free(config); return(0); Index: pacman-lib/src/pacman/conf.h diff -u pacman-lib/src/pacman/conf.h:1.13 pacman-lib/src/pacman/conf.h:1.14 --- pacman-lib/src/pacman/conf.h:1.13 Fri Jan 19 04:28:46 2007 +++ pacman-lib/src/pacman/conf.h Wed Jan 31 01:10:22 2007 @@ -25,9 +25,6 @@ typedef struct __config_t { /* command line options */ - char *root; - const char *dbpath; - const char *cachedir; char *configfile; unsigned short op; unsigned short verbose; @@ -49,14 +46,12 @@ unsigned short op_s_clean; unsigned short op_s_dependsonly; unsigned short op_s_downloadonly; - alpm_list_t *op_s_ignore; unsigned short op_s_info; unsigned short op_s_sync; unsigned short op_s_search; unsigned short op_s_upgrade; unsigned short group; unsigned int flags; - unsigned short debug; unsigned short noask; unsigned int ask; } config_t; Index: pacman-lib/src/pacman/deptest.c diff -u pacman-lib/src/pacman/deptest.c:1.12 pacman-lib/src/pacman/deptest.c:1.13 --- pacman-lib/src/pacman/deptest.c:1.12 Wed Jan 24 03:51:51 2007 +++ pacman-lib/src/pacman/deptest.c Wed Jan 31 01:10:22 2007 @@ -67,7 +67,7 @@ ERR(NL, "%s", alpm_strerror(pm_errno)); if(pm_errno == PM_ERR_HANDLE_LOCK) { MSG(NL, _(" if you're sure a package manager is not already running,\n" - " you can remove %s%s\n"), config->root, PM_LOCK); + " you can remove %s%s\n"), alpm_option_get_root(), PM_LOCK); } return(1); } Index: pacman-lib/src/pacman/log.c diff -u pacman-lib/src/pacman/log.c:1.25 pacman-lib/src/pacman/log.c:1.26 --- pacman-lib/src/pacman/log.c:1.25 Wed Jan 24 11:57:19 2007 +++ pacman-lib/src/pacman/log.c Wed Jan 31 01:10:22 2007 @@ -66,13 +66,10 @@ case PM_LOG_WARNING: sprintf(str, _("warning")); break; - case PM_LOG_FLOW1: - sprintf(str, _("flow1")); - break; - case PM_LOG_FLOW2: - sprintf(str, _("flow2")); - break; case PM_LOG_FUNCTION: + /* TODO we should increase the indent level when this occurs so we can see + * program flow easier. It'll be fun + */ sprintf(str, _("function")); break; default: @@ -82,7 +79,7 @@ #ifdef PACMAN_DEBUG /* If debug is on, we'll timestamp the output */ - if(config->debug&PM_LOG_DEBUG) { + if(alpm_option_get_logmask() & PM_LOG_DEBUG) { time_t t; struct tm *tmp; char timestr[10] = {0}; Index: pacman-lib/src/pacman/pacman.c diff -u pacman-lib/src/pacman/pacman.c:1.84 pacman-lib/src/pacman/pacman.c:1.85 --- pacman-lib/src/pacman/pacman.c:1.84 Thu Jan 25 21:13:16 2007 +++ pacman-lib/src/pacman/pacman.c Wed Jan 31 01:10:22 2007 @@ -251,7 +251,7 @@ {"noconfirm", no_argument, 0, 1000}, {"config", required_argument, 0, 1001}, {"ignore", required_argument, 0, 1002}, - {"debug", required_argument, 0, 1003}, + {"debug", optional_argument, 0, 1003}, {"noprogressbar", no_argument, 0, 1004}, {"noscriptlet", no_argument, 0, 1005}, {"ask", required_argument, 0, 1006}, @@ -259,6 +259,7 @@ }; char root[PATH_MAX]; struct stat st; + unsigned short logmask; while((opt = getopt_long(argc, argv, "ARUFQSTDYr:b:vkhscVfmnoldepiuwyg", opts, &option_index))) { if(opt < 0) { @@ -277,8 +278,32 @@ config->configfile = strndup(optarg, PATH_MAX); #endif break; - case 1002: config->op_s_ignore = alpm_list_add(config->op_s_ignore, strdup(optarg)); break; - case 1003: config->debug = atoi(optarg); break; + case 1002: alpm_option_add_ignorepkg(strdup(optarg)); break; + case 1003: + /* debug levels are made more 'human readable' than using a raw logmask + * here, we will ALWAYS set error and warning for now, though perhaps a + * --quiet option will remove these later */ + logmask = PM_LOG_ERROR | PM_LOG_WARNING; + + if(optarg) { + unsigned short debug = atoi(optarg); + printf("setting logmask to %s\n", optarg); + switch(debug) { + case 3: logmask |= PM_LOG_FUNCTION; /* fall through */ + case 2: logmask |= PM_LOG_DOWNLOAD; /*fall through */ + case 1: logmask |= PM_LOG_DEBUG; break; + default: + pm_fprintf(stderr, NL, _("error: '%s' is not a valid debug level"), optarg); + return(1); + } + printf("logmask = %d\n", logmask); + } else { + logmask |= PM_LOG_DEBUG; + } + /* progress bars get wonky with debug on, shut them off */ + config->noprogressbar = 1; + alpm_option_set_logmask(logmask); + break; case 1004: config->noprogressbar = 1; break; case 1005: config->flags |= PM_TRANS_FLAG_NOSCRIPTLET; break; case 1006: config->noask = 1; config->ask = atoi(optarg); break; @@ -305,10 +330,9 @@ case 'b': if(stat(optarg, &st) == -1 || !S_ISDIR(st.st_mode)) { pm_fprintf(stderr, NL, _("error: '%s' is not a valid db path\n"), optarg); - exit(EXIT_FAILURE); + return(1); } alpm_option_set_dbpath(optarg); - config->dbpath = alpm_option_get_dbpath(optarg); break; case 'c': (config->op_s_clean)++; @@ -334,12 +358,11 @@ config->flags |= PM_TRANS_FLAG_PRINTURIS; break; case 'r': - printf("setting root path=%s\n", optarg); if(realpath(optarg, root) == NULL) { - perror(_("bad root path")); + pm_fprintf(stderr, NL, _("error: '%s' is not a valid root path\n"), optarg); return(1); } - config->root = strdup(root); + alpm_option_set_root(strdup(root)); break; case 's': config->op_s_search = 1; @@ -416,13 +439,17 @@ /* init config data */ config = config_new(); config->op = PM_OP_MAIN; - config->debug |= PM_LOG_ERROR; - config->debug |= PM_LOG_WARNING; /* disable progressbar if the output is redirected */ if(!isatty(1)) { config->noprogressbar = 1; } + /* initialize pm library */ + if(alpm_initialize() == -1) { + ERR(NL, _("failed to initilize alpm library (%s)\n"), alpm_strerror(pm_errno)); + cleanup(1); + } + /* parse the command line */ ret = parseargs(argc, argv); if(ret != 0) { @@ -447,7 +474,7 @@ (config->op_s_search || config->group || config->op_q_list || config->op_q_info || config->flags & PM_TRANS_FLAG_PRINTURIS)) || (config->op == PM_OP_DEPTEST && !config->op_d_resolve) - || (config->root != NULL)) { + || (strcmp(alpm_option_get_root(), PM_ROOT) != 0)) { /* special case: PM_OP_SYNC can be used w/ config->op_s_search by any user */ /* special case: ignore root user check if -r is specified, fall back on * normal FS checking */ @@ -460,18 +487,7 @@ } #endif - if(config->root == NULL) { - config->root = strdup(PM_ROOT); - } - - /* initialize pm library */ - if(alpm_initialize(config->root) == -1) { - ERR(NL, _("failed to initilize alpm library (%s)\n"), alpm_strerror(pm_errno)); - cleanup(1); - } - /* Setup logging as soon as possible, to print out maximum debugging info */ - alpm_option_set_logmask(config->debug); alpm_option_set_logcb(cb_log); if(config->configfile == NULL) { @@ -486,17 +502,9 @@ /* set library parameters */ alpm_option_set_dlcb(log_progress); - config->dbpath = alpm_option_get_dbpath(); - config->cachedir = alpm_option_get_cachedir(); - - alpm_list_t *i; - for(i = config->op_s_ignore; i; i = alpm_list_next(i)) { - alpm_option_add_ignorepkg(alpm_list_getdata(i)); - } - if(config->verbose > 0) { - printf("Root : %s\n", config->root); - printf("DBPath: %s\n", config->dbpath); + printf("Root : %s\n", alpm_option_get_root()); + printf("DBPath: %s\n", alpm_option_get_dbpath()); list_display(_("Targets:"), pm_targets); } Index: pacman-lib/src/pacman/query.c diff -u pacman-lib/src/pacman/query.c:1.19 pacman-lib/src/pacman/query.c:1.20 --- pacman-lib/src/pacman/query.c:1.19 Wed Jan 24 03:51:51 2007 +++ pacman-lib/src/pacman/query.c Wed Jan 31 01:10:22 2007 @@ -260,7 +260,7 @@ if(config->op_q_changelog) { char changelog[PATH_MAX]; snprintf(changelog, PATH_MAX, "%s%s/%s/%s-%s/changelog", - config->root, alpm_option_get_dbpath(), + alpm_option_get_root(), alpm_option_get_dbpath(), alpm_db_get_name(db_local), alpm_pkg_get_name(info), alpm_pkg_get_version(info)); Index: pacman-lib/src/pacman/remove.c diff -u pacman-lib/src/pacman/remove.c:1.26 pacman-lib/src/pacman/remove.c:1.27 --- pacman-lib/src/pacman/remove.c:1.26 Tue Jan 30 23:42:39 2007 +++ pacman-lib/src/pacman/remove.c Wed Jan 31 01:10:22 2007 @@ -77,7 +77,7 @@ ERR(NL, _("failed to init transaction (%s)\n"), alpm_strerror(pm_errno)); if(pm_errno == PM_ERR_HANDLE_LOCK) { MSG(NL, _(" if you're sure a package manager is not already running,\n" - " you can remove %s%s\n"), config->root, PM_LOCK); + " you can remove %s%s\n"), alpm_option_get_root(), PM_LOCK); } FREELIST(finaltargs); return(1); Index: pacman-lib/src/pacman/sync.c diff -u pacman-lib/src/pacman/sync.c:1.103 pacman-lib/src/pacman/sync.c:1.104 --- pacman-lib/src/pacman/sync.c:1.103 Tue Jan 30 13:25:54 2007 +++ pacman-lib/src/pacman/sync.c Wed Jan 31 01:10:22 2007 @@ -439,7 +439,7 @@ ERR(NL, _("failed to init transaction (%s)\n"), alpm_strerror(pm_errno)); if(pm_errno == PM_ERR_HANDLE_LOCK) { MSG(NL, _(" if you're sure a package manager is not already running,\n" - " you can remove %s%s\n"), config->root, PM_LOCK); + " you can remove %s%s\n"), alpm_option_get_root(), PM_LOCK); } return(1); } @@ -488,7 +488,7 @@ ERR(NL, _("failed to init transaction (%s)\n"), alpm_strerror(pm_errno)); if(pm_errno == PM_ERR_HANDLE_LOCK) { MSG(NL, _(" if you're sure a package manager is not already running,\n" - " you can remove %s%s\n"), config->root, PM_LOCK); + " you can remove %s%s\n"), alpm_option_get_root(), PM_LOCK); } return(1); }

1 0

[pacman-dev] CVS update of pacman-lib/src/pacman (add.c remove.c)
by Aaron Griffin 31 Jan '07

31 Jan '07

Date: Tuesday, January 30, 2007 @ 23:42:39 Author: aaron Path: /home/cvs-pacman/pacman-lib/src/pacman Modified: add.c (1.25 -> 1.26) remove.c (1.25 -> 1.26) More uninitialized stuff... I believe these were removed at some point. Should be fixed. ----------+ add.c | 2 +- remove.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) Index: pacman-lib/src/pacman/add.c diff -u pacman-lib/src/pacman/add.c:1.25 pacman-lib/src/pacman/add.c:1.26 --- pacman-lib/src/pacman/add.c:1.25 Wed Jan 24 03:51:51 2007 +++ pacman-lib/src/pacman/add.c Tue Jan 30 23:42:39 2007 @@ -38,7 +38,7 @@ int pacman_add(alpm_list_t *targets) { - alpm_list_t *i = targets, *data; + alpm_list_t *i = targets, *data = NULL; int retval = 0; if(targets == NULL) { Index: pacman-lib/src/pacman/remove.c diff -u pacman-lib/src/pacman/remove.c:1.25 pacman-lib/src/pacman/remove.c:1.26 --- pacman-lib/src/pacman/remove.c:1.25 Wed Jan 24 03:51:51 2007 +++ pacman-lib/src/pacman/remove.c Tue Jan 30 23:42:39 2007 @@ -39,7 +39,7 @@ int pacman_remove(alpm_list_t *targets) { - alpm_list_t *data, *i, *j, *finaltargs = NULL; + alpm_list_t *data = NULL, *i, *j, *finaltargs = NULL; int retval = 0; if(targets == NULL) {

1 0

[pacman-dev] New developer: Dan McGee
by Aaron Griffin 31 Jan '07

31 Jan '07

Well, "new" isn't the best word here. For those following this list for a while, Dan has been a flurry of pacman development recently, with lots of great work. So he's been brought on board, because I'm sick of applying his patches (hah!). Anyway, Dan is now entitled to all the glory, fame, and piles upon piles of cash that all archlinux developers are entitled to. So give him a warm welcome, and expect to see lots of CVS commits from him here too. Welcome aboard Dan!

7 6

[pacman-dev] CVS update of pacman-lib/lib/libalpm (add.c alpm.h db.h)
by dan＠archlinux.org 31 Jan '07

31 Jan '07

Date: Tuesday, January 30, 2007 @ 21:58:12 Author: dan Path: /home/cvs-pacman/pacman-lib/lib/libalpm Modified: add.c (1.107 -> 1.108) alpm.h (1.68 -> 1.69) db.h (1.23 -> 1.24) slight line spacing/indent updates --------+ add.c | 4 ++-- alpm.h | 1 - db.h | 1 + 3 files changed, 3 insertions(+), 3 deletions(-) Index: pacman-lib/lib/libalpm/add.c diff -u pacman-lib/lib/libalpm/add.c:1.107 pacman-lib/lib/libalpm/add.c:1.108 --- pacman-lib/lib/libalpm/add.c:1.107 Tue Jan 30 03:14:10 2007 +++ pacman-lib/lib/libalpm/add.c Tue Jan 30 21:58:12 2007 @@ -276,8 +276,8 @@ if(lp == NULL) { break; } - } - } + } + } /* Removal code should go here, as described above. Instead of simply * removing items, perhaps throw them in another list to be removed, then * proceed as sync.c would? I'm not sure because I'm not familiar enough Index: pacman-lib/lib/libalpm/alpm.h diff -u pacman-lib/lib/libalpm/alpm.h:1.68 pacman-lib/lib/libalpm/alpm.h:1.69 --- pacman-lib/lib/libalpm/alpm.h:1.68 Tue Jan 30 02:47:20 2007 +++ pacman-lib/lib/libalpm/alpm.h Tue Jan 30 21:58:12 2007 @@ -401,7 +401,6 @@ /* * Helpers */ - /* md5sums */ char *alpm_get_md5sum(char *name); Index: pacman-lib/lib/libalpm/db.h diff -u pacman-lib/lib/libalpm/db.h:1.23 pacman-lib/lib/libalpm/db.h:1.24 --- pacman-lib/lib/libalpm/db.h:1.23 Tue Jan 23 22:02:54 2007 +++ pacman-lib/lib/libalpm/db.h Tue Jan 30 21:58:12 2007 @@ -45,6 +45,7 @@ alpm_list_t *grpcache; alpm_list_t *servers; }; + /* db.c, database general calls */ pmdb_t *_alpm_db_new(char *root, char *dbpath, char *treename); void _alpm_db_free(void *data);

1 0

[pacman-dev] CVS update of pacman-lib/src/pacman (trans.c trans.h)
by dan＠archlinux.org 31 Jan '07

31 Jan '07

Date: Tuesday, January 30, 2007 @ 20:37:41 Author: dan Path: /home/cvs-pacman/pacman-lib/src/pacman Modified: trans.c (1.29 -> 1.30) trans.h (1.7 -> 1.8) * Refactored some functions to clean up variable declaration. * Reduced magic number usage by fill_progress function (new). * Some switch indent fixing. * Remove use of log10 call. ---------+ trans.c | 197 ++++++++++++++++++++++++++++++++------------------------------ trans.h | 4 - 2 files changed, 106 insertions(+), 95 deletions(-) Index: pacman-lib/src/pacman/trans.c diff -u pacman-lib/src/pacman/trans.c:1.29 pacman-lib/src/pacman/trans.c:1.30 --- pacman-lib/src/pacman/trans.c:1.29 Thu Jan 25 20:34:26 2007 +++ pacman-lib/src/pacman/trans.c Tue Jan 30 20:37:41 2007 @@ -26,7 +26,6 @@ #include <sys/stat.h> #include <unistd.h> #include <dirent.h> -#include <math.h> #include <libintl.h> #include <alpm.h> @@ -42,13 +41,79 @@ static int prevpercent=0; /* for less progressbar output */ +/* refactored function from cb_trans_evt */ +static void retrieve_local(void *data1, void *data2) +{ + const unsigned int maxcols = getcols(); + char out[PATH_MAX]; + unsigned int i; + + MSG(NL, " %s [", (char*)data1); + STRNCPY(out, (char*)data2, maxcols-42); + MSG(CL, "%s", out); + for(i = strlen(out); i < maxcols-43; i++) { + MSG(CL, " "); + } + fputs(_("] 100% LOCAL "), stdout); +} + +/* refactored from cb_trans_progress */ +/* TODO with a little more work, we may be able to incorporate this + * into the download progress bar as well. */ +static void fill_progress(const int percent, const int proglen) +{ + const unsigned short chomp = alpm_option_get_chomp(); + const unsigned int hashlen = proglen - 8; + const unsigned int hash = percent * hashlen / 100; + unsigned int lasthash = 0, mouth = 0; + unsigned int i; + + printf(" ["); + for(i = hashlen; i > 1; --i) { + /* if special progress bar enabled */ + if(chomp) { + if(i > hashlen - hash) { + printf("-"); + } else if(i == hashlen - hash) { + if(lasthash == hash) { + if(mouth) { + printf("\033[1;33mC\033[m"); + } else { + printf("\033[1;33mc\033[m"); + } + } else { + lasthash = hash; + mouth = mouth == 1 ? 0 : 1; + if(mouth) { + printf("\033[1;33mC\033[m"); + } else { + printf("\033[1;33mc\033[m"); + } + } + } else if(i%3 == 0) { + printf("\033[0;37mo\033[m"); + } else { + printf("\033[0;37m \033[m"); + } + } /* else regular progress bar */ + else if(i > hashlen - hash) { + printf("#"); + } else { + printf("-"); + } + } + printf("] %3d%%\r", percent); + + if(percent == 100) { + printf("\n"); + } +} + /* Callback to handle transaction events */ void cb_trans_evt(pmtransevt_t event, void *data1, void *data2) { char str[LOG_STR_LEN] = ""; - char out[PATH_MAX]; - int i; switch(event) { case PM_TRANS_EVT_CHECKDEPS_START: @@ -155,14 +220,7 @@ fflush(stdout); break; case PM_TRANS_EVT_RETRIEVE_LOCAL: - MSG(NL, " %s [", (char*)data1); - unsigned int maxcols = getcols(); - STRNCPY(out, (char*)data2, maxcols-42); - MSG(CL, "%s", out); - for(i = strlen(out); i < maxcols-43; i++) { - MSG(CL, " "); - } - fputs(_("] 100% LOCAL "), stdout); + retrieve_local(data1, data2); break; } } @@ -286,14 +344,12 @@ } } -void cb_trans_progress(pmtransprog_t event, char *pkgname, int percent, - int howmany, int remain) +void cb_trans_progress(pmtransprog_t event, char *pkgname, const int percent, + const int howmany, const int remain) { - static int lasthash = 0, mouth = 0; - int i, hash; - long chomp = 0; - unsigned int maxcols = getcols(); - unsigned int maxpkglen, progresslen = maxcols - 57; + /* size of line to allocate for text printing (e.g. not progressbar) */ + const int infolen = 50; + int i, digits, textlen, pkglen, proglen; char *ptr = NULL; if(config->noprogressbar) { @@ -306,103 +362,58 @@ set_output_padding(0); /* shut it off again */ } - if (!pkgname) - return; - if (percent > 100) - return; - if(percent == prevpercent) + /* if no pkgname, percent is too high or unchanged, then return */ + if (!pkgname || percent > 100 || percent == prevpercent) { return; + } prevpercent=percent; switch (event) { case PM_TRANS_PROGRESS_ADD_START: ptr = _("installing"); break; - case PM_TRANS_PROGRESS_UPGRADE_START: ptr = _("upgrading"); break; - case PM_TRANS_PROGRESS_REMOVE_START: ptr = _("removing"); break; - case PM_TRANS_PROGRESS_CONFLICTS_START: ptr = _("checking for file conflicts"); break; } - hash=percent*progresslen/100; - // if the package name is too long, then slice the ending - maxpkglen=46-strlen(ptr)-(3+2*(int)log10(howmany)); - if(strlen(pkgname)>maxpkglen) - pkgname[maxpkglen]='\0'; + /* find # of digits in package counts to scale output */ + digits = 1; + i = howmany; + while((i /= 10)) { + ++digits; + } + + /* determine room left for non-digits text [not ( 1/12) part] */ + textlen = infolen - 3 - (2 * digits); + /* room left for package name */ + pkglen = textlen - strlen(ptr) - 1; switch (event) { - case PM_TRANS_PROGRESS_ADD_START: - case PM_TRANS_PROGRESS_UPGRADE_START: - case PM_TRANS_PROGRESS_REMOVE_START: - putchar('('); - for(i=0;i<(int)log10(howmany)-(int)log10(remain);i++) - putchar(' '); - printf("%d/%d) %s %s ", remain, howmany, ptr, pkgname); - if (strlen(pkgname)<maxpkglen) - for (i=maxpkglen-strlen(pkgname)-1; i>0; i--) - putchar(' '); - break; - - case PM_TRANS_PROGRESS_CONFLICTS_START: - printf("%s (", ptr); - for(i=0;i<(int)log10(howmany)-(int)log10(remain);i++) - putchar(' '); - printf("%d/%d) ", remain, howmany); - for (i=maxpkglen; i>0; i--) - putchar(' '); - break; - } - - chomp = alpm_option_get_chomp(); - - /* hide the cursor, prevent flicker during fancy graphics - printf("\033[?25l\033[?1c["); - */ - printf("["); - for(i = progresslen; i > 0; --i) { - if(chomp) { - if(i > progresslen - hash) { - printf("-"); - } else if(i == progresslen - hash) { - if(lasthash == hash) { - if(mouth) { - printf("\033[1;33mC\033[m"); - } else { - printf("\033[1;33mc\033[m"); - } - } else { - lasthash = hash; - mouth = mouth == 1 ? 0 : 1; - if(mouth) { - printf("\033[1;33mC\033[m"); - } else { - printf("\033[1;33mc\033[m"); - } - } - } else if(i%3 == 0) { - printf("\033[0;37mo\033[m"); - } else { - printf("\033[0;37m \033[m"); - } - } else if(i > progresslen - hash) { - printf("#"); - } else { - printf("-"); - } - } - printf("] %3d%%\r", percent); + case PM_TRANS_PROGRESS_ADD_START: + case PM_TRANS_PROGRESS_UPGRADE_START: + case PM_TRANS_PROGRESS_REMOVE_START: + /* TODO clean up so digits and pkglen aren't passed twice */ + printf("(%*d/%*d) %s %-*.*s", digits, remain, digits, howmany, + ptr, pkglen, pkglen, pkgname); + break; + + case PM_TRANS_PROGRESS_CONFLICTS_START: + printf("(%*d/%*d) %-*s", digits, remain, digits, howmany, + textlen, ptr); + break; - if(percent == 100) { - printf("\n"); } + + /* call refactored fill progress function */ + proglen = getcols() - infolen; + fill_progress(percent, proglen); } /* vim: set ts=2 sw=2 noet: */ Index: pacman-lib/src/pacman/trans.h diff -u pacman-lib/src/pacman/trans.h:1.7 pacman-lib/src/pacman/trans.h:1.8 --- pacman-lib/src/pacman/trans.h:1.7 Thu Jan 25 21:13:16 2007 +++ pacman-lib/src/pacman/trans.h Tue Jan 30 20:37:41 2007 @@ -29,8 +29,8 @@ void *data3, int *response); /* callback to handle display of the progress bar for transactions */ -void cb_trans_progress(pmtransprog_t event, char *pkgname, int percent, - int howmany, int remain); +void cb_trans_progress(pmtransprog_t event, char *pkgname, const int percent, + const int howmany, const int remain); #endif /* _PM_TRANS_H */

1 0

[pacman-dev] CVS update of pacman-lib/scripts (repo-add)
by dan＠archlinux.org 31 Jan '07

31 Jan '07

Date: Tuesday, January 30, 2007 @ 19:38:13 Author: dan Path: /home/cvs-pacman/pacman-lib/scripts Modified: repo-add (1.3 -> 1.4) * Updated repo-add script to remove same package, different version when adding a package to a database. Also added commenting. :) ----------+ repo-add | 93 ++++++++++++++++++++++++++++++++++++++++++++++--------------- 1 file changed, 71 insertions(+), 22 deletions(-) Index: pacman-lib/scripts/repo-add diff -u pacman-lib/scripts/repo-add:1.3 pacman-lib/scripts/repo-add:1.4 --- pacman-lib/scripts/repo-add:1.3 Wed Dec 20 20:53:41 2006 +++ pacman-lib/scripts/repo-add Tue Jan 30 19:38:13 2007 @@ -1,4 +1,5 @@ #!/bin/bash +# # repo-add : add a package to a given repo database file # # Copyright (c) 2006 Aaron Griffin <aaron(a)archlinux.org> @@ -18,6 +19,8 @@ # Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, # USA. +myver='3.0.0' + FORCE=0 REPO_DB_FILE="" @@ -25,12 +28,27 @@ DB_CHECKSUMS=(md5) TMP_DIR="" -if [ $# -lt 2 ]; then - echo "repo-add /path/to/repo.db.tar.gz [--force] [packages-to-add]" - exit 1 -fi - +# print usage instructions +usage() { + echo "repo-add $myver" + echo + echo "usage: repo-add <path-to-db> [--force] <package> ..." + echo + echo "repo-add will update a package database by reading a package file." + echo "Multiple packages to add can be specified on the command line." + echo + echo "The --force flag will add a 'force' entry to the sync database, which" + echo "tells pacman to skip its internal version number checking and update" + echo "the package regardless." + echo + echo "Example:" + echo " repo-add /path/to/repo.db.tar.gz pacman-3.0.0.pkg.tar.gz" + echo +} +# return calculated checksum of package +# arg1 - checksum type +# arg2 - path to package get_checksum () { case "$(echo "$1" | tr A-Z a-z)" in md5) sum=$(md5sum $2); echo ${sum% *} ;; @@ -41,6 +59,8 @@ esac } +# return PKGINFO string for checksum type +# arg1 - checksum type checksum_name () { case "$(echo "$1" | tr A-Z a-z)" in md5) echo "MD5SUM" ;; @@ -51,6 +71,7 @@ esac } +# test if a file is a repository DB test_repo_db_file () { if [ -f "$REPO_DB_FILE" ]; then [ "$(tar tf "$REPO_DB_FILE" | grep -c "/desc")" -gt 0 ] || return 1 @@ -59,8 +80,11 @@ fi } +# write an entry to the pacman database +# arg1 - path to package db_write_entry() { + # blank out all variables and set pkgfile pkgfile=$(readlink -f $1) export pkgname="" pkgver="" @@ -79,9 +103,10 @@ _conflicts="" OLDIFS="$IFS" - #gross... IFS == new line - IFS=' - ' + # IFS (field seperator) is only the newline character + IFS=$(echo) + + # read info from the zipped package for i in $(tar xOf "$pkgfile" .PKGINFO | grep -v "^#" |sed 's|$\w*$\s*=\s*$.*$|\1="\2"|'); do eval "${i}" case "$i" in @@ -94,28 +119,33 @@ conflicts=*) _conflicts="$_conflicts $conflicts" ;; esac done + IFS=$OLDIFS + # get compressed size of package csize="$(du -b $pkgfile | cut -f1)" cd $gstmpdir + # ensure $pkgname and $pkgver variables were found if [ -z "$pkgname" -o -z "$pkgver" ]; then echo " error: invalid package file" return 1 fi - if [ ! -d "$pkgname-$pkgver" ]; then - [ -e "$pkgname-$pkgver" ] && rm -rf "$pkgname-$pkgver" - mkdir "$pkgname-$pkgver" - cd "$pkgname-$pkgver" - else - cd "$pkgname-$pkgver" - [ -e desc ] && rm desc - [ -e depends ] && rm depends - fi + # remove any other package in the DB with same name + for existing in *; do + if [ "${existing%-*-*}" = "$pkgname" ]; then + echo ":: removing existing package '$existing'" + rm -rf $existing + fi + done + + # create package directory + mkdir "$pkgname-$pkgver" + cd "$pkgname-$pkgver" - # desc + # create desc entry echo ":: creating 'desc' db entry" echo -e "%FILENAME%\n$1\n" >> desc echo -e "%NAME%\n$pkgname\n" >>desc @@ -131,6 +161,7 @@ [ -n $csize ] && echo -e "%CSIZE%\n$csize\n" >>desc [ -n $size ] && echo -e "%ISIZE%\n$size\n" >>desc + # compute checksums for chk in ${DB_CHECKSUMS[@]}; do name="$(checksum_name $chk)" echo ":: computing $name checksums" @@ -156,7 +187,7 @@ fi [ "$FORCE" = "1" ] && echo -e "%FORCE%\n" >>desc - # depends + # create depends entry echo ":: creating 'depends' db entry" if [ -n "$depends" ]; then echo "%DEPENDS%" >>depends @@ -176,21 +207,37 @@ # preserve the modification time touch -r "$pkgfile" desc depends -} +} # end db_write_entry + +# PROGRAM START + +# check for help flags +if [ "$1" = "-h" -o "$1" = "--help" ]; then + usage + exit 0 +fi + +# check for correct number of args +if [ $# -lt 2 ]; then + usage + exit 1 +fi +# main routine if [ $# -gt 1 ]; then gstmpdir=$(mktemp -d /tmp/gensync.XXXXXXXXXX) || (\ echo "cannot create temp directory for database building"; \ exit 1) success=0 + # parse arguements for arg in $@; do if [ "$arg" == "--force" -o "$arg" == "-f" ]; then FORCE=1 - elif [ "x$REPO_DB_FILE" == "x" ]; then + elif [ -z "$REPO_DB_FILE" ]; then REPO_DB_FILE="$(readlink -f $arg)" if ! test_repo_db_file; then - echo " repository db file '$REPO_DB_FILE' is not a proper pacman db" + echo "error: repository file '$REPO_DB_FILE' is not a proper pacman db" exit 1 elif [ -f "$REPO_DB_FILE" ]; then echo ":: extracting database to a temporary location" @@ -215,6 +262,7 @@ fi done + # if all operations were a success, rezip database if [ "$success" = "1" ]; then echo ":: creating updated database file ${REPO_DB_FILE}" cd $gstmpdir @@ -233,6 +281,7 @@ fi fi +# remove the temp directory used to unzip [ -d "$gstmpdir" ] && rm -rf $gstmpdir # vim: set ts=2 sw=2 noet:

1 0

[pacman-dev] CVS update of pacman-lib (configure.ac)
by Juergen Hoetzel 30 Jan '07

30 Jan '07

Date: Tuesday, January 30, 2007 @ 15:37:59 Author: juergen Path: /home/cvs-pacman/pacman-lib Modified: configure.ac (1.25 -> 1.26) * doxygen check cleanup --------------+ configure.ac | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) Index: pacman-lib/configure.ac diff -u pacman-lib/configure.ac:1.25 pacman-lib/configure.ac:1.26 --- pacman-lib/configure.ac:1.25 Tue Jan 30 03:17:02 2007 +++ pacman-lib/configure.ac Tue Jan 30 15:37:59 2007 @@ -195,25 +195,17 @@ fi dnl Check for doxygen support -AC_MSG_CHECKING(for support Doxygen) if test x$wantdoxygen = xyes ; then AC_CHECK_PROGS([DOXYGEN], [doxygen]) - AM_CONDITIONAL(HAS_DOXYGEN, test $DOXYGEN) if test $DOXYGEN ; then - AC_DEFINE([HAS_DOXYGEN], [TRUE], [Enabled Doxygen Support]) - AC_MSG_RESULT(yes) DOXYSTATUS="yes, path: `which $DOXYGEN`" else - AC_DEFINE([HAS_DOXYGEN], [FALSE], [Disabled Doxygen support]) - AC_MSG_RESULT(no, doxygen missing) DOXYSTATUS="doxygen binary not found" fi else - AM_CONDITIONAL(HAS_DOXYGEN, test $DOXYGEN) - AC_DEFINE([HAS_DOXYGEN], [FALSE], [Not specified at configure line]) - AC_MSG_RESULT(not requested by configure) DOXYSTATUS="no, disabled by configure" fi +AM_CONDITIONAL(HAS_DOXYGEN, test $DOXYGEN) dnl Check for swig binary AC_CHECK_PROGS([SWIG], [swig])

1 0

[pacman-dev] CVS update of pacman-lib/src/pacman (sync.c)
by dan＠archlinux.org 30 Jan '07

30 Jan '07

Date: Tuesday, January 30, 2007 @ 13:25:54 Author: dan Path: /home/cvs-pacman/pacman-lib/src/pacman Modified: sync.c (1.102 -> 1.103) Fixing line spacings of download/install size messages. --------+ sync.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: pacman-lib/src/pacman/sync.c diff -u pacman-lib/src/pacman/sync.c:1.102 pacman-lib/src/pacman/sync.c:1.103 --- pacman-lib/src/pacman/sync.c:1.102 Tue Jan 30 03:35:40 2007 +++ pacman-lib/src/pacman/sync.c Tue Jan 30 13:25:54 2007 @@ -680,7 +680,7 @@ indentprint(str, 9); MSG(NL, _("\nTotal Package Size: %.1f MB\n"), mb); if(umb > 0) { - MSG(NL, _("\nTotal Installed Size: %.1f MB\n"), umb); + MSG(NL, _("Total Installed Size: %.1f MB\n"), umb); } FREELIST(list_install); FREE(str);

1 0

[pacman-dev] function tracing
by K. Piche 30 Jan '07

30 Jan '07

I developed the following patch to print a message on function entry to trace program flow. Best part is that it's a #define so it can be undefined later to keep extra code out of the library/turned on with a --debug flag/whatever. The only odd thing I found was that the PM_* constants are in alpm.h instead of log.h so some .c's have to include 2 .h files for logging: one for the function _alpm_log (which should be public since pacman uses it) and the other for the constants. The constants should be moved to the log.h file. k

2 2

[pacman-dev] "memory clobber before allocated" -S bug
by James Rosten 30 Jan '07

30 Jan '07

This is similar to what Travis reported, but instead of happening on packages that exist it happens when pacman3 can't find the package in the db. For example: [~] james->monkeybox : sudo pacman3 -S random_junk --debug=1 [20:23:18] debug: config: new section 'options' [20:23:18] debug: config: logfile: /var/log/pacman.log [20:23:18] debug: config: noupgrade: etc/passwd [20:23:18] debug: config: noupgrade: etc/group [20:23:18] debug: config: noupgrade: etc/shadow [20:23:18] debug: config: noupgrade: etc/sudoers [20:23:18] debug: config: noupgrade: etc/fstab [20:23:18] debug: config: noupgrade: etc/raidtab [20:23:18] debug: config: noupgrade: etc/mdadm.conf [20:23:18] debug: config: noupgrade: etc/ld.so.conf [20:23:18] debug: config: noupgrade: etc/inittab [20:23:18] debug: config: noupgrade: etc/rc.conf [20:23:18] debug: config: noupgrade: etc/rc.local [20:23:18] debug: config: noupgrade: etc/modprobe.conf [20:23:18] debug: config: noupgrade: etc/modules.conf [20:23:18] debug: config: noupgrade: etc/lilo.conf [20:23:18] debug: config: noupgrade: boot/grub/menu.lst [20:23:18] debug: config: noupgrade: etc/mkinitrd.conf [20:23:18] debug: config: holdpkg: pacman [20:23:18] debug: config: holdpkg: glibc [20:23:18] debug: config: new section 'current' [20:23:18] debug: opening database 'current' [20:23:18] debug: config: including /etc/pacman3.d/current [20:23:18] debug: attempt to re-register the 'current' database, using existing [20:23:18] debug: config: new section 'current' [20:23:18] debug: attempt to re-register the 'current' database, using existing [20:23:18] debug: config: new section 'extra' [20:23:18] debug: opening database 'extra' [20:23:18] debug: config: including /etc/pacman3.d/extra [20:23:18] debug: attempt to re-register the 'extra' database, using existing [20:23:18] debug: config: new section 'extra' [20:23:18] debug: attempt to re-register the 'extra' database, using existing [20:23:18] debug: config: new section 'community' [20:23:18] debug: opening database 'community' [20:23:18] debug: config: including /etc/pacman3.d/community [20:23:18] debug: attempt to re-register the 'community' database, using existing [20:23:18] debug: config: new section 'community' [20:23:18] debug: attempt to re-register the 'community' database, using existing [20:23:18] debug: opening database 'local' [20:23:18] debug: loading package cache (infolevel=0) for repository 'current' [20:23:18] debug: loading package cache (infolevel=0) for repository 'extra' [20:23:18] debug: loading package cache (infolevel=0) for repository 'community' [20:23:18] debug: package cache reloaded (infolevel=0x2) for repository 'current' [20:23:18] debug: package cache reloaded (infolevel=0x2) for repository 'extra' [20:23:18] debug: package cache reloaded (infolevel=0x2) for repository 'community' [20:23:18] debug: returning error 31 from _alpm_sync_addtarget : could not find or read package [20:23:18] debug: loading group cache for repository 'current' [20:23:19] debug: package cache reloaded (infolevel=0x1) for repository 'current' [20:23:19] debug: loading group cache for repository 'extra' [20:23:19] debug: package cache reloaded (infolevel=0x1) for repository 'extra' [20:23:19] debug: loading group cache for repository 'community' [20:23:19] debug: package cache reloaded (infolevel=0x1) for repository 'community' error: 'random_junk': not found in sync db memory clobbered before allocated block Aborted Yet, (credit goes to Dan for finding this) doing the samething with pacman3.static works perfectly. ~ Jamie / yankees26

3 2

[pacman-dev] CVS update of pacman-lib/src/pacman (sync.c)
by Aaron Griffin 30 Jan '07

30 Jan '07

Date: Tuesday, January 30, 2007 @ 03:35:40 Author: aaron Path: /home/cvs-pacman/pacman-lib/src/pacman Modified: sync.c (1.101 -> 1.102) Well that silly memory clobber was plain old "uninitialized data" - fixed --------+ sync.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: pacman-lib/src/pacman/sync.c diff -u pacman-lib/src/pacman/sync.c:1.101 pacman-lib/src/pacman/sync.c:1.102 --- pacman-lib/src/pacman/sync.c:1.101 Tue Jan 30 00:41:13 2007 +++ pacman-lib/src/pacman/sync.c Tue Jan 30 03:35:40 2007 @@ -405,7 +405,7 @@ { int confirm = 0; int retval = 0; - alpm_list_t *packages, *data, *i, *j, *k, *sync_dbs; + alpm_list_t *packages, *data = NULL, *i, *j, *k, *sync_dbs; sync_dbs = alpm_option_get_syncdbs(); if(sync_dbs == NULL || alpm_list_count(sync_dbs) == 0) {

1 0