Minor cleanup of mn/net and fixes for CLI.

All commands should work now.

bin/mn

@@ -113,7 +113,6 @@ class MininetRunner( object ):
             index = sys.argv.index( '--custom' )
             if len( sys.argv ) > index + 1:
                 custom = sys.argv[ index + 1 ]
-                print "custom = %s" % custom
                 self.parseCustomFile( custom )
             else:
                 raise Exception( 'Custom file name not found' )

mininet/cli.py

@@ -47,8 +47,8 @@ class CLI( Cmd ):
 
     def __init__( self, mininet ):
         self.mn = mininet
+        self.nodelist = self.mn.hosts + self.mn.switches + self.mn.controllers
         self.nodemap = {} # map names to Node objects
-        self.nodelist = self.mn.switches + self.mn.hosts + self.mn.controllers
         for node in self.nodelist:
             self.nodemap[ node.name ] = node
         Cmd.__init__( self )
@@ -91,7 +91,7 @@ def do_net( self, args ):
         for switch in self.mn.switches:
             info( '%s <->', switch.name )
             for intf in switch.intfs:
-                node = switch.connection[ intf ]
+                node, name = switch.connection[ intf ]
                 info( ' %s' % node.name )
             info( '\n' )
 
@@ -118,12 +118,12 @@ def do_iperfudp( self, args ):
 
     def do_intfs( self, args ):
         "List interfaces."
-        for node in self.mn.nodes.values():
+        for node in self.nodelist:
             info( '%s: %s\n' % ( node.name, ' '.join( node.intfs ) ) )
 
     def do_dump( self, args ):
         "Dump node info."
-        for node in self.mn.nodes.values():
+        for node in self.nodelist:
             info( '%s\n' % node )
 
     def do_exit( self, args ):

mininet/net.py

@@ -384,7 +384,7 @@ def stop( self ):
         info( '\n' )
         info( '*** Stopping %i switches\n' % len( self.switches ) )
         for switch in self.switches:
-            info( '%s ' % switch.name )
+            info( switch.name )
             switch.stop()
         info( '\n' )
         info( '*** Stopping %i controllers\n' % len( self.controllers ) )