Send patches - preferably formatted by git format-patch - to patches at archlinux32 dot org.
summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2007-06-04 12:51:23 -0400
committerDan McGee <dan@archlinux.org>2007-06-04 23:10:49 -0400
commit358cc5804a2df873180e6d9ef2420ab3247f8437 (patch)
tree9254dd518c1873aebe78f548d311202a6f1fe2aa /src
parent6949ab97613de3622a4c392a3d33080164eec794 (diff)
Rip alpm_parse_config out of libalpm
Switch over to the new frontend parseconfig. * Fix a few issues in parseconfig * Remove unused callback upon database registration * Remove conf file related errors from error.c/alpm.h Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'src')
-rw-r--r--src/pacman/callback.c3
-rw-r--r--src/pacman/pacman.c69
-rw-r--r--src/pacman/query.c2
-rw-r--r--src/pacman/sync.c2
-rw-r--r--src/pacman/util.c4
5 files changed, 41 insertions, 39 deletions
diff --git a/src/pacman/callback.c b/src/pacman/callback.c
index e13a7e7d..baefacc0 100644
--- a/src/pacman/callback.c
+++ b/src/pacman/callback.c
@@ -91,7 +91,6 @@ static float get_update_timediff(int first_call)
/* refactored from cb_trans_progress */
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;
static unsigned int lasthash = 0, mouth = 0;
@@ -109,7 +108,7 @@ static void fill_progress(const int percent, const int proglen)
printf(" [");
for(i = hashlen; i > 1; --i) {
/* if special progress bar enabled */
- if(chomp) {
+ if(config->chomp) {
if(i > hashlen - hash) {
printf("-");
} else if(i == hashlen - hash) {
diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c
index 19ffc759..fdd399fd 100644
--- a/src/pacman/pacman.c
+++ b/src/pacman/pacman.c
@@ -495,20 +495,24 @@ static int _parseconfig(const char *file, const char *givensection)
}
section = strdup(ptr);
section[strlen(section)-1] = '\0';
- printf(_("config: new section '%s'"), section);
+ printf(_("config: new section '%s'\n"), section);
if(!strlen(section)) {
- printf("PM_ERR_CONF_BAD_SECTION");
+ printf("PM_ERR_CONF_BAD_SECTION\n");
return(1);
}
+ /* a section/database named local is not allowed */
if(!strcmp(section, "local")) {
- printf("PM_ERR_CONF_LOCAL");
+ printf("PM_ERR_CONF_LOCAL\n");
return(1);
}
+ /* if we are not looking at the options section, register a db */
+ if(strcmp(section, "options") != 0) {
+ alpm_db_register(section);
+ }
} else {
/* directive */
char *key;
const char *upperkey;
-
/* strsep modifies the 'line' string: 'key \0 ptr' */
key = line;
ptr = line;
@@ -517,12 +521,12 @@ static int _parseconfig(const char *file, const char *givensection)
strtrim(ptr);
if(key == NULL) {
- printf("PM_ERR_CONF_BAD_SYNTAX");
+ printf("PM_ERR_CONF_BAD_SYNTAX\n");
return(1);
}
upperkey = strtoupper(strdup(key));
- if(!strlen(section) && strcmp(key, "INCLUDE")) {
- printf("PM_ERR_CONF_DIRECTIVE_OUTSIDE_SECTION");
+ if(section == NULL && (strcmp(key, "Include") == 0 || strcmp(upperkey, "INCLUDE") == 0)) {
+ printf("PM_ERR_CONF_DIRECTIVE_OUTSIDE_SECTION\n");
return(1);
}
if(ptr == NULL) {
@@ -530,27 +534,27 @@ static int _parseconfig(const char *file, const char *givensection)
/* TODO shouldn't we check if these are in the [options] section? */
if(strcmp(key, "NoPassiveFTP") == 0 || strcmp(upperkey, "NOPASSIVEFTP") == 0) {
alpm_option_set_nopassiveftp(1);
- printf(_("config: nopassiveftp"));
+ printf(_("config: nopassiveftp\n"));
} else if(strcmp(key, "UseSyslog") == 0 || strcmp(upperkey, "USESYSLOG") == 0) {
alpm_option_set_usesyslog(1);
- printf(_("config: usesyslog"));
+ printf(_("config: usesyslog\n"));
} else if(strcmp(key, "ILoveCandy") == 0 || strcmp(upperkey, "ILOVECANDY") == 0) {
config->chomp = 1;
- printf(_("config: chomp"));
+ printf(_("config: chomp\n"));
} else if(strcmp(key, "UseColor") == 0 || strcmp(upperkey, "USECOLOR") == 0) {
config->usecolor = 1;
- printf(_("config: usecolor"));
+ printf(_("config: usecolor\n"));
} else if(strcmp(key, "ShowSize") == 0 || strcmp(upperkey, "SHOWSIZE") == 0) {
config->showsize= 1;
- printf(_("config: showsize"));
+ printf(_("config: showsize\n"));
} else {
- printf("PM_ERR_CONF_BAD_SYNTAX");
+ printf("PM_ERR_CONF_BAD_SYNTAX\n");
return(1);
}
} else {
/* directives with settings */
if(strcmp(key, "Include") == 0 || strcmp(upperkey, "INCLUDE") == 0) {
- printf(_("config: including %s"), ptr);
+ printf(_("config: including %s\n"), ptr);
_parseconfig(ptr, section);
} else if(strcmp(section, "options") == 0) {
if(strcmp(key, "NoUpgrade") == 0 || strcmp(upperkey, "NOUPGRADE") == 0) {
@@ -561,12 +565,12 @@ static int _parseconfig(const char *file, const char *givensection)
while((q = strchr(p, ' '))) {
*q = '\0';
alpm_option_add_noupgrade(p);
- printf(_("config: noupgrade: %s"), p);
+ printf(_("config: noupgrade: %s\n"), p);
p = q;
p++;
}
alpm_option_add_noupgrade(p);
- printf(_("config: noupgrade: %s"), p);
+ printf(_("config: noupgrade: %s\n"), p);
} else if(strcmp(key, "NoExtract") == 0 || strcmp(upperkey, "NOEXTRACT") == 0) {
char *p = ptr;
char *q;
@@ -574,12 +578,12 @@ static int _parseconfig(const char *file, const char *givensection)
while((q = strchr(p, ' '))) {
*q = '\0';
alpm_option_add_noextract(p);
- printf(_("config: noextract: %s"), p);
+ printf(_("config: noextract: %s\n"), p);
p = q;
p++;
}
alpm_option_add_noextract(p);
- printf(_("config: noextract: %s"), p);
+ printf(_("config: noextract: %s\n"), p);
} else if(strcmp(key, "IgnorePkg") == 0 || strcmp(upperkey, "IGNOREPKG") == 0) {
char *p = ptr;
char *q;
@@ -592,7 +596,7 @@ static int _parseconfig(const char *file, const char *givensection)
p++;
}
alpm_option_add_ignorepkg(p);
- printf(_("config: ignorepkg: %s"), p);
+ printf(_("config: ignorepkg: %s\n"), p);
} else if(strcmp(key, "HoldPkg") == 0 || strcmp(upperkey, "HOLDPKG") == 0) {
char *p = ptr;
char *q;
@@ -600,37 +604,37 @@ static int _parseconfig(const char *file, const char *givensection)
while((q = strchr(p, ' '))) {
*q = '\0';
alpm_option_add_holdpkg(p);
- printf(_("config: holdpkg: %s"), p);
+ printf(_("config: holdpkg: %s\n"), p);
p = q;
p++;
}
alpm_option_add_holdpkg(p);
- printf(_("config: holdpkg: %s"), p);
+ printf(_("config: holdpkg: %s\n"), p);
} else if(strcmp(key, "DBPath") == 0 || strcmp(upperkey, "DBPATH") == 0) {
alpm_option_set_dbpath(ptr);
- printf(_("config: dbpath: %s"), ptr);
+ printf(_("config: dbpath: %s\n"), ptr);
} else if(strcmp(key, "CacheDir") == 0 || strcmp(upperkey, "CACHEDIR") == 0) {
alpm_option_set_cachedir(ptr);
- printf(_("config: cachedir: %s"), ptr);
+ printf(_("config: cachedir: %s\n"), ptr);
} else if(strcmp(key, "RootDir") == 0 || strcmp(upperkey, "ROOTDIR") == 0) {
alpm_option_set_root(ptr);
- printf(_("config: rootdir: %s"), ptr);
+ printf(_("config: rootdir: %s\n"), ptr);
} else if (strcmp(key, "LogFile") == 0 || strcmp(upperkey, "LOGFILE") == 0) {
alpm_option_set_logfile(ptr);
- printf(_("config: logfile: %s"), ptr);
+ printf(_("config: logfile: %s\n"), ptr);
} else if (strcmp(key, "LockFile") == 0 || strcmp(upperkey, "LOCKFILE") == 0) {
alpm_option_set_lockfile(ptr);
- printf(_("config: lockfile: %s"), ptr);
+ printf(_("config: lockfile: %s\n"), ptr);
} else if (strcmp(key, "XferCommand") == 0 || strcmp(upperkey, "XFERCOMMAND") == 0) {
alpm_option_set_xfercommand(ptr);
- printf(_("config: xfercommand: %s"), ptr);
+ printf(_("config: xfercommand: %s\n"), ptr);
} else if (strcmp(key, "UpgradeDelay") == 0 || strcmp(upperkey, "UPGRADEDELAY") == 0) {
/* The config value is in days, we use seconds */
time_t ud = atol(ptr) * 60 * 60 *24;
alpm_option_set_upgradedelay(ud);
- printf(_("config: upgradedelay: %d"), (int)ud);
+ printf(_("config: upgradedelay: %d\n"), (int)ud);
} else {
- printf("PM_ERR_CONF_BAD_SYNTAX");
+ printf("PM_ERR_CONF_BAD_SYNTAX\n");
return(1);
}
} else {
@@ -640,16 +644,15 @@ static int _parseconfig(const char *file, const char *givensection)
if(alpm_db_setserver(db, server) != 0) {
/* pm_errno is set by alpm_db_setserver */
- return(-1);
+ return(1);
}
free(server);
} else {
- printf("PM_ERR_CONF_BAD_SYNTAX");
+ printf("PM_ERR_CONF_BAD_SYNTAX\n");
return(1);
}
}
- line[0] = '\0';
}
}
}
@@ -751,7 +754,7 @@ int main(int argc, char *argv[])
config->configfile = strdup(CONFFILE);
}
- if(alpm_parse_config(config->configfile, NULL, "") != 0) {
+ if(parseconfig(config->configfile) != 0) {
fprintf(stderr, _("error: failed to parse config (%s)\n"),
alpm_strerror(pm_errno));
cleanup(1);
diff --git a/src/pacman/query.c b/src/pacman/query.c
index 7df16b17..7a19ae35 100644
--- a/src/pacman/query.c
+++ b/src/pacman/query.c
@@ -150,7 +150,7 @@ static int query_search(alpm_list_t *targets)
printf("local/%s %s", alpm_pkg_get_name(pkg), alpm_pkg_get_version(pkg));
/* print the package size with the output if ShowSize option set */
- if(alpm_option_get_showsize()) {
+ if(config->showsize) {
/* Convert byte size to MB */
double mbsize = alpm_pkg_get_size(pkg) / (1024.0 * 1024.0);
diff --git a/src/pacman/sync.c b/src/pacman/sync.c
index 9f345c69..92a1607a 100644
--- a/src/pacman/sync.c
+++ b/src/pacman/sync.c
@@ -250,7 +250,7 @@ static int sync_search(alpm_list_t *syncs, alpm_list_t *targets)
alpm_pkg_get_version(pkg));
/* print the package size with the output if ShowSize option set */
- if(alpm_option_get_showsize()) {
+ if(config->showsize) {
/* Convert byte size to MB */
double mbsize = alpm_pkg_get_size(pkg) / (1024.0 * 1024.0);
diff --git a/src/pacman/util.c b/src/pacman/util.c
index 406145e3..e88001ac 100644
--- a/src/pacman/util.c
+++ b/src/pacman/util.c
@@ -210,7 +210,7 @@ char *strtrim(char *str)
{
char *pch = str;
- if(*str == '\0') {
+ if(str == NULL || *str == '\0') {
/* string is empty, so we're done. */
return(str);
}
@@ -350,7 +350,7 @@ void display_targets(alpm_list_t *syncpkgs)
isize += alpm_pkg_get_isize(pkg);
/* print the package size with the output if ShowSize option set */
- if(alpm_option_get_showsize()) {
+ if(config->showsize) {
/* Convert byte size to MB */
mbdispsize = dispsize / (1024.0 * 1024.0);