On Tue, Oct 27, 2015 at 08:48:41AM +0000, Remy Horton wrote: > > > On 23/10/2015 15:23, Tahhan, Maryam wrote: > > >> # binary name > >>-APP = l2fwd > >>+APP = l2fwd-keepalive > >Hi Remy, > >Looks great overall, just a couple of comments > >For backward compatibility the rename could be a nightmare? What do you > >think? > > True. Undecided whether to change the Makefile(s) so that both keepalive and > non-keepalive l2fwd are built, or to simply clone all of l2fwd.. >
My opinion is that l2fwd should be kept as simple as possible, so I'd suggest cloning to provide a separate keepalive example. /Bruce
