Allow logfile parameter to be set to optional for Application object.

Also fix a minor issue with traffic generator experiment imports
not consistently working by relocating to NLSR subdirectory.

Change-Id: Iad04ee0e3bb64a89133f45721c5d2c1e4c468dbf

examples/traffic_generator.py → examples/nlsr/traffic_generator.py

@@ -39,7 +39,7 @@
 from minindn.apps.nfd import Nfd
 from minindn.apps.nlsr import Nlsr
 from minindn.util import copyExistentFile
-from examples.nlsr.nlsr_common import getParser
+from nlsr_common import getParser
 
 def trafficServer(node, serverConfFile):
     """

minindn/apps/application.py

@@ -22,6 +22,7 @@
 # If not, see <http://www.gnu.org/licenses/>.
 
 from minindn.util import getPopen
+from typing import Union, Optional
 
 class Application(object):
     def __init__(self, node):
@@ -34,11 +35,14 @@ def __init__(self, node):
         self.logDir = '{}/log'.format(self.homeDir)
         self.node.cmd('mkdir -p {}'.format(self.logDir))
 
-    def start(self, command, logfile, envDict=None):
+    def start(self, command: Union[str, list], logfile: Optional[str]=None, envDict: Optional[dict]=None) -> None:
         if self.process is None:
-            self.logfile = open('{}/{}'.format(self.logDir, logfile), 'w')
             if isinstance(command, str):
                 command = command.split()
+            if not logfile:
+                self.process = getPopen(self.node, command, envDict)
+                return
+            self.logfile = open('{}/{}'.format(self.logDir, logfile), 'w')
             self.process = getPopen(self.node, command, envDict,
                                     stdout=self.logfile, stderr=self.logfile)