Applied, thx!
On Fri, May 30, 2014 at 5:30 PM, Alex Wang <al...@nicira.com> wrote: > LGTM, > > Acked-by: Alex Wang <al...@nicira.com> > > > On Fri, May 30, 2014 at 4:59 PM, Ryan Wilson <wr...@nicira.com> wrote: > >> Older versions of Python do not have ctypes as a default installed >> package. This patch puts the 'import ctypes' statement inside a try >> statement. >> >> This fixes a bug introduced by commit 8396f (timeval: Use monotonic >> time in OVS Python timeval library). >> >> Signed-off-by: Ryan Wilson <wr...@nicira.com> >> --- >> python/ovs/timeval.py | 16 ++++++++-------- >> 1 file changed, 8 insertions(+), 8 deletions(-) >> >> diff --git a/python/ovs/timeval.py b/python/ovs/timeval.py >> index f2681ac..4463e87 100644 >> --- a/python/ovs/timeval.py >> +++ b/python/ovs/timeval.py >> @@ -12,20 +12,20 @@ >> # See the License for the specific language governing permissions and >> # limitations under the License. >> >> -import ctypes >> -import sys >> import time >> >> LIBRT = 'librt.so.1' >> CLOCK_MONOTONIC = 1 >> >> -class timespec(ctypes.Structure): >> - _fields_ = [ >> - ('tv_sec', ctypes.c_long), >> - ('tv_nsec', ctypes.c_long), >> - ] >> - >> try: >> + import ctypes >> + >> + class timespec(ctypes.Structure): >> + _fields_ = [ >> + ('tv_sec', ctypes.c_long), >> + ('tv_nsec', ctypes.c_long), >> + ] >> + >> librt = ctypes.CDLL(LIBRT) >> clock_gettime = librt.clock_gettime >> clock_gettime.argtypes = [ctypes.c_int, ctypes.POINTER(timespec)] >> -- >> 1.7.9.5 >> >> _______________________________________________ >> dev mailing list >> dev@openvswitch.org >> http://openvswitch.org/mailman/listinfo/dev >> > >
_______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev