Ben Warren via dev <[email protected]> writes: > Hi Aaron, > >> On Jan 5, 2017, at 7:52 AM, Aaron Conole <[email protected]> wrote: >> >> <snip> >> >>>>> Should we bump the libtool version now? We're nearing a release, so >>>>> it's about the right time. >>>> >>>> Sorry for the late response. I think it's probably appropriate to bump >>>> at least the libtool 'age'. I'm not sure what kinds of ABI changes have >>>> happened since introduction though - the interface 'major' version may >>>> need to change. >>> >>> I'm pretty sure we've busted ABIs left and right since 2014, so a major >>> version update should be appropriate. >> >> I was about to post a patch that simply bumped this version, but it sent >> me off onto a tangent leading to this question: >> >> Will Open vSwitch support the libopenvswitch.so as an externally >> linkable library? >> > Yes, as an example Debian now has a ‘openvswitch-dev’ package that > bundles the library and header files. Here’s how it currently looks > (Debian ‘sid’): > > ben@ben-lab-sc1:~$ dpkg -c > openvswitch-dev_2.6.2-pre+git20161223-2_amd64.deb | grep "lib.*\.so" > lrwxrwxrwx root/root 0 2016-12-23 16:35 ./usr/lib/libofproto.so -> > libofproto.so.1.0.0 > lrwxrwxrwx root/root 0 2016-12-23 16:35 ./usr/lib/libopenvswitch.so -> > libopenvswitch.so.1.0.0 > lrwxrwxrwx root/root 0 2016-12-23 16:35 ./usr/lib/libovn.so -> > libovn.so.1.0.0 > lrwxrwxrwx root/root 0 2016-12-23 16:35 ./usr/lib/libovsdb.so -> > libovsdb.so.1.0.0 > lrwxrwxrwx root/root 0 2016-12-23 16:35 ./usr/lib/libsflow.so -> > libsflow.so.1.0.0 > lrwxrwxrwx root/root 0 2016-12-23 16:35 ./usr/lib/libvtep.so -> > libvtep.so.1.0.0 > >> I know there was some work done in this direction, but nothing for ABI >> versioning or API compatibility has been done yet. >> > As you can see, the library naming isn’t very useful. As for ABI > versioning, I don’t think anything meaningful has been done. IMHO we > should probably change this so the major release is part of the name, > since that’s where I believe we should guarantee compatibility, along > the l lines of ‘libopenvswitch_2.5.so’ or whatever. I think there are > libtool directives for managing this naming, but we haven’t done that.
Posted an RFC patch with this type of functionality at: https://mail.openvswitch.org/pipermail/ovs-dev/2017-January/327472.html Thanks everyone for the input. -Aaron _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
