[Xorp-hackers] [PATCH 00/15] Add new operators for changing metric in policy code

[Ben Greear]

On 10/01/2012 01:12 AM, Igor Maravić wrote:
> Ben,
> I've just returned from my vacation.
> BR
> Igor

I pushed the 15 patches.  Seems it breaks the 'scons check' on Fedora 8 and the BSD
machine:

http://dmz2.candelatech.com:8010/waterfall

Don't worry about 'clang', it always fails.

Thanks,
Ben

>
> 2012/9/13 Igor Maravić <igorm at etf.rs>:
>> Ben,
>> I'm going on vacation this Saturday, and I'm going to be out of my
>> office for two weeks.
>> You can wait up with patches, if you wish, until I come back.
>> BR
>> Igor
>>
>> 2012/9/4 Ben Greear <greearb at candelatech.com>:
>>> On 08/31/2012 04:33 AM, igorm at etf.rs wrote:
>>>>
>>>> From: Igor Maravic <igorm at etf.rs>
>>>>
>>>> Hi,
>>>>
>>>> As suggested by Phil, I've added new operators for changing metric in
>>>> policy code.
>>>> New operators are *=, /=, <<=, >>=, &=, |= and ^=.
>>>>
>>>> I didn't have the time to test it thouroughly, so any volunteers to test
>>>> the code are welcome. :)
>>>>
>>>> In the first patch I've added new patern for one line comments.
>>>> Everything after %% is considered a comment.
>>>>
>>>> In the last patch I've hacked RIP so the routes that have metric >
>>>> RIP_INFINITY would be pushed through the EXPORT filter. I've tested it, and
>>>> it works. Unfortunatelly I'm not 100% sure if this hack is ok.
>>>>
>>>> All other patches should be fine!
>>>
>>>
>>> Just FYI, I haven't forgotten about these yet...and the previous patches
>>> for the xorp protocol plugin feature set.
>>>
>>> But, it will be a bit longer before I get to this..need to make sure
>>> I have the multicast fixes I've been working on all working properly
>>> first.
>>>
>>> I think after all of the pending patches are in, I'll start
>>> thinking about doing a release...
>>>
>>> Thanks,
>>> Ben
>>>
>>> --
>>> Ben Greear <greearb at candelatech.com>
>>> Candela Technologies Inc  http://www.candelatech.com
>>>


-- 
Ben Greear <greearb at candelatech.com>
Candela Technologies Inc  http://www.candelatech.com