From 33f251853155fd1e04d03fbaffb3849ab8e953dc Mon Sep 17 00:00:00 2001 From: Dan McGee Date: Wed, 21 Sep 2011 10:14:51 -0500 Subject: Move default siglevel value from backend to frontend This takes the libraries hidden default out of the equation: hidden in the sense that we can't even find out what it is until we create a handle. This is a chicken-and-egg problem where we have probably already parsed the config, so it is hard to get the bitmask value right. Move it to the frontend so the caller can do whatever the heck they want. This also exposes a shortcoming where the frontend doesn't know if the library even supports signatures, so we should probably add a alpm_capabilities() method which exposes things like HAS_DOWNLOADER, HAS_SIGNATURES, etc. Signed-off-by: Dan McGee --- lib/libalpm/handle.c | 7 +------ src/pacman/conf.c | 10 +++------- 2 files changed, 4 insertions(+), 13 deletions(-) diff --git a/lib/libalpm/handle.c b/lib/libalpm/handle.c index dcd81cea..7402be50 100644 --- a/lib/libalpm/handle.c +++ b/lib/libalpm/handle.c @@ -39,17 +39,12 @@ #include "trans.h" #include "alpm.h" -alpm_handle_t *_alpm_handle_new() +alpm_handle_t *_alpm_handle_new(void) { alpm_handle_t *handle; CALLOC(handle, 1, sizeof(alpm_handle_t), return NULL); -#ifdef HAVE_LIBGPGME - handle->siglevel = ALPM_SIG_PACKAGE | ALPM_SIG_PACKAGE_OPTIONAL | - ALPM_SIG_DATABASE | ALPM_SIG_DATABASE_OPTIONAL; -#endif - return handle; } diff --git a/src/pacman/conf.c b/src/pacman/conf.c index e8b34f7d..cc1fb724 100644 --- a/src/pacman/conf.c +++ b/src/pacman/conf.c @@ -54,7 +54,8 @@ config_t *config_new(void) newconfig->op = PM_OP_MAIN; newconfig->logmask = ALPM_LOG_ERROR | ALPM_LOG_WARNING; newconfig->configfile = strdup(CONFFILE); - newconfig->siglevel = ALPM_SIG_USE_DEFAULT; + newconfig->siglevel = ALPM_SIG_PACKAGE | ALPM_SIG_PACKAGE_OPTIONAL | + ALPM_SIG_DATABASE | ALPM_SIG_DATABASE_OPTIONAL; return newconfig; } @@ -567,12 +568,7 @@ static int setup_libalpm(void) alpm_option_set_cachedirs(handle, config->cachedirs); } - if(config->siglevel != ALPM_SIG_USE_DEFAULT) { - alpm_option_set_default_siglevel(handle, config->siglevel); - } - /* retrieve the set or default siglevel from the backend at this point; - * this way all future DB settings base their setting off this value */ - config->siglevel = alpm_option_get_default_siglevel(handle); + alpm_option_set_default_siglevel(handle, config->siglevel); if(config->xfercommand) { alpm_option_set_fetchcb(handle, download_with_xfercommand); -- cgit v1.2.3-70-g09d2