On Sat, Jun 09, 2012 at 07:29:47PM +0900, FUJITA Tomonori wrote:
> On Sat, 9 Jun 2012 19:27:12 +0900
> Isaku Yamahata <yamah...@valinux.co.jp> wrote:
> 
> > On Sat, Jun 09, 2012 at 07:06:21PM +0900, FUJITA Tomonori wrote:
> >> RyuApp class is used as a base class for Ryu network application. So
> >> let's pass Ryu application module name instead of class name.
> > 
> > The patch assumes that the .py file contains only single RyuApp class.
> 
> Yes.
> 
> > What if more than one RyuApp sub class are included?
> 
> Needs to support it? I prefer the simplicity for users rather than the
> flexibility with the complication.

Hmm, so you don't like the patch that makes ryu accept both
.py and class name?


> > For example, discovery.py includes main RyuApp subclass and
> > another RyuApp subclass that is used for test/debug the main app.

At least discovery, gre_tunnel (which haven't merged yet) depends on the
current behavior.
-- 
yamahata

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Ryu-devel mailing list
Ryu-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to