On Thu, May 16, 2013 at 11:13 AM, Bernardo Dal Seno <[email protected]>wrote:
> On 16 May 2013 10:54, Thomas Thrainer <[email protected]> wrote: > > Based on the discussion I think this patch series is definitely not the > > right place for such a change. > > Agreed. > > > I guess first we should change the way we handle version numbers during > > development, and then we should also gather realistic requirements for > > cfgupgrade, which could then lead to a re-write. > > Hopefully we won't need to change that much. > > > In the meantime, I'll rewrite this patch to only add/remove the > additional > > flag in cfgupgrade. IIRC the problem with upgrades in masterd were, that > > confd has to be able to read the config as well... > > But first we need a proper cfgupgrade for 2.9, i.e., one that > dowgrades to 2.8 instead of 2.7. I have the patch ready and I can send > it, if you need it, but I was waiting for 2.8 to be packaged so we had > the proper version numbers in place and avoid confusion. But maybe you > want to push this series after the cmdlib split anyway. Please let me > know which you prefer. > As you are much deeper into this topic, and my changes are really minor (see new patch), I'll leave that to you :). I'd personally prefer to push this series after the cmdlib split, if that goes through soon-ish. Thanks, Thomas > > Bernardo > -- Thomas Thrainer | Software Engineer | [email protected] | Google Germany GmbH Dienerstr. 12 80331 München Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg Geschäftsführer: Graham Law, Katherine Stephens
