Here is the git format-patch after passing run_tests and fixing the pep8
warning.



On Sat, Dec 21, 2013 at 12:31 AM, FUJITA Tomonori <
[email protected]> wrote:

> On Fri, 20 Dec 2013 20:54:44 -0800
> Srini Seetharaman <[email protected]> wrote:
>
> > I noticed in the function to_match in ryu/lib/ofctl_v1_3.py that there is
> > an incorrect match_append statement. I've attached a patch to fix that.
> >
> > The patch also includes setting the default priority for the rules in
> > ofproto_v1_3 and ofproto_v1_3_parser. I think this is missing.
>
> Please fix the following pep8 warning:
>
> ryu/ofproto/ofproto_v1_3_parser.py:2362:80: E501 line too long (92 >
> 79 characters)
>                  idle_timeout=0, hard_timeout=0,
>                  priority=ofproto_v1_3.OFP_DEFAULT_PRIORITY,
>
> Please execute 'run_test.sh' script before posting to check pep8.
>
> Also please send a patch with the description and 'Signed-off-by' like:
>
> http://sourceforge.net/mailarchive/message.php?msg_id=31716803
>
> Thanks!
>

Attachment: match_append.patch
Description: Binary data

------------------------------------------------------------------------------
Rapidly troubleshoot problems before they affect your business. Most IT 
organizations don't have a clear picture of how application performance 
affects their revenue. With AppDynamics, you get 100% visibility into your 
Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro!
http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to