[ovs-dev] [PATCH] debian: Backup OVSDB db file before converting it.
Justin Pettit
jpettit at nicira.com
Mon Feb 7 00:52:04 UTC 2011
On Feb 6, 2011, at 9:47 AM, Ben Pfaff wrote:
> On Sun, Feb 06, 2011 at 01:32:22AM -0800, Justin Pettit wrote:
>> The XenServer init script makes a backup before converting the OVSDB
>> database file. This may be useful for debugging, so do the same on
>> Debian. Also, store the schema version in the database.
>
> Did you test this? The bash online help for the "local" keyword (from
> "help local") says:
>
> Local variables can only be used within a function; they are visible
> only to the function where they are defined and its children.
>
> but your use of local is not within a function.
Yes, I tested this, and it does work fine even outside of functions (I just tested it with a simple script). However, it's clearly not correct, so I removed the "local" designations.
> It looks otherwise reasonable to me.
With that change, I went ahead and pushed.
Thanks for the review!
--Justin
More information about the dev
mailing list