summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAurelien Foret <aurelien@archlinux.org>2006-01-05 19:37:54 +0000
committerAurelien Foret <aurelien@archlinux.org>2006-01-05 19:37:54 +0000
commit4819d21c74609de165361695427e89fbd796079d (patch)
treebf7c62893fbdb8b042ed7be4aa616451829288d8
parent9d55476c9c5f94876de2a0c338e1f550e24fe977 (diff)
terminated .lastupdate implementation for HTTP protocol
-rw-r--r--TODO2
-rw-r--r--lib/libalpm/alpm.c7
-rw-r--r--src/pacman/download.c41
-rw-r--r--src/pacman/sync.c2
4 files changed, 31 insertions, 21 deletions
diff --git a/TODO b/TODO
index 7925aa85..8c953f5e 100644
--- a/TODO
+++ b/TODO
@@ -24,7 +24,7 @@ during packages replacement)
- review how things are displayed in the frontend (normal display,
verbose mode, which usage for the library log callback, debug levels, ...)
-- add HTTP support for .lastupdate files
+- add .lastupdate files support for "files" db downloads
ADDITIONAL IDEAS FOR PERFORMANCE IMPROVEMENT
diff --git a/lib/libalpm/alpm.c b/lib/libalpm/alpm.c
index 6c005ba1..833737f4 100644
--- a/lib/libalpm/alpm.c
+++ b/lib/libalpm/alpm.c
@@ -279,11 +279,8 @@ int alpm_db_update(PM_DB *db, char *archive, char *ts)
}
if(ts && strlen(ts) != 0) {
- char lastupdate[15];
- if(db_getlastupdate(db, lastupdate) != -1) {
- if(strcmp(ts, lastupdate) == 0) {
- RET_ERR(PM_ERR_DB_UPTODATE, -1);
- }
+ if(strcmp(ts, db->lastupdate) == 0) {
+ RET_ERR(PM_ERR_DB_UPTODATE, -1);
}
}
diff --git a/src/pacman/download.c b/src/pacman/download.c
index d93ea9fc..9ea92836 100644
--- a/src/pacman/download.c
+++ b/src/pacman/download.c
@@ -356,7 +356,7 @@ int downloadfiles_forreal(list_t *servers, const char *localpath,
fprintf(stderr, "warning: failed to get filesize for %s\n", fn);
}
/* check mtimes */
- if(mtime1 || mtime2) {
+ if(mtime1 && strlen(mtime1)) {
char fmtime[64];
if(!FtpModDate(fn, fmtime, sizeof(fmtime)-1, control)) {
fprintf(stderr, "warning: failed to get mtime for %s\n", fn);
@@ -433,21 +433,30 @@ int downloadfiles_forreal(list_t *servers, const char *localpath,
} else {
snprintf(src, PATH_MAX, "%s://%s%s%s", server->protocol, server->server, server->path, fn);
}
- if(mtime1) {
+ if(mtime1 && strlen(mtime1)) {
+ struct tm tmref;
+ time_t t, tref;
+ int diff;
/* date conversion from YYYYMMDDHHMMSS to "rfc1123-date" */
sscanf(mtime1, "%4d%2d%2d%2d%2d%2d",
&fmtime1.tm_year, &fmtime1.tm_mon, &fmtime1.tm_mday,
&fmtime1.tm_hour, &fmtime1.tm_min, &fmtime1.tm_sec);
fmtime1.tm_year -= 1900;
fmtime1.tm_mon--;
- /* ORE - really compute the week day because some web servers (like lighttpd)
- * need them.
- * Fortunately, Apache does not use it: so our loosy implementation with a
- * hardcoded week day will work in almost all cases. */
- fmtime1.tm_wday = 0;
+ /* compute the week day because some web servers (like lighttpd) need them. */
+ /* we set tmref to "Thu, 01 Jan 1970 00:00:00" */
+ memset(&tmref, 0, sizeof(struct tm));
+ tmref.tm_mday = 1;
+ tref = mktime(&tmref);
+ /* then we compute the difference with mtime1 */
+ t = mktime(&fmtime1);
+ diff = ((t-tref)/3600/24)%7;
+ fmtime1.tm_wday = diff+(diff >= 3 ? -3 : 4);
+
}
fmtime2.tm_year = 0;
- if(!HttpGet(server->server, output, src, &fsz, control, offset, &fmtime1, &fmtime2)) {
+ if(!HttpGet(server->server, output, src, &fsz, control, offset,
+ (mtime1) ? &fmtime1 : NULL, (mtime2) ? &fmtime2 : NULL)) {
if(strstr(FtpLastResponse(control), "304")) {
vprint("mtimes are identical, skipping %s\n", fn);
filedone = -1;
@@ -457,14 +466,18 @@ int downloadfiles_forreal(list_t *servers, const char *localpath,
/* we leave the partially downloaded file in place so it can be resumed later */
}
} else {
+ if(mtime2) {
+ if(fmtime2.tm_year) {
+ /* date conversion from "rfc1123-date" to YYYYMMDDHHMMSS */
+ sprintf(mtime2, "%4d%02d%02d%02d%02d%02d",
+ fmtime2.tm_year+1900, fmtime2.tm_mon+1, fmtime2.tm_mday,
+ fmtime2.tm_hour, fmtime2.tm_min, fmtime2.tm_sec);
+ } else {
+ fprintf(stderr, "warning: failed to get mtime for %s\n", fn);
+ }
+ }
filedone = 1;
}
- if(mtime2 && fmtime2.tm_year) {
- /* date conversion from "rfc1123-date" to YYYYMMDDHHMMSS */
- sprintf(mtime2, "%4d%02d%02d%02d%02d%02d",
- fmtime2.tm_year+1900, fmtime2.tm_mon+1, fmtime2.tm_mday,
- fmtime2.tm_hour, fmtime2.tm_min, fmtime2.tm_sec);
- }
} else if(!strcmp(server->protocol, "file")) {
char src[PATH_MAX];
snprintf(src, PATH_MAX, "%s%s", server->path, fn);
diff --git a/src/pacman/sync.c b/src/pacman/sync.c
index b8c44f40..41200e8e 100644
--- a/src/pacman/sync.c
+++ b/src/pacman/sync.c
@@ -166,7 +166,7 @@ static int sync_synctree(list_t *syncs)
/* get the lastupdate time */
lastupdate = alpm_db_getinfo(sync->db, PM_DB_LASTUPDATE);
- if(lastupdate == NULL) {
+ if(strlen(lastupdate) == 0) {
vprint("failed to get lastupdate time for %s (no big deal)\n", sync->treename);
}