Send patches - preferably formatted by git format-patch - to patches at archlinux32 dot org.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2007-03-12 03:02:57 +0000
committerDan McGee <dan@archlinux.org>2007-03-12 03:02:57 +0000
commitba1806f5aca616390355a4258f7edb43ab8b0b66 (patch)
tree9731fe669fd233406a3f118e7e5a3c69a427e13a
parent9ba23c8248eac4406b374f25feafcad873db89be (diff)
* Removed ${CFLAGS} from Makefile.am(s) as it was causing all CFLAGS to be
duplicated. * Updated the util Makefile.am to link with the proper libalpm.la. * Fixed bitmasking issues in be_files.c and db.h. * Rankmirrors updates from James Rosten (with some cleaning up of my own). KeyboardInterrupts are now handled gracefully.
-rw-r--r--lib/libalpm/Makefile.am2
-rw-r--r--lib/libalpm/be_files.c7
-rw-r--r--lib/libalpm/db.h3
-rw-r--r--scripts/rankmirrors98
-rw-r--r--src/pacman/Makefile.am2
-rw-r--r--src/util/Makefile.am4
6 files changed, 70 insertions, 46 deletions
diff --git a/lib/libalpm/Makefile.am b/lib/libalpm/Makefile.am
index f96fdd0e..0381d9d8 100644
--- a/lib/libalpm/Makefile.am
+++ b/lib/libalpm/Makefile.am
@@ -8,7 +8,7 @@ include_HEADERS = alpm_list.h alpm.h
localedir = $(datadir)/locale
DEFS = -DLOCALEDIR=\"$(localedir)\" @DEFS@
-AM_CFLAGS = -fvisibility=hidden -pedantic -D_GNU_SOURCE $(CFLAGS)
+AM_CFLAGS = -fvisibility=hidden -pedantic -D_GNU_SOURCE
EXTRA_DIST = Doxyfile
diff --git a/lib/libalpm/be_files.c b/lib/libalpm/be_files.c
index 6e2f4c4f..30f6f905 100644
--- a/lib/libalpm/be_files.c
+++ b/lib/libalpm/be_files.c
@@ -227,7 +227,12 @@ int _alpm_db_read(pmdb_t *db, pmpkg_t *info, pmdbinfrq_t inforeq)
return(-1);
}
- if(info->infolevel & inforeq) {
+ /* bitmask logic here:
+ * infolevel: 00001111
+ * inforeq: 00010100
+ * & result: 00000100
+ * == to inforeq? nope, we need to load more info. */
+ if((info->infolevel & inforeq) == inforeq) {
/* already loaded this info, do nothing */
return(0);
}
diff --git a/lib/libalpm/db.h b/lib/libalpm/db.h
index 4ef1e591..26f1fdf4 100644
--- a/lib/libalpm/db.h
+++ b/lib/libalpm/db.h
@@ -33,7 +33,8 @@ typedef enum _pmdbinfrq_t {
INFRQ_DEPENDS = 0x04,
INFRQ_FILES = 0x08,
INFRQ_SCRIPTLET = 0x10,
- INFRQ_ALL = 0xFF
+ /* ALL should be sum of all above */
+ INFRQ_ALL = 0x1F
} pmdbinfrq_t;
/* Database */
diff --git a/scripts/rankmirrors b/scripts/rankmirrors
index 64293440..b2cfd18c 100644
--- a/scripts/rankmirrors
+++ b/scripts/rankmirrors
@@ -23,9 +23,6 @@
import os, sys, datetime, time, socket, urllib2
from optparse import OptionParser
-# TODO: handle KeyboardInterrupt better, print list of already timed
-# servers and then exit. Easier if program is function-ized.
-
def createOptParser():
usage = "usage: %prog [options] MIRRORFILE | URL"
description = "Ranks pacman mirrors by their connection and opening " \
@@ -33,27 +30,28 @@ def createOptParser():
"can also rank one mirror if the URL is provided."
parser = OptionParser(usage = usage, description = description)
parser.add_option("-n", type = "int", dest = "num", default = 0,
- help = "number of servers to output, 0 for all")
+ help = "number of servers to output, 0 for all")
parser.add_option("-t", "--times", action = "store_true",
- dest = "times", default = False,
- help = "only output mirrors and their response times")
+ dest = "times", default = False,
+ help = "only output mirrors and their response times")
parser.add_option("-u", "--url", action = "store_true", dest = "url",
- default=False, help="test a specific url")
+ default = False, help = "test a specific url")
parser.add_option("-v", "--verbose", action = "store_true",
- dest = "verbose", default = False, help ="be verbose in output")
+ dest = "verbose", default = False,
+ help = "be verbose in ouptut")
return parser
def timeCmd(cmd):
- before = time.time();
+ before = time.time()
try:
- cmd();
+ cmd()
except KeyboardInterrupt, ki:
raise ki
except socket.timeout, ioe:
return 'timeout'
except Exception, e:
return 'unreachable'
- return time.time() - before;
+ return time.time() - before
def talkToServer(serverUrl):
opener = urllib2.build_opener()
@@ -63,10 +61,32 @@ def getFuncToTime(serverUrl):
return lambda : talkToServer(serverUrl)
def cmpPairBySecond(p1, p2):
- if p1[1] == p2[1]: return 0
- if p1[1] < p2[1]: return -1
+ if p1[1] == p2[1]:
+ return 0
+ if p1[1] < p2[1]:
+ return -1
return 1
+def printResults(servers, time, verbose, num):
+ items = servers.items()
+ items.sort(cmpPairBySecond)
+ itemsLen = len(items)
+ numToShow = num
+ if numToShow > itemsLen or numToShow == 0:
+ numToShow = itemsLen
+ if itemsLen > 0:
+ if time:
+ print
+ print ' Servers sorted by time (seconds):'
+ for i in items[0:numToShow]:
+ if i[1] == 'timeout' or i[1] == 'unreachable':
+ print i[0], ':', i[1]
+ else:
+ print i[0], ':', "%.2f" % i[1]
+ else:
+ for i in items[0:numToShow]:
+ print 'Server =', i[0]
+
if __name__ == "__main__":
parser = createOptParser()
(options, args) = parser.parse_args()
@@ -81,12 +101,18 @@ if __name__ == "__main__":
if options.url:
if options.verbose:
print 'Testing', args[0] + '...'
- serverToTime = timeCmd(getFuncToTime(args[0]))
- print args[0], ':', serverToTime
+ try:
+ serverToTime = timeCmd(getFuncToTime(args[0]))
+ except KeyboardInterrupt, ki:
+ sys.exit(1)
+ if serverToTime == 'timeout' or serverToTime == 'unreachable':
+ print args[0], ':', serverToTime
+ else:
+ print args[0], ':', "%.2f" % serverToTime
sys.exit(0)
if not os.path.isfile(args[0]):
- print 'file', args[0], 'does not exist.'
+ print >>sys.stderr, 'rankmirrors: file', args[0], 'does not exist.'
sys.exit(1)
fl = open(args[0], 'r')
@@ -104,36 +130,28 @@ if __name__ == "__main__":
continue
serverUrl = splitted[1].strip()
- if serverUrl[-1] == '\n': serverUrl = serverUrl[0:-1]
+ if serverUrl[-1] == '\n':
+ serverUrl = serverUrl[0:-1]
if options.verbose and options.times:
- print serverUrl,'...',
+ print serverUrl, '...',
elif options.verbose:
- print '#',serverUrl,'...',
+ print '#', serverUrl, '...',
elif options.times:
print ' * ',
sys.stdout.flush()
- serverToTime[serverUrl] = timeCmd(getFuncToTime(serverUrl))
- if options.verbose:
- try:
- print "%.2f" % serverToTime[serverUrl]
- except:
- print serverToTime[serverUrl]
-
- items = serverToTime.items()
- items.sort(cmpPairBySecond)
- numToShow = int(options.num)
- if numToShow == 0: numToShow = len(items)
- if len(items) > 0:
- if options.times:
- print
- print ' Servers sorted by time (seconds):'
- for i in items[0:numToShow]:
+ try:
+ serverToTime[serverUrl] = timeCmd(getFuncToTime(serverUrl))
+ if options.verbose:
try:
- print "%.2f" % i[1], ':', i[0]
+ print "%.2f" % serverToTime[serverUrl]
except:
- print i[1], ':', i[0]
- else:
- for i in items[0:numToShow]:
- print 'Server =', i[0]
+ print serverToTime[serverUrl]
+ except:
+ print
+ printResults(serverToTime, options.times, options.verbose,
+ options.num)
+ sys.exit(0)
+
+ printResults(serverToTime, options.times, options.verbose, options.num)
# vim: set ts=4 sw=4 et:
diff --git a/src/pacman/Makefile.am b/src/pacman/Makefile.am
index ee03372d..82ec4d6b 100644
--- a/src/pacman/Makefile.am
+++ b/src/pacman/Makefile.am
@@ -10,7 +10,7 @@ localedir = $(datadir)/locale
DEFS = -DLOCALEDIR=\"$(localedir)\" @DEFS@
INCLUDES = -I$(top_srcdir)/lib/libalpm
-AM_CFLAGS = -D_GNU_SOURCE $(CFLAGS)
+AM_CFLAGS = -D_GNU_SOURCE
pacman_SOURCES = \
add.h add.c \
diff --git a/src/util/Makefile.am b/src/util/Makefile.am
index 407eb055..e79e726d 100644
--- a/src/util/Makefile.am
+++ b/src/util/Makefile.am
@@ -3,7 +3,7 @@ bin_PROGRAMS = vercmp testpkg
INCLUDES = -I$(top_srcdir)/lib/libalpm
vercmp_SOURCES = vercmp.c
-vercmp_LDADD = $(top_builddir)/lib/libalpm/libalpm.la
+vercmp_LDADD = $(top_builddir)/lib/libalpm/.libs/libalpm.la
testpkg_SOURCES = testpkg.c
-testpkg_LDADD = $(top_builddir)/lib/libalpm/libalpm.la
+testpkg_LDADD = $(top_builddir)/lib/libalpm/.libs/libalpm.la