- Pacman-dev - lists.archlinux.org

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

11 Feb '07

Date: Sunday, February 11, 2007 @ 15:40:19 Author: dan Path: /home/cvs-pacman/pacman-lib/scripts Added: repo-remove (1.1) Adding repo-remove script, allows for a package to be removed from a repo. More or less a reverse 'repo-add'. -------------+ repo-remove | 136 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 136 insertions(+) Index: pacman-lib/scripts/repo-remove diff -u /dev/null pacman-lib/scripts/repo-remove:1.1 --- /dev/null Sun Feb 11 15:40:19 2007 +++ pacman-lib/scripts/repo-remove Sun Feb 11 15:40:19 2007 @@ -0,0 +1,136 @@ +#!/bin/bash +# +# repo-remove : remove a package entry from a given repo database file +# +# Copyright (c) 2007 Dan McGee <dan(a)archlinux.org> +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 2 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software +# Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, +# USA. + +myver='3.0.0' + +FORCE=0 +REPO_DB_FILE="" + +DB_COMPRESSION="gz" #TODO this is gross +DB_CHECKSUMS=(md5) +TMP_DIR="" + +# print usage instructions +usage() { + echo "repo-remove $myver" + echo + echo "usage: repo-remove <path-to-db> <packagename> ..." + echo + echo "repo-remove will update a package database by removing the package name" + echo "specified on the command line from the given repo database. Multiple" + echo "packages to remove can be specified on the command line." + echo + echo "Example:" + echo " repo-remove /path/to/repo.db.tar.gz kernel26" + echo +} + +# 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 + else + true + fi +} + +# remove existing entries from the DB +db_remove_entry() +{ + cd $gstmpdir + + # remove any other package in the DB with same name + for existing in *; do + if [ "${existing%-*-*}" = "$1" ]; then + echo ":: removing existing package '$existing'" + rm -rf $existing + fi + done +} # end db_remove_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 [ -z "$REPO_DB_FILE" ]; then + REPO_DB_FILE="$(readlink -f $arg)" + if ! test_repo_db_file; then + 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" + tar xf "$REPO_DB_FILE" -C "$gstmpdir" + fi + else + echo ":: searching for package '$arg'" + + this_dir="$(pwd)" + if db_remove_entry "$arg"; then + success=1 + else + echo "error: package matching '$arg' not found" + fi + cd $this_dir + fi + done + + # if all operations were a success, rezip database + if [ "$success" = "1" ]; then + echo ":: creating updated database file ${REPO_DB_FILE}" + cd $gstmpdir + if [ -n "$(ls)" ]; then + [ -f "${REPO_DB_FILE}.old" ] && rm "${REPO_DB_FILE}.old" + [ -f "$REPO_DB_FILE" ] && mv "$REPO_DB_FILE" "${REPO_DB_FILE}.old" + case "$DB_COMPRESSION" in + gz) tar c * | gzip -9 >$REPO_DB_FILE ;; + bz2) tar c * | bzip2 -9 >$REPO_DB_FILE ;; + *) echo "warning: no compression set" + tar c * >$REPO_DB_FILE;; + esac + fi + else + echo ":: no packages modified, nothing to do" + 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/doc (pacman.8)
by dan＠archlinux.org 11 Feb '07

11 Feb '07

Date: Saturday, February 10, 2007 @ 19:27:45 Author: dan Path: /home/cvs-pacman/pacman-lib/doc Modified: pacman.8 (1.5 -> 1.6) A few more slight changes to the pacman man page ----------+ pacman.8 | 43 +++++++++++++++++++++++-------------------- 1 file changed, 23 insertions(+), 20 deletions(-) Index: pacman-lib/doc/pacman.8 diff -u pacman-lib/doc/pacman.8:1.5 pacman-lib/doc/pacman.8:1.6 --- pacman-lib/doc/pacman.8:1.5 Thu Feb 8 22:44:54 2007 +++ pacman-lib/doc/pacman.8 Sat Feb 10 19:27:45 2007 @@ -13,18 +13,22 @@ .SH DESCRIPTION \fBpacman\fP is a \fIpackage management\fP utility that tracks installed -packages on a Linux system. It has dependency support, package groups, install -and uninstall hooks, and the ability to sync your local machine with a remote -ftp server to automatically upgrade packages. \fBpacman\fP packages are a -zipped tar format. +packages on a Linux system. It features dependency support, package groups, +install and uninstall hooks, and the ability to sync your local machine with a +remote ftp server to automatically upgrade packages. \fBpacman\fP packages are +a zipped tar format. + +Since version 3.0.0, \fBpacman\fP has been the frontend to \fBlibalpm\fP, the +"Arch Linux Package Management" library. This library allows alternative front +ends to be written (for instance, a GUI front end). .SH OPERATIONS .TP .B \-A, --add (deprecated) -Add a package to the system. Package will be uncompressed into the installation -root and the database will be updated. The package will not be installed if -another version is already installed. Please use \fB--upgrade\fP in place of -this option. +Add a package to the system. The package will be uncompressed into the +installation root and the database will be updated. The package will not be +installed if another version is already installed. Please use \fB--upgrade\fP +in place of this option. .TP .B \-F, --freshen This is like \fB--upgrade\fP except it will only upgrade packages already @@ -32,7 +36,7 @@ .TP .B \-Q, --query Query the package database. This operation allows you to view installed -packages and their files, as well as meta-info about individual packages +packages and their files, as well as meta-information about individual packages (dependencies, conflicts, install date, build date, size). This can be run against the local package database or can be used on individual .tar.gz packages. See \fBQUERY OPTIONS\fP below. @@ -45,7 +49,7 @@ .TP .B \-S, --sync Synchronize packages. Packages are installed directly from the ftp servers, -complete with all dependencies required to run the packages. For example, +including all dependencies required to run the packages. For example, \fBpacman -S qt\fP will download and install \fBqt\fP and all the packages it depends on. You can also use \fBpacman -Su\fP to upgrade all packages that are out of date. See \fBSYNC OPTIONS\fP below. @@ -76,7 +80,7 @@ used with care, ideally not at all. .TP .B \-r, --root \fIpath\fP -Specify alternative installation root (default is "/"). However, this should +Specify an alternative installation root (default is "/"). However, this should \fInot\fP be used as a way to install software into /usr/local instead of /usr, for example. This option should be used if you want to install a package on a temporary mounted partition, which is "owned" by another system. By using this @@ -104,8 +108,8 @@ longer required by any installed package. .TP .B \-g, --groups -Display all package members of a named group, or all grouped packages if -no name is specified. +Display all packages that are members of a named group. If not name is +specified, list all grouped packages. .TP .B \-i, --info Display information on a given package. The \fB-p\fP option can be used if @@ -125,7 +129,7 @@ .B \-p, --file Signifies that the package supplied on the command line is a file and not an entry in the database. The file will be decompressed and queried. This is -useful with \fB--info\fP and \fB--list\fP. +useful in combination with \fB--info\fP and \fB--list\fP. .TP .B \-s, --search \fIregexp\fP This will search each locally-installed package for names or descriptions that @@ -150,10 +154,9 @@ renamed with a .pacsave extension. .TP .B \-s, --recursive -For each target specified, remove it and all its dependencies, provided that -(A) they are not required by other packages; and (B) they were not explicitly -installed by the user. This option is analogous to a backwards \fB--sync\fP -operation. +Remove each target specified including all dependencies, provided that (A) they +are not required by other packages; and (B) they were not explicitly installed +by the user. This option is analogous to a backwards \fB--sync\fP operation. .SH SYNC OPTIONS .TP @@ -177,8 +180,8 @@ .TP .B \-p, --print-uris Print out URIs for each package that will be installed, including any -dependencies that have yet to be installed. These can be piped to a file and -downloaded at a later time, using a program like wget. +dependencies yet to be installed. These can be piped to a file and downloaded +at a later time, using a program like wget. .TP .B \-s, --search \fIregexp\fP This will search each package in the sync databases for names or descriptions

1 0

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

10 Feb '07

Date: Saturday, February 10, 2007 @ 18:50:31 Author: aaron Path: /home/cvs-pacman/pacman-lib/lib/libalpm Modified: package.h (1.27 -> 1.28) -----------+ package.h | 8 ++++++++ 1 file changed, 8 insertions(+) Index: pacman-lib/lib/libalpm/package.h diff -u pacman-lib/lib/libalpm/package.h:1.27 pacman-lib/lib/libalpm/package.h:1.28 --- pacman-lib/lib/libalpm/package.h:1.27 Wed Feb 7 00:57:28 2007 +++ pacman-lib/lib/libalpm/package.h Sat Feb 10 18:50:31 2007 @@ -84,6 +84,14 @@ /* internal */ pmpkgfrom_t origin; void *data; + /* TODO replace 'data' with this: + origin == PKG_FROM_CACHE, use pkg->origin_data.db + origin == PKG_FROM_FILE, use pkg->origin_data.fd + union { + int fd; + pmdb_t *db; + } origin_data; + */ pmdbinfrq_t infolevel; };

1 0

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

10 Feb '07

Date: Saturday, February 10, 2007 @ 18:44:40 Author: aaron Path: /home/cvs-pacman/pacman-lib/lib/libalpm Modified: add.c (1.112 -> 1.113) package.c (1.65 -> 1.66) * Package file parsing - fixed size and isize - isize is the "size" variable from the PKGINFO, and size is the stat() size of the archive * Removed the useless 'output' param from package.c:parse_descfile * Installation progress - Call progress callback once at 0% for initialization - 'needdisp' was useless - alpm_list_count is called an excessive amount in these nested loops. Now we only call it once per iteration - Use the compressed sizes for PROGRESS calcs as uncompressed (isize) is not exact (it is missing metadata sizes), and thus produces > 100% numbers -----------+ add.c | 41 +++++++++++++++++++++++------------------ package.c | 26 +++++++++++++------------- 2 files changed, 36 insertions(+), 31 deletions(-) Index: pacman-lib/lib/libalpm/add.c diff -u pacman-lib/lib/libalpm/add.c:1.112 pacman-lib/lib/libalpm/add.c:1.113 --- pacman-lib/lib/libalpm/add.c:1.112 Sat Feb 10 04:36:36 2007 +++ pacman-lib/lib/libalpm/add.c Sat Feb 10 18:44:39 2007 @@ -351,8 +351,7 @@ int _alpm_add_commit(pmtrans_t *trans, pmdb_t *db) { - int i, ret = 0, errors = 0, needdisp = 0; - double percent = 0.0; + int i, ret = 0, errors = 0, pkg_count = 0; register struct archive *archive; struct archive_entry *entry; char expath[PATH_MAX], cwd[PATH_MAX] = "", *what; @@ -369,7 +368,11 @@ return(0); } + pkg_count = alpm_list_count(trans->targets); + for(targ = trans->packages; targ; targ = targ->next) { + int targ_count = 0; + double percent = 0.0; unsigned short pmo_upgrade; char pm_install[PATH_MAX]; pmpkg_t *info = (pmpkg_t *)targ->data; @@ -468,13 +471,14 @@ _alpm_log(PM_LOG_DEBUG, _("extracting files")); /* Extract the package */ - if ((archive = archive_read_new ()) == NULL) + if ((archive = archive_read_new()) == NULL) { RET_ERR(PM_ERR_LIBARCHIVE_ERROR, -1); + } - archive_read_support_compression_all (archive); - archive_read_support_format_all (archive); + archive_read_support_compression_all(archive); + archive_read_support_format_all(archive); - if (archive_read_open_file (archive, info->data, ARCHIVE_DEFAULT_BYTES_PER_BLOCK) != ARCHIVE_OK) { + if(archive_read_open_file(archive, info->data, ARCHIVE_DEFAULT_BYTES_PER_BLOCK) != ARCHIVE_OK) { RET_ERR(PM_ERR_PKG_OPEN, -1); } @@ -488,6 +492,10 @@ /* libarchive requires this for extracting hard links */ chdir(handle->root); + targ_count = alpm_list_count(targ); + /* call PROGRESS once with 0 percent, as we sort-of skip that here */ + PROGRESS(trans, cb_state, what, 0, pkg_count, (pkg_count - targ_count +1)); + for(i = 0; archive_read_next_header (archive, &entry) == ARCHIVE_OK; i++) { int nb = 0; int notouch = 0; @@ -498,21 +506,19 @@ STRNCPY(pathname, archive_entry_pathname(entry), PATH_MAX); - if (info->size != 0) { - /* There's a problem here. These sizes don't match up. info->size is - * the COMPRESSED size, and info->isize is uncompressed. It appears, - * however, that these are the only two sizes available. It appears - * to be close enough, BUT easilly goes over 100%, so we'll stall - * there for now */ - percent = (double)archive_position_uncompressed(archive) / info->size; + if(info->size != 0) { + /* Using compressed size for calculations here, as info->isize is not + * exact when it comes to comparing to the ACTUAL uncompressed size + * (missing metadata sizes) */ + unsigned long pos = archive_position_compressed(archive); + percent = (double)pos / (double)info->size; + _alpm_log(PM_LOG_DEBUG, "decompression progress: %f%% (%ld / %ld)", percent*100.0, pos, info->size); if(percent >= 1.0) { percent = 1.0; } } - if (needdisp == 0) { - PROGRESS(trans, cb_state, what, (int)(percent * 100), alpm_list_count(trans->packages), (alpm_list_count(trans->packages) - alpm_list_count(targ) +1)); - } + PROGRESS(trans, cb_state, what, (int)(percent * 100), pkg_count, (pkg_count - targ_count +1)); if(strcmp(pathname, ".PKGINFO") == 0 || strcmp(pathname, ".FILELIST") == 0) { archive_read_data_skip (archive); @@ -855,8 +861,7 @@ } } - PROGRESS(trans, cb_state, what, 100, alpm_list_count(trans->packages), (alpm_list_count(trans->packages) - alpm_list_count(targ) +1)); - needdisp = 0; + PROGRESS(trans, cb_state, what, 100, pkg_count, (pkg_count - targ_count +1)); EVENT(trans, PM_TRANS_EVT_EXTRACT_DONE, NULL, NULL); FREE(what); Index: pacman-lib/lib/libalpm/package.c diff -u pacman-lib/lib/libalpm/package.c:1.65 pacman-lib/lib/libalpm/package.c:1.66 --- pacman-lib/lib/libalpm/package.c:1.65 Fri Feb 9 16:54:57 2007 +++ pacman-lib/lib/libalpm/package.c Sat Feb 10 18:44:39 2007 @@ -30,6 +30,9 @@ #include <libintl.h> #include <locale.h> #include <errno.h> +#include <sys/types.h> +#include <sys/stat.h> +#include <unistd.h> /* pacman */ #include "package.h" #include "log.h" @@ -177,7 +180,7 @@ * Returns: 0 on success, 1 on error * */ -static int parse_descfile(char *descfile, pmpkg_t *info, int output) +static int parse_descfile(char *descfile, pmpkg_t *info) { FILE* fp = NULL; char line[PATH_MAX]; @@ -199,14 +202,11 @@ if(strlen(line) == 0 || line[0] == '#') { continue; } - if(output) { - _alpm_log(PM_LOG_DEBUG, "%s", line); - } ptr = line; key = strsep(&ptr, "="); if(key == NULL || ptr == NULL) { _alpm_log(PM_LOG_DEBUG, _("%s: syntax error in description file line %d"), - info->name[0] != '\0' ? info->name : "error", linenum); + info->name[0] != '\0' ? info->name : "error", linenum); } else { _alpm_strtrim(key); key = _alpm_strtoupper(key); @@ -245,12 +245,7 @@ } else if(!strcmp(key, "ARCH")) { STRNCPY(info->arch, ptr, sizeof(info->arch)); } else if(!strcmp(key, "SIZE")) { - char tmp[32]; - STRNCPY(tmp, ptr, sizeof(tmp)); - info->size = atol(ptr); - } else if(!strcmp(key, "ISIZE")) { - char tmp[32]; - STRNCPY(tmp, ptr, sizeof(tmp)); + /* size in the raw package is uncompressed (installed) size */ info->isize = atol(ptr); } else if(!strcmp(key, "DEPEND")) { info->depends = alpm_list_add(info->depends, strdup(ptr)); @@ -266,7 +261,7 @@ info->backup = alpm_list_add(info->backup, strdup(ptr)); } else { _alpm_log(PM_LOG_DEBUG, _("%s: syntax error in description file line %d"), - info->name[0] != '\0' ? info->name : "error", linenum); + info->name[0] != '\0' ? info->name : "error", linenum); } } line[0] = '\0'; @@ -290,6 +285,7 @@ char *descfile = NULL; int fd = -1; alpm_list_t *all_files = NULL; + struct stat st; ALPM_LOG_FUNC; @@ -314,6 +310,10 @@ RET_ERR(PM_ERR_MEMORY, NULL); } + if(stat(pkgfile, &st) == 0) { + info->size = st.st_size; + } + /* 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 @@ -333,7 +333,7 @@ fd = mkstemp(descfile); archive_read_data_into_fd (archive, fd); /* parse the info file */ - if(parse_descfile(descfile, info, 0) == -1) { + if(parse_descfile(descfile, info) == -1) { _alpm_log(PM_LOG_ERROR, _("could not parse the package description file")); goto pkg_invalid; }

1 0

[pacman-dev] CVS update of pacman-lib/src/pacman (4 files)
by Aaron Griffin 10 Feb '07

10 Feb '07

Date: Saturday, February 10, 2007 @ 04:34:59 Author: aaron Path: /home/cvs-pacman/pacman-lib/src/pacman Modified: Makefile.am (1.13 -> 1.14) downloadprog.c (1.13 -> 1.14) log.c (1.28 -> 1.29) util.c (1.28 -> 1.29) Cleaned up some more output * Questions no longer start with "error:" * downloaded size is output as a float now ----------------+ Makefile.am | 8 ++++---- downloadprog.c | 20 +++++++++++--------- log.c | 2 +- util.c | 4 +++- 4 files changed, 19 insertions(+), 15 deletions(-) Index: pacman-lib/src/pacman/Makefile.am diff -u pacman-lib/src/pacman/Makefile.am:1.13 pacman-lib/src/pacman/Makefile.am:1.14 --- pacman-lib/src/pacman/Makefile.am:1.13 Mon Jan 29 17:18:32 2007 +++ pacman-lib/src/pacman/Makefile.am Sat Feb 10 04:34:59 2007 @@ -16,10 +16,10 @@ pacman_static_SOURCES = $(pacman_SOURCES) -pacman_LDADD = -L$(top_builddir)/lib/libalpm/.libs \ - -ldownload -lalpm +pacman_LDADD = $(top_builddir)/lib/libalpm/.libs/libalpm.so \ + -ldownload -pacman_static_LDADD = -L$(top_builddir)/lib/libalpm/.libs/ \ - -ldownload -lalpm +pacman_static_LDADD = $(top_builddir)/lib/libalpm/.libs/libalpm.la \ + -ldownload pacman_static_LDFLAGS = $(LDFLAGS) -all-static Index: pacman-lib/src/pacman/downloadprog.c diff -u pacman-lib/src/pacman/downloadprog.c:1.13 pacman-lib/src/pacman/downloadprog.c:1.14 --- pacman-lib/src/pacman/downloadprog.c:1.13 Thu Feb 8 15:44:47 2007 +++ pacman-lib/src/pacman/downloadprog.c Sat Feb 10 04:34:59 2007 @@ -53,7 +53,7 @@ const int infolen = 50; char *fname, *p; - float rate = 0.0, timediff = 0.0; + float rate = 0.0, timediff = 0.0, f_xfered = 0.0; unsigned int eta_h = 0, eta_m = 0, eta_s = 0; int percent; char rate_size = 'K', xfered_size = 'K'; @@ -112,11 +112,13 @@ eta_s -= eta_m * 60; fname = strdup(filename); + /* strip extension if it's there + * NOTE: in the case of package files, only the pkgname is sent now */ if((p = strstr(fname, PM_EXT_PKG)) || (p = strstr(fname, PM_EXT_DB))) { *p = '\0'; } if(strlen(fname) > FILENAME_TRIM_LEN) { - fname[FILENAME_TRIM_LEN] = '\0'; + strcpy(fname + FILENAME_TRIM_LEN -3,"..."); } /* Awesome formatting for progress bar. We need a mess of Kb->Mb->Gb stuff @@ -132,13 +134,13 @@ } } - xfered /= 1024; /* convert to K by default */ + f_xfered = (float) xfered / 1024.0; /* convert to K by default */ /* xfered_size = 'K'; was set above */ - if(xfered > 2048) { - xfered /= 1024; + if(f_xfered > 2048.0) { + f_xfered /= 1024.0; xfered_size = 'M'; - if(xfered > 2048) { - xfered /= 1024; + if(f_xfered > 2048.0) { + f_xfered /= 1024.0; xfered_size = 'G'; /* I should seriously hope that archlinux packages never break * the 9999.9GB mark... we'd have more serious problems than the progress @@ -146,8 +148,8 @@ } } - printf(" %-*s %6d%c %#6.1f%c/s %02u:%02u:%02u", FILENAME_TRIM_LEN, fname, - xfered, xfered_size, rate, rate_size, eta_h, eta_m, eta_s); + printf(" %-*s %6.1f%c %#6.1f%c/s %02u:%02u:%02u", FILENAME_TRIM_LEN, fname, + f_xfered, xfered_size, rate, rate_size, eta_h, eta_m, eta_s); free(fname); Index: pacman-lib/src/pacman/log.c diff -u pacman-lib/src/pacman/log.c:1.28 pacman-lib/src/pacman/log.c:1.29 --- pacman-lib/src/pacman/log.c:1.28 Fri Feb 9 11:02:01 2007 +++ pacman-lib/src/pacman/log.c Sat Feb 10 04:34:59 2007 @@ -175,7 +175,7 @@ va_end(args); /* Use stderr so questions are always displayed when redirecting output */ - ERR(NL, str); + pm_fprintf(stderr, NL, str); \ if(fgets(response, 32, stdin)) { /* trim whitespace and newlines */ Index: pacman-lib/src/pacman/util.c diff -u pacman-lib/src/pacman/util.c:1.28 pacman-lib/src/pacman/util.c:1.29 --- pacman-lib/src/pacman/util.c:1.28 Thu Feb 8 15:44:48 2007 +++ pacman-lib/src/pacman/util.c Sat Feb 10 04:34:59 2007 @@ -418,10 +418,12 @@ printf("-"); } } - printf("] %3d%%\r", percent); + printf("] %3d%%", percent); if(percent == 100) { printf("\n"); + } else { + printf("\r"); } fflush(stdout); }

2 1

[pacman-dev] CVS update of pacman-lib/lib/libalpm (add.c alpm.c server.c)
by Aaron Griffin 10 Feb '07

10 Feb '07

Date: Saturday, February 10, 2007 @ 04:36:36 Author: aaron Path: /home/cvs-pacman/pacman-lib/lib/libalpm Modified: add.c (1.111 -> 1.112) alpm.c (1.115 -> 1.116) server.c (1.20 -> 1.21) * 'add' percentages now cap at 100% (see the comment) * downloaded packages now pass ONLY the package name to the callback ----------+ add.c | 16 +++++++++++++--- alpm.c | 9 +++++---- server.c | 15 +++++++++++++-- 3 files changed, 31 insertions(+), 9 deletions(-) Index: pacman-lib/lib/libalpm/add.c diff -u pacman-lib/lib/libalpm/add.c:1.111 pacman-lib/lib/libalpm/add.c:1.112 --- pacman-lib/lib/libalpm/add.c:1.111 Fri Feb 9 16:08:10 2007 +++ pacman-lib/lib/libalpm/add.c Sat Feb 10 04:36:36 2007 @@ -496,10 +496,20 @@ char pathname[PATH_MAX]; struct stat buf; - STRNCPY(pathname, archive_entry_pathname (entry), PATH_MAX); + STRNCPY(pathname, archive_entry_pathname(entry), PATH_MAX); - if (info->size != 0) - percent = (double)archive_position_uncompressed(archive) / info->size; + if (info->size != 0) { + /* There's a problem here. These sizes don't match up. info->size is + * the COMPRESSED size, and info->isize is uncompressed. It appears, + * however, that these are the only two sizes available. It appears + * to be close enough, BUT easilly goes over 100%, so we'll stall + * there for now */ + percent = (double)archive_position_uncompressed(archive) / info->size; + if(percent >= 1.0) { + percent = 1.0; + } + } + if (needdisp == 0) { PROGRESS(trans, cb_state, what, (int)(percent * 100), alpm_list_count(trans->packages), (alpm_list_count(trans->packages) - alpm_list_count(targ) +1)); } Index: pacman-lib/lib/libalpm/alpm.c diff -u pacman-lib/lib/libalpm/alpm.c:1.115 pacman-lib/lib/libalpm/alpm.c:1.116 --- pacman-lib/lib/libalpm/alpm.c:1.115 Fri Feb 9 16:54:57 2007 +++ pacman-lib/lib/libalpm/alpm.c Sat Feb 10 04:36:36 2007 @@ -593,14 +593,15 @@ arch = _supported_archs[i]; /* whee, case insensitive compare */ - while(*arch && *cmp && tolower(*arch++) == tolower(*cmp++)) ; - if(*arch || *cmp) continue; + if(*arch || *cmp) { + continue; + } - return p; + return(p); } } - return NULL; + return(NULL); } /** @} */ Index: pacman-lib/lib/libalpm/server.c diff -u pacman-lib/lib/libalpm/server.c:1.20 pacman-lib/lib/libalpm/server.c:1.21 --- pacman-lib/lib/libalpm/server.c:1.20 Fri Feb 9 05:10:49 2007 +++ pacman-lib/lib/libalpm/server.c Sat Feb 10 04:36:36 2007 @@ -37,6 +37,7 @@ #include "util.h" #include "handle.h" #include "log.h" +#include "package.h" pmserver_t *_alpm_server_new(const char *url) { @@ -180,12 +181,22 @@ char realfile[PATH_MAX]; char output[PATH_MAX]; char *fn = (char *)lp->data; + char pkgname[PKG_NAME_LEN]; + char *p; fileurl = url_for_file(server, fn); if(!fileurl) { return(-1); } + /* Try to get JUST the name of the package from the filename */ + p = alpm_pkg_name_hasarch(fn); /* TODO remove this later */ + _alpm_pkg_splitname(fn, pkgname, NULL, (p != NULL)); + if(!strlen(pkgname)) { + /* just use the raw filename if we can't find crap */ + STRNCPY(pkgname, fn, PKG_NAME_LEN); + } + snprintf(realfile, PATH_MAX, "%s/%s", localpath, fn); snprintf(output, PATH_MAX, "%s/%s.part", localpath, fn); @@ -278,7 +289,7 @@ } /* Progress 0 - initialize */ - if(handle->dlcb) handle->dlcb(fn, 0, ust.size); + if(handle->dlcb) handle->dlcb(pkgname, 0, ust.size); int nread = 0; char buffer[PM_DLBUF_LEN]; @@ -287,7 +298,7 @@ while((nwritten += fwrite(buffer, 1, (nread - nwritten), localf)) < nread) ; dltotal_bytes += nread; - if(handle->dlcb) handle->dlcb(fn, dltotal_bytes, ust.size); + if(handle->dlcb) handle->dlcb(pkgname, dltotal_bytes, ust.size); } fclose(localf);

1 0

[pacman-dev] CVS update of pacman-lib (Makefile.am)
by Juergen Hoetzel 10 Feb '07

10 Feb '07

Date: Saturday, February 10, 2007 @ 03:10:22 Author: juergen Path: /home/cvs-pacman/pacman-lib Modified: Makefile.am (1.10 -> 1.11) * fixed dist target -------------+ Makefile.am | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) Index: pacman-lib/Makefile.am diff -u pacman-lib/Makefile.am:1.10 pacman-lib/Makefile.am:1.11 --- pacman-lib/Makefile.am:1.10 Wed Feb 7 17:29:21 2007 +++ pacman-lib/Makefile.am Sat Feb 10 03:10:22 2007 @@ -1,11 +1,9 @@ SUBDIRS = lib/libalpm src/util src/pacman scripts doc etc EXTRA_DIST = \ - ChangeLog \ + NEWS \ COPYING \ - README \ - TODO \ - TODO.autoconf + README check: src/pacman cd pactest; python pactest.py --test=tests/*.py -p ../src/pacman/pacman --debug=-1

1 0

[pacman-dev] Man pages (and a quick ML note)
by Dan McGee 10 Feb '07

10 Feb '07

Hey guys, I've been working on the manpages a lot in the past day or two, and I believe they are pretty up to date now with the changes that have been made. I'd appreciate people looking them over, giving me some (constructive) criticism, and pointing anything out that needs changing, updating, or a typo fixed, etc. If it is small, just send an email telling me what needs fixing, if large, feel free to submit a patch (but if you are unfamiliar with the man page format, then just describe the change and I'll implement it). On a side note, if the topic of a thread changes on the ML, can you guys update the subject before you reply? That way talk about bug #4744 doesn't end up in the bug #5775 thread, and makes keeping track of multiple topics a lot easier. -Dan

3 6

[pacman-dev] Fwd: FS#6378: pacman 3: output issues
by Dan McGee 09 Feb '07

09 Feb '07

> FS#6378 - pacman 3: output issues > User who did this: - Roman Kyrylych (Romashka) > > ---------- > 1) I meant that it would be nice to have output like 2,0M -> 2,3M -> > 2,7M -> 3,2M, than just 2M -> 3M That is what the int -> float change does, if you know the C language types- it allows decimal places. > 2) I don't understand what you mean by "translate from language to > language" :-/ Does "..." mean "name continues but it is too long" in all languages? Of course, gettext should be able to translate this string, but the original question still stands if there is a punctuation mark in all languages used for the same purpose the English "..." is. -Dan

2 1

Re: [pacman-dev] FS#6316: Pacman 3 release bugcatcher
by Dan McGee 09 Feb '07

09 Feb '07

> The following task has a new comment added: > > FS#6316 - Pacman 3 release bugcatcher > User who did this: - Roman Kyrylych (Romashka) > > ---------- > I added Total Download Size feature request because it can be > implemented along with other output fixes/changes. > ---------- > > More information can be found at the following URL: > http://bugs.archlinux.org/task/6316#comment14228 Roman, the problem with this is it truely is a feature request- it may not be that easy to add at this point in the development cycle. I believe when the other sizes are presented, we have done nothing yet to see if those packages are cached or not, so thus do not know the actual download size. It may have to wait for pacman 3.1. -Dan

3 2