May 2009 - Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH] Remove unused string.h from src/pacman/remove.c
by Gerardo Exequiel Pozzi 01 Jul '09

01 Jul '09

This header was used in the code for the function strdup() that is not used anymore. Signed-off-by: Gerardo Exequiel Pozzi <vmlinuz386(a)yahoo.com.ar> --- src/pacman/remove.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/src/pacman/remove.c b/src/pacman/remove.c index 11bc157..1968744 100644 --- a/src/pacman/remove.c +++ b/src/pacman/remove.c @@ -21,7 +21,6 @@ #include <stdlib.h> #include <stdio.h> -#include <string.h> #include <alpm.h> #include <alpm_list.h> -- 1.6.3.1

2 5

[pacman-dev] [PATCH] NEWS file updates for makepkg and contrib
by Allan McRae 01 Jul '09

01 Jul '09

Signed-off-by: Allan McRae <allan(a)archlinux.org> --- We should really enforce that people do this when they submit patches... NEWS | 20 ++++++++++++++++++++ 1 files changed, 20 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index f0a88fd..c2eb8af 100644 --- a/NEWS +++ b/NEWS @@ -1,5 +1,25 @@ VERSION DESCRIPTION ----------------------------------------------------------------------------- +3.3.0 - makepkg: + - package splitting support! - see PKGBUILD-split.proto + - limit fakeroot usage with addition of package() function + - info pages are handled like man pages and not other documentation + - configuration option for man/info page directories + - added ability to automatically remove files from package + - configuration option for default LDFLAGS + - specify alternative configuration file with --config flag + - check all integrity checksums provided in PKGBUILD + - fix pkgver/pkgrel updating in SCM packages + - BUILDSCRIPT option removed from makepkg.conf. Now specified + during configure + - enforce no ">" or "<" in provides array + - package compression autodetection + - check PKGBUILD for CRLF line endings + - fix reading PKGBUILD from pipe + - various fixes to increase compatibility with BSDs & Mac OSX + - contrib + - pacscripts - print install scripts for a package + 3.2.2 - log pacsave warnings to pacman.log (FS#12531) - separate local DB creation and writing (FS#12263) - pacman-optimize: rewrite and refresh (FS#11767) -- 1.6.3.1

3 4

[pacman-dev] Mime-type filtering?
by Aaron Griffin 30 Jun '09

30 Jun '09

On Mon, Mar 23, 2009 at 5:11 PM, Dan McGee <dpmcgee(a)gmail.com> wrote: > 2009/3/23 Samed Beyribey <ras0ir(a)eventualis.org>: >> Well i just noticed that, (probably) mailman rejects my attachment (it >> was 9.5k) >> so i've put it on my web site: >> http://eventualis.org/pacman/0001-fixed-typos-and-alignment-problem-in-Turk… >> >> This is my last mail about this patch, again, i feel so sorry. > > This isn't your fault. > > Aaron, do we not allow gzipped and bzipped files through? I guess not. What's the mime-type for these? Considering mailman uses a whitelist, would anyone be able to construct a decent list that covers pretty much all attachment types we'd want?

3 5

[pacman-dev] Bump on upgrade046.py
by Nagy Gabor 01 Jun '09

01 Jun '09

Hi! I think this bug has just hurted my system today. Since I have no net in my sublet, I do -Su very rarely, so today's sysupgrade upgraded 415 packages (my system was ~0.7 year-old). I got lots of false fileconflict errors first (probably due to the symlink bug). I was lazy to resolve all of them by hand, and since I never put untracked files out of my home dir, I thought -Suf cannot hurt anything, so I just did -Suf. During sysupgrade, lots of "gconftool-2: error while loading shared libraries: libdbus-1.so.3: cannot open shared object file: No such file or directory" messages popped up (install scriptlets). Wow :-(. After pacman -Suf had finished, I did some investigation (I didn't want to press ctrl-c during sysupgrade). First I thought, that there is a missing dependency around gconf (testdb said that my local database was consistent), but I was wrong: I had the package dbus-core installed (up-to-date version), which should contain the missing lib (-Ql), but the file libdbus-1.so.3 was not found in /usr/lib. I guess, I've just run into the file relocation bug (otherwise, we have a more serious bug here), showed by upgrade046.py. Briefly, the file relocation check is hacked into the fileconflict check, which is omitted with -f. Bye P.S.: I used pacman 3.2.2-1.

2 2

[pacman-dev] [PATCH] Change package to package(s) and file to file(s) in documentation
by Nagy Gabor 22 May '09

22 May '09

From 2f51241a85f1df7219ae0649bff9a0de369a8c84 Mon Sep 17 00:00:00 2001 From: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Fri, 22 May 2009 01:06:33 +0200 Subject: [PATCH] Change package to package(s) and file to file(s) in documentation The pacman --help pages and the manual suggested that only one package can be upgraded/removed per transaction. Signed-off-by: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> --- doc/pacman.8.txt | 4 ++-- src/pacman/pacman.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/doc/pacman.8.txt b/doc/pacman.8.txt index 4b7cf2d..195135b 100644 --- a/doc/pacman.8.txt +++ b/doc/pacman.8.txt @@ -45,7 +45,7 @@ Operations list. See <<QO,Query Options>> below. *-R, \--remove*:: - Remove a package from the system. Groups can also be specified to be + Remove package(s) from the system. Groups can also be specified to be removed, in which case every package in that group will be removed. Files belonging to the specified package will be deleted, and the database will be updated. Most configuration files will be saved @@ -81,7 +81,7 @@ to determine which packages need upgrading. This behavior operates as follows: 1 < 1.0 < 1.1 < 1.1.1 < 1.2 < 2.0 < 3.0.0 *-U, \--upgrade*:: - Upgrade or add a package to the system. Either a URL or file path can be + Upgrade or add package(s) to the system. Either a URL or file path can be specified. This is a ``remove-then-add'' process. See <<HCF,Handling Config Files>> for an explanation on how pacman takes care of config files. diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c index c817828..1629171 100644 --- a/src/pacman/pacman.c +++ b/src/pacman/pacman.c @@ -65,8 +65,8 @@ static void usage(int op, const char * const myname) /* prefetch some strings for usage below, which moves a lot of calls * out of gettext. */ char const * const str_opt = _("options"); - char const * const str_file = _("file"); - char const * const str_pkg = _("package"); + char const * const str_file = _("file(s)"); + char const * const str_pkg = _("package(s)"); char const * const str_usg = _("usage"); char const * const str_opr = _("operation"); -- 1.6.3.1

1 0

[pacman-dev] [PATCH] Introduce -D, --database
by Nagy Gabor 21 May '09

21 May '09

>From d20f84a86432bee553abdc2c485d953dc9ec6b3f Mon Sep 17 00:00:00 2001 From: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> Date: Wed, 20 May 2009 21:46:23 +0200 Subject: [PATCH] Introduce -D, --database The request of FS#12950 is implemented. On the backend side, I introduced a new function, alpm_db_set_pkgreason(), to modify the install reason of a package in the local database. On the front-end side, I introduced a new main operation, -D/--database, which has two options, --asdeps and --asexplicit. I documented this in pacman manual. I've created two pactests to test -D: database001.py and database002.py. Signed-off-by: Nagy Gabor <ngaba(a)bibl.u-szeged.hu> --- doc/pacman.8.txt | 12 +++++- lib/libalpm/alpm.h | 16 +++++--- lib/libalpm/db.c | 34 ++++++++++++++++ pactest/tests/database001.py | 11 +++++ pactest/tests/database002.py | 11 +++++ src/pacman/Makefile.am | 1 + src/pacman/conf.h | 3 +- src/pacman/database.c | 90 ++++++++++++++++++++++++++++++++++++++++++ src/pacman/pacman.c | 22 ++++++++-- src/pacman/pacman.h | 2 + src/pacman/util.c | 2 +- 11 files changed, 189 insertions(+), 15 deletions(-) create mode 100644 pactest/tests/database001.py create mode 100644 pactest/tests/database002.py create mode 100644 src/pacman/database.c diff --git a/doc/pacman.8.txt b/doc/pacman.8.txt index e914ab4..4b7cf2d 100644 --- a/doc/pacman.8.txt +++ b/doc/pacman.8.txt @@ -28,6 +28,12 @@ front ends to be written (for instance, a GUI front end). Operations ---------- +*-D, \--database*:: + Modify the package database. This options allows you to modify certain + attributes of the installed packages in pacman's database. At the + moment, you can only change the install reason. See the '\--asdeps' + and '\--asexplicit' options below. + *-Q, \--query*:: Query the package database. This operation allows you to view installed packages and their files, as well as meta-information about individual @@ -93,13 +99,15 @@ Options Install packages non-explicitly; in other words, fake their install reason to be installed as a dependency. This is useful for makepkg and other build from source tools that need to install dependencies before building - the package. + the package. This option can be also used in combination with '-D' to + change the install reason of local packages without reinstalling. *\--asexplicit*:: Install packages explicitly; in other words, fake their install reason to be explicitly installed. This is useful if you want to mark a dependency as explicitly installed so it will not be removed by the '\--recursive' - remove operation. + remove operation. This option can be also used in combination with '-D' + to change the install reason of local packages without reinstalling. *-b, \--dbpath* <'path'>:: Specify an alternative database location (a typical default is diff --git a/lib/libalpm/alpm.h b/lib/libalpm/alpm.h index 1e22a59..94c3b67 100644 --- a/lib/libalpm/alpm.h +++ b/lib/libalpm/alpm.h @@ -147,6 +147,15 @@ pmdb_t *alpm_option_get_localdb(); alpm_list_t *alpm_option_get_syncdbs(); /* + * Install reasons -- ie, why the package was installed + */ + +typedef enum _pmpkgreason_t { + PM_PKG_REASON_EXPLICIT = 0, /* explicitly requested by the user */ + PM_PKG_REASON_DEPEND = 1 /* installed as a dependency for another package */ +} pmpkgreason_t; + +/* * Databases */ @@ -169,6 +178,7 @@ alpm_list_t *alpm_db_get_pkgcache(pmdb_t *db); pmgrp_t *alpm_db_readgrp(pmdb_t *db, const char *name); alpm_list_t *alpm_db_get_grpcache(pmdb_t *db); alpm_list_t *alpm_db_search(pmdb_t *db, const alpm_list_t* needles); +int alpm_db_set_pkgreason(pmdb_t *db, const char *name, pmpkgreason_t reason); /* * Packages @@ -176,12 +186,6 @@ alpm_list_t *alpm_db_search(pmdb_t *db, const alpm_list_t* needles); /* Info parameters */ -/* reasons -- ie, why the package was installed */ -typedef enum _pmpkgreason_t { - PM_PKG_REASON_EXPLICIT = 0, /* explicitly requested by the user */ - PM_PKG_REASON_DEPEND = 1 /* installed as a dependency for another package */ -} pmpkgreason_t; - int alpm_pkg_load(const char *filename, unsigned short full, pmpkg_t **pkg); int alpm_pkg_free(pmpkg_t *pkg); int alpm_pkg_checkmd5sum(pmpkg_t *pkg); diff --git a/lib/libalpm/db.c b/lib/libalpm/db.c index 561967c..941b56d 100644 --- a/lib/libalpm/db.c +++ b/lib/libalpm/db.c @@ -319,6 +319,40 @@ alpm_list_t SYMEXPORT *alpm_db_search(pmdb_t *db, const alpm_list_t* needles) return(_alpm_db_search(db, needles)); } +/* Set install reason for a package in db + * @param db pointer to the package database + * @param name the name of the package + * @param reason the new install reason + * @return 0 on success, -1 on error (pm_errno is set accordingly) + */ +int SYMEXPORT alpm_db_set_pkgreason(pmdb_t *db, const char *name, pmpkgreason_t reason) +{ + ALPM_LOG_FUNC; + + /* Sanity checks */ + ASSERT(handle != NULL, RET_ERR(PM_ERR_HANDLE_NULL, -1)); + ASSERT(db != NULL && name != NULL, RET_ERR(PM_ERR_WRONG_ARGS, -1)); + + pmpkg_t *pkg = _alpm_db_get_pkgfromcache(db, name); + if(pkg == NULL) { + RET_ERR(PM_ERR_PKG_NOT_FOUND, -1); + } + + _alpm_log(PM_LOG_DEBUG, "setting install reason %u for %s/%s\n", reason, db->treename, name); + /* read DESC */ + if(_alpm_db_read(db, pkg, INFRQ_DESC)) { + return(-1); + } + /* set reason (in pkgcache) */ + pkg->reason = reason; + /* write DESC */ + if(_alpm_db_write(db, pkg, INFRQ_DESC)) { + return(-1); + } + + return(0); +} + /** @} */ pmdb_t *_alpm_db_new(const char *dbpath, const char *treename) diff --git a/pactest/tests/database001.py b/pactest/tests/database001.py new file mode 100644 index 0000000..de4c040 --- /dev/null +++ b/pactest/tests/database001.py @@ -0,0 +1,11 @@ +self.description = "-D --asdeps" + +lp = pmpkg("pkg") +lp.reason = 0 +self.addpkg2db("local", lp) + +self.args = "-D pkg --asdeps" + +self.addrule("PACMAN_RETCODE=0") +self.addrule("PKG_EXIST=pkg") +self.addrule("PKG_REASON=pkg|1") diff --git a/pactest/tests/database002.py b/pactest/tests/database002.py new file mode 100644 index 0000000..05fa7f4 --- /dev/null +++ b/pactest/tests/database002.py @@ -0,0 +1,11 @@ +self.description = "-D --asexplicit" + +lp = pmpkg("pkg") +lp.reason = 1 +self.addpkg2db("local", lp) + +self.args = "-D pkg --asexplicit" + +self.addrule("PACMAN_RETCODE=0") +self.addrule("PKG_EXIST=pkg") +self.addrule("PKG_REASON=pkg|0") diff --git a/src/pacman/Makefile.am b/src/pacman/Makefile.am index 220ee9c..8c14562 100644 --- a/src/pacman/Makefile.am +++ b/src/pacman/Makefile.am @@ -24,6 +24,7 @@ endif pacman_SOURCES = \ conf.h conf.c \ + database.c \ deptest.c \ package.h package.c \ pacman.h pacman.c \ diff --git a/src/pacman/conf.h b/src/pacman/conf.h index 466d983..f6cbb85 100644 --- a/src/pacman/conf.h +++ b/src/pacman/conf.h @@ -80,7 +80,8 @@ enum { PM_OP_UPGRADE, PM_OP_QUERY, PM_OP_SYNC, - PM_OP_DEPTEST + PM_OP_DEPTEST, + PM_OP_DATABASE }; /* clean method */ diff --git a/src/pacman/database.c b/src/pacman/database.c new file mode 100644 index 0000000..89ea5d9 --- /dev/null +++ b/src/pacman/database.c @@ -0,0 +1,90 @@ +/* + * database.c + * + * Copyright (c) 2002-2007 by Judd Vinet <jvinet(a)zeroflux.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, see <http://www.gnu.org/licenses/>. + */ + +#include "config.h" + +#include <stdlib.h> +#include <stdio.h> +#include <string.h> + +#include <alpm.h> +#include <alpm_list.h> + +/* pacman */ +#include "pacman.h" +#include "conf.h" +#include "util.h" + +extern pmdb_t *db_local; + +/** + * @brief Modify the 'local' package database. + * + * @param targets a list of packages (as strings) to modify + * + * @return 0 on success, 1 on failure + */ +int pacman_database(alpm_list_t *targets) +{ + alpm_list_t *i; + int retval = 0; + pmpkgreason_t reason; + + if(targets == NULL) { + pm_printf(PM_LOG_ERROR, _("no targets specified (use -h for help)\n")); + return(1); + } + + if(config->flags & PM_TRANS_FLAG_ALLDEPS) { /* --asdeps */ + reason = PM_PKG_REASON_DEPEND; + } else if(config->flags & PM_TRANS_FLAG_ALLEXPLICIT) { /* --asexplicit */ + reason = PM_PKG_REASON_EXPLICIT; + } else { + pm_printf(PM_LOG_ERROR, _("no install reason specified (use -h for help)\n")); + return(1); + } + + /* Lock database */ + if(trans_init(PM_TRANS_TYPE_SYNC, 0) == -1) { + return(1); + } + + for(i = targets; i; i = alpm_list_next(i)) { + char *pkgname = i->data; + if(alpm_db_set_pkgreason(db_local, pkgname, reason)) { + pm_printf(PM_LOG_ERROR, _("could not set install reason for package %s (%s)\n"), + pkgname, alpm_strerrorlast()); + retval = 1; + } else { + if(reason == PM_PKG_REASON_DEPEND) { + printf(_("%s: install reason has been changed to 'installed as dependency'\n"), pkgname); + } else { + printf(_("%s: install reason has been changed to 'explicitly installed'\n"), pkgname); + } + } + } + + /* Unlock database */ + if(trans_release() == -1) { + return(1); + } + return(retval); +} + +/* vim: set ts=2 sw=2 noet: */ diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c index 3b8636c..c817828 100644 --- a/src/pacman/pacman.c +++ b/src/pacman/pacman.c @@ -75,10 +75,11 @@ static void usage(int op, const char * const myname) printf(_("operations:\n")); printf(" %s {-h --help}\n", myname); printf(" %s {-V --version}\n", myname); - printf(" %s {-Q --query} [%s] [%s]\n", myname, str_opt, str_pkg); - printf(" %s {-R --remove} [%s] <%s>\n", myname, str_opt, str_pkg); - printf(" %s {-S --sync} [%s] [%s]\n", myname, str_opt, str_pkg); - printf(" %s {-U --upgrade} [%s] <%s>\n", myname, str_opt, str_file); + printf(" %s {-D --database} <%s> <%s>\n", myname, str_opt, str_pkg); + printf(" %s {-Q --query} [%s] [%s]\n", myname, str_opt, str_pkg); + printf(" %s {-R --remove} [%s] <%s>\n", myname, str_opt, str_pkg); + printf(" %s {-S --sync} [%s] [%s]\n", myname, str_opt, str_pkg); + printf(" %s {-U --upgrade} [%s] <%s>\n", myname, str_opt, str_file); printf(_("\nuse '%s {-h --help}' with an operation for available options\n"), myname); } else { @@ -136,6 +137,11 @@ static void usage(int op, const char * const myname) printf(_(" --ignoregroup <grp>\n" " ignore a group upgrade (can be used more than once)\n")); printf(_(" -q, --quiet show less information for query and search\n")); + } else if (op == PM_OP_DATABASE) { + printf("%s: %s {-D --database} <%s> <%s>\n", str_usg, myname, str_opt, str_pkg); + printf("%s:\n", str_opt); + printf(_(" --asdeps mark packages as non-explicitly installed\n")); + printf(_(" --asexplicit mark packages as explicitly installed\n")); } printf(_(" --config <path> set an alternate configuration file\n")); printf(_(" --logfile <path> set an alternate log file\n")); @@ -326,6 +332,8 @@ static int parseargs(int argc, char *argv[]) int option_index = 0; static struct option opts[] = { + + {"database", no_argument, 0, 'D'}, {"query", no_argument, 0, 'Q'}, {"remove", no_argument, 0, 'R'}, {"sync", no_argument, 0, 'S'}, @@ -376,7 +384,7 @@ static int parseargs(int argc, char *argv[]) {0, 0, 0, 0} }; - while((opt = getopt_long(argc, argv, "RUFQSTr:b:vkhscVfmnoldepqituwygz", opts, &option_index))) { + while((opt = getopt_long(argc, argv, "RUDQSTr:b:vkhscVfmnoldepqituwygz", opts, &option_index))) { alpm_list_t *list = NULL, *item = NULL; /* lists for splitting strings */ if(opt < 0) { @@ -447,6 +455,7 @@ static int parseargs(int argc, char *argv[]) case 1012: config->flags |= PM_TRANS_FLAG_ALLEXPLICIT; break; + case 'D': config->op = (config->op != PM_OP_MAIN ? 0 : PM_OP_DATABASE); break; case 'Q': config->op = (config->op != PM_OP_MAIN ? 0 : PM_OP_QUERY); break; case 'R': config->op = (config->op != PM_OP_MAIN ? 0 : PM_OP_REMOVE); break; case 'S': config->op = (config->op != PM_OP_MAIN ? 0 : PM_OP_SYNC); break; @@ -932,6 +941,9 @@ int main(int argc, char *argv[]) /* start the requested operation */ switch(config->op) { + case PM_OP_DATABASE: + ret = pacman_database(pm_targets); + break; case PM_OP_REMOVE: ret = pacman_remove(pm_targets); break; diff --git a/src/pacman/pacman.h b/src/pacman/pacman.h index 97d0301..7c7801d 100644 --- a/src/pacman/pacman.h +++ b/src/pacman/pacman.h @@ -21,6 +21,8 @@ #include <alpm_list.h> +/* database.c */ +int pacman_database(alpm_list_t *targets); /* query.c */ int pacman_query(alpm_list_t *targets); /* remove.c */ diff --git a/src/pacman/util.c b/src/pacman/util.c index 8056fbd..b7248d7 100644 --- a/src/pacman/util.c +++ b/src/pacman/util.c @@ -71,7 +71,7 @@ int trans_release(void) int needs_root(void) { - if(config->op == PM_OP_UPGRADE || config->op == PM_OP_REMOVE || /* -U, -R */ + if(config->op == PM_OP_UPGRADE || config->op == PM_OP_REMOVE || config->op == PM_OP_DATABASE || /* -U, -R, -D */ (config->op == PM_OP_SYNC && (config->op_s_clean || config->op_s_sync || /* -Sc, -Sy */ (!config->group && !config->op_s_info && !config->op_q_list /* all other -S combinations, where */ && !config->op_s_search && !config->op_s_printuris)))) { /* -g, -i, -l, -s, -p is not set */ -- 1.6.0.3

1 1

[pacman-dev] [PATCH 1/2] Don't print prompt with noconfirm
by Brendan Hide 20 May '09

20 May '09

Remove extraneous checks to config->noconfirm, shouldn't prompt user if noconfirm is set. Returns appropriate value immediately rather than later. Signed-off-by: Brendan Hide <brendan(a)swiftspirit.co.za> --- src/pacman/util.c | 13 ++++--------- 1 files changed, 4 insertions(+), 9 deletions(-) diff --git a/src/pacman/util.c b/src/pacman/util.c index b80b09a..9f2c6b5 100644 --- a/src/pacman/util.c +++ b/src/pacman/util.c @@ -645,12 +645,12 @@ static int question(short preset, char *fmt, va_list args) FILE *stream; if(config->noconfirm) { - stream = stdout; - } else { - /* Use stderr so questions are always displayed when redirecting output */ - stream = stderr; + return(preset); } + /* Use stderr so questions are always displayed when redirecting output */ + stream = stderr; + vfprintf(stream, fmt, args); if(preset) { @@ -659,11 +659,6 @@ static int question(short preset, char *fmt, va_list args) fprintf(stream, " %s ", _("[y/N]")); } - if(config->noconfirm) { - fprintf(stream, "\n"); - return(preset); - } - if(fgets(response, 32, stdin)) { strtrim(response); if(strlen(response) == 0) { -- 1.6.3.1

1 0

[pacman-dev] noconfirm trivial bug - prompt printed but unused/unnecessary
by Brendan Hide 20 May '09

20 May '09

Hi all I'm still getting my head around git and pacman-dev's workflow - I know its not that complicated, just need a bit of time. When using --noconfirm the prompt is presented (odd part) and the user (rightly) is not given an opportunity to confirm: > [ brendan@swift : 12:36:40 : ~/src/pacman ] > :) sudo pacman -Sc --noconfirm > Cache directory: /var/cache/pacman/pkg/ > Do you want to remove outdated packages from cache? > [Y/n] <------ > removing old packages from cache... done. > Database directory: /var/lib/pacman/ > Do you want to remove unused repositories? [Y/n] <------ > Database directory cleaned up > [ brendan@swift : 12:36:57 : ~/src/pacman ] > :) Best fix I can think of is to just mention that we're using a default because the --noconfirm flag has been used. The alternative (which is less informative for the user) is to not present the prompt at all. I've inserted 2 strings below. Assuming this is the right change to make, what else would I need to do to ensure that this is compatible with the i18n measures already in place? util.c, lines 641-680: > /* presents a prompt and gets a Y/N answer */ > static int question(short preset, char *fmt, va_list args) > { > char response[32]; > FILE *stream; > > if(config->noconfirm) { > stream = stdout; > } else { > /* Use stderr so questions are always displayed when > redirecting output */ > stream = stderr; > } > > vfprintf(stream, fmt, args); > > if(preset) { > fprintf(stream, " %s ", _("[Y/n]")); > } else { > fprintf(stream, " %s ", _("[y/N]")); > } > > if(config->noconfirm) { > fprintf(stream, "\n"); Replace the 'fprintf(stream, "\n");' line above with something like: if(preset) { fprintf(stream, "%s", _("noconfirm option set, defaulting to Yes\n")); } else { fprintf(stream, "%s", _("noconfirm option set, defaulting to No\n")); } > return(preset); > } > > if(fgets(response, 32, stdin)) { > strtrim(response); > if(strlen(response) == 0) { > return(preset); > } > > if(!strcasecmp(response, _("Y")) || > !strcasecmp(response, _("YES"))) { > return(1); > } else if (!strcasecmp(response, _("N")) || > !strcasecmp(response, _("NO"))) { > return(0); > } > } > return(0); > } -- __________ Brendan Hide

3 4

Re: [pacman-dev] [arch-general] Marking a package as a dependency
by Xavier 20 May '09

20 May '09

On Wed, May 20, 2009 at 9:50 AM, Magnus Therning <magnus(a)therning.org> wrote: > I just recently bothered reading part of pacman's man page and found > that I probably should have performed a `pacman -U --asdeps > mypkg.tar.gz` as I installed dependencies for packages I build > manually. However I didn't, so can I retro-actively change the > reason-for-installation of an already installed package without > re-installing it? (No, I couldn't decipher whether the man page > already tells me how to do this :-) > See http://bugs.archlinux.org/task/12950 and http://bugs.archlinux.org/task/13964 especially Dan comment : You can always go and tweak the database file to change the reason if truly necessary. and Nagy comment : If someone suggest me good pacman switches, I will provide a patch for this. ;-) (I think we need a new main operation, -D/--database for example).

1 0

[pacman-dev] [PATCH] xdelta : only handle gz compression specifically
by Xavier 19 May '09

19 May '09

There is apparently no need to handle the re-compression manually when applying a xdelta patch in case of bzip2 or xz. Only gzip needs to be handled specifically for disabling timestamp with the -n option. After this patch, if xdelta is enhanced with xz support (1-line patch), it will be transparent from pacman side. Signed-off-by: Xavier Chantry <shiningxc(a)gmail.com> --- lib/libalpm/sync.c | 11 ++++------- 1 files changed, 4 insertions(+), 7 deletions(-) Note that this patch is completely untested :P diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c index abda147..35e6fca 100644 --- a/lib/libalpm/sync.c +++ b/lib/libalpm/sync.c @@ -622,15 +622,12 @@ static int apply_deltas(pmtrans_t *trans) snprintf(to, len, "%s/%s", cachedir, d->to); /* build the patch command */ - /* compression command */ - char *compress = "cat"; if(endswith(to, ".gz")) { - compress = "gzip -n"; - } else if(endswith(to, ".bz2")) { - compress = "bzip"; + /* special handling for gzip : we disable timestamp with -n option */ + snprintf(command, PATH_MAX, "xdelta3 -d -q -R -c -s %s %s | gzip -n > %s", from, delta, to); + } else { + snprintf(command, PATH_MAX, "xdelta3 -d -q -s %s %s %s", from, delta, to); } - /* -R for disabling external recompression, -c for sending to stdout */ - snprintf(command, PATH_MAX, "xdelta3 -d -q -R -c -s %s %s | %s > %s", from, delta, compress, to); _alpm_log(PM_LOG_DEBUG, "command: %s\n", command); -- 1.6.3.1

1 1