* fix: use subprocess.PIPE, tested, works for faik's test script

* add an --xml option to info command, to do this add a writexml
  command to xmlfilepiks (a bit incomplete at the moment)

pisi/cli/commands.py

@@ -667,6 +667,8 @@ Usage: info <package1> <package2> ... <packagen>
                                help=_("show only paths."))
         self.parser.add_option("-s", "--short", action="store_true",
                                default=False, help=_("do not show details"))
+        self.parser.add_option("", "--xml", action="store_true",
+                               default=False, help=_("output in xml format"))
 
     def run(self):
 
@@ -676,13 +678,30 @@ Usage: info <package1> <package2> ... <packagen>
             self.help()
             return
             
+        index = pisi.index.Index()
+        index.distribution = None
+        
         for arg in self.args:
             if ctx.componentdb.has_component(arg):
                 component = ctx.componentdb.get_union_comp(arg)
-                #if self.options.long:
+                if self.options.xml:
+                    index.add_component(component)
+                else:
+                    if not self.options.short:
                         ctx.ui.info(unicode(component))
+                    else:
+                        ctx.ui.info("%s - %s" % (component.name, component.summary)) 
             else: # then assume it was a package 
+                if self.options.xml:
+                    index.packages.append(pisi.api.info(arg)[0].package)
+                else:
                     self.info_package(arg)
+        if self.options.xml:
+            errs = []
+            index.newDocument()
+            index.encode(index.rootNode(), errs)
+            index.writexmlfile(sys.stdout)
+            sys.stdout.write('\n')
         self.finalize()
 
 
@@ -694,21 +713,19 @@ Usage: info <package1> <package2> ... <packagen>
         else:
             if ctx.installdb.is_installed(arg):
                 metadata, files = pisi.api.info_name(arg, True)
-                if ctx.get_option('short'):
+                if self.options.short:
                     ctx.ui.info(_('[inst] '), noln=True)
                 else:
                     ctx.ui.info(_('Installed package:'))
                 self.print_pkginfo(metadata, files,pisi.itembyrepodb.installed)
-                print
                 
             if ctx.packagedb.has_package(arg):
                 metadata, files = pisi.api.info_name(arg, False)
-                if ctx.get_option('short'):
+                if self.options.short:
                     ctx.ui.info(_('[repo] '), noln=True)
                 else:
                     ctx.ui.info(_('Package found in repository:'))
                 self.print_pkginfo(metadata, files, pisi.itembyrepodb.repos)
-                print
 
     def print_pkginfo(self, metadata, files, repo = None):
         import os.path
@@ -733,7 +750,7 @@ Usage: info <package1> <package2> ... <packagen>
                         print fileinfo.path
             else:
                 ctx.ui.warning(_('File information not available'))
-        if ctx.get_option('long'):
+        if not self.options.short:
             print
 
 
@@ -867,6 +884,7 @@ Usage: list-installed
                 ctx.ui.info('%15s - %s' % (package.name, unicode(package.summary)))
         self.finalize()
 
+        
 class RebuildDb(Command):
     """Rebuild Databases
 
@@ -1086,6 +1104,7 @@ all repositories.
                 p = p + ' ' * max(0, 15 - lenp)
                 ctx.ui.info('%s - %s ' % (p, unicode(package.summary)))
 
+                
 class ListComponents(Command):
     """List available components
 

pisi/pxml/xmlfilepiks.py

@@ -55,6 +55,15 @@ class XmlFile(object):
         """returns root document element"""
         return self.doc
         
+    def readxmlfile(self, file):
+        raise Exception("not implemented")
+        try:
+            self.doc = iks.parse(file)
+            return self.doc
+        except Exception, e:
+            raise Error(_("File '%s' has invalid XML") % (localpath) )
+
+
     def readxml(self, uri, tmpDir='/tmp', sha1sum=False, 
                 compress=None, sign=None, copylocal = False):
         uri = File.make_uri(uri)
@@ -73,3 +82,6 @@ class XmlFile(object):
         f = File(uri, File.write, sha1sum=sha1sum, compress=compress, sign=sign)
         f.write(self.doc.toPrettyString())
         f.close()
+
+    def writexmlfile(self, f):
+        f.write(self.doc.toPrettyString())

pisi/util.py

@@ -155,7 +155,7 @@ def run_batch(cmd, realtime = False):
     if ctx.stderr:
         stderr = ctx.stderr
     else:
-        stderr = subprocess.STDOUT
+        stderr = subprocess.PIPE
 
     out = err = ""
     p = subprocess.Popen(cmd, shell=True, stdout=stdout, stderr=stderr)