diff options
| author | Ilya Maximets <i.maximets@ovn.org> | 2022-07-25 14:19:36 +0200 |
|---|---|---|
| committer | Ilya Maximets <i.maximets@ovn.org> | 2022-07-29 17:05:42 +0200 |
| commit | a68002cd9a93907c831667fcdb1fe2cae6b8c41f (patch) | |
| tree | 694d38c2e7c3e492fc84bb882e571836596b4b2a /ofproto | |
| parent | 099d1c7454477bbb0245ac1f5aed7c4061c65e91 (diff) | |
| download | openvswitch-a68002cd9a93907c831667fcdb1fe2cae6b8c41f.tar.gz | |
rhel: Stop installing internal headers.
Currently, openvswitch-devel installs following header tree:
/usr/include
/openflow/*.h
/openvswitch
/*.h
/openflow/*.h
/openvswitch/*.h
/sparse/*.h
/lib/*.h
Few issues with that:
1. openflow and openvswitch headers are installed twice. Once in the
main /usr/include and second time in the /usr/include/openvswitch/.
2. For some reason internal headers such as lib/*.h and fairly useless
headers such as sparse/*.h are installed as well.
One more issue is that current pkg-config files doesn't work with
builds installed with 'make install', because 'make install' doesn't
create this weird header tree.
While double install of same headers is not a huge problem, it doesn't
seem right. Installation of the internal headers is a bigger issue.
They are not part of API/ABI and we do not provide any stability
guarantees for them. We are making incompatible changes constantly in
minor updates, so users should not rely on these headers.
If it's necessary for some external application to use them, this
external application should not link with libopenvswitch dynamically
and also it can't expect the static library to not break these API/ABI,
hence there is no real point installing them. Application should use
OVS as a submodule like OVN does or compile itself by obtaining
required version of OVS sources otherwise. Another option is to
properly export and install required headers.
pkg-config configuration files updated as necessary.
Fixes: 4886d4d2495b ("debian, rhel: Ship ovs shared libraries and header files")
Reviewed-by: David Marchand <david.marchand@redhat.com>
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
Diffstat (limited to 'ofproto')
| -rw-r--r-- | ofproto/libofproto.pc.in | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ofproto/libofproto.pc.in b/ofproto/libofproto.pc.in index 274071250..49894fb69 100644 --- a/ofproto/libofproto.pc.in +++ b/ofproto/libofproto.pc.in @@ -8,4 +8,4 @@ Description: OpenFlow library of Open vSwitch Version: @VERSION@ Libs: -L${libdir} -lofproto Libs.private: @LIBS@ -Cflags: -I${includedir}/openvswitch +Cflags: -I${includedir} |