Re: [Ipsec-tools-devel] [PATCH 1/2] adminport event refactoring

[<ma...@ne...>]

Timo Ter=E4s <tim...@ik...> wrote:

> We could make it backward compatible, by adding some flag the client need=
s
> to sent to get the pushed events. Though then there needs to be quite a b=
it
> of unneeded code for to support the old style API.

That's an extra work that needs to be balanced with the problems users
will experience (and will report to the list) because of botched
upgrades.=20

Full blown backward compatibility would be great. If that's too much
work, then IMO we should detect the version mismatch and issue a
meaningful error message.

As I understand,detecting a version mismatch is easy: you just have to
avoid reusing the same message types when you altered message semantics.
When you enhance a command, just use a new message type.

What I'm not sure about is the difficulty of keeping the obsoleted
message types functionnal. Can it be done easily? If it does not, then
we can go the meaningful error message path: racoon could log an error
and exit, leaving no way for the admin to miss the problem.

--=20
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
ma...@ne...