launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Colin Watson has proposed merging lp:~cjwatson/launchpad/remove-check-geoip-db into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/remove-check-geoip-db/+merge/117590

utilities/check-geoip-db used to be called from the top-level Makefile, but r6934 changed things around so that the appservers check for it on the fly instead (see bug 262102; plus it's in launchpad-dependencies).  Since nothing calls this any more, we should drop it.
-- 
https://code.launchpad.net/~cjwatson/launchpad/remove-check-geoip-db/+merge/117590
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~cjwatson/launchpad/remove-check-geoip-db into lp:launchpad.

=== removed file 'utilities/check-geoip-db'
--- utilities/check-geoip-db	2009-06-24 20:22:29 +0000
+++ utilities/check-geoip-db	1970-01-01 00:00:00 +0000
@@ -1,16 +0,0 @@
-#! /bin/bash
-#
-# Copyright 2009 Canonical Ltd.  This software is licensed under the
-# GNU Affero General Public License version 3 (see the file LICENSE).
-#
-# Check that we have a GeoIP city database.
-
-if [ ! -e /usr/share/GeoIP/GeoLiteCity.dat -a \
-     ! -e /usr/share/GeoIP/GeoIPCity.dat ]
-  then 
-    echo "*********************************************************"
-    echo "No GeoIP db found.  Please install launchpad-dependencies"
-    echo "*********************************************************"
-    exit 1
-fi
-exit 0