From 4819d21c74609de165361695427e89fbd796079d Mon Sep 17 00:00:00 2001
From: Aurelien Foret <aurelien@archlinux.org>
Date: Thu, 5 Jan 2006 19:37:54 +0000
Subject: terminated .lastupdate implementation for HTTP protocol

---
 TODO                  |  2 +-
 lib/libalpm/alpm.c    |  7 ++-----
 src/pacman/download.c | 41 +++++++++++++++++++++++++++--------------
 src/pacman/sync.c     |  2 +-
 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);
 		}
 
-- 
cgit v1.2.3