sage --package update-latest: Accept package classes :standard:, :opt…

…ional: etc., restrict to normal Python packages

build/sage_bootstrap/app.py

@@ -140,19 +140,22 @@ def update_latest(self, package_name):
         else:
             pypi.update(Package(package_name))
 
-    def update_latest_all(self):
-        log.debug('Attempting to update all packages')
+    def update_latest_cls(self, package_name_or_class):
         exclude = [
-            'atlas', 'flint', 'bzip2', 'ecm', 'freetype', 'gap', 'glpk', 'graphs',
-            'iconv', 'patch', 'r', 'configure', 'bliss', 'readline', 'decorator',
-            'igraph', 'rw', 'planarity', 'gambit', 
+            'cypari'   # Name conflict
         ]
-        pc = PackageClass(':standard:')
-        for package_name in pc.names:
+        # Restrict to normal Python packages
+        pc = PackageClass(package_name_or_class, has_files=['checksums.ini', 'install-requires.txt'])
+        if not pc.names:
+            log.warn('nothing to do (does not name a normal Python package)')
+        for package_name in sorted(pc.names):
             if package_name in exclude:
                 log.debug('skipping %s because of pypi name collision', package_name)
                 continue
-            self.update_latest(package_name)
+            try:
+                self.update_latest(package_name)
+            except PyPiError as e:
+                log.warn('updating %s failed: %s', package_name, e)
 
     def download(self, package_name, allow_upstream=False):
         """

build/sage_bootstrap/cmdline.py

@@ -334,10 +334,7 @@ def run():
     elif args.subcommand == 'update':
         app.update(args.package_name, args.new_version, url=args.url)
     elif args.subcommand == 'update-latest':
-        if args.package_name == ':all:':
-            app.update_latest_all()
-        else:
-            app.update_latest(args.package_name)
+        app.update_latest_cls(args.package_name)
     elif args.subcommand == 'download':
         app.download_cls(args.package_name,
                          allow_upstream=args.allow_upstream,