diff mbox series

[ovs-dev] debian: Move libovn out from package libopenvswitch.

Message ID 1535155616-6551-1-git-send-email-zhouhan@gmail.com
State Superseded
Headers show
Series [ovs-dev] debian: Move libovn out from package libopenvswitch. | expand

Commit Message

Han Zhou Aug. 25, 2018, 12:06 a.m. UTC 
From: Han Zhou <hzhou8@ebay.com>

Since we are packaging OVN and OVS components separately, libovn
shouldn't belong to OVS, so move it to ovn-common. Also, remove
it from libopenvswitch-dev.

Signed-off-by: Han Zhou <zhouhan@gmail.com>
---
 debian/libopenvswitch-dev.install | 12 ++++++++++--
 debian/libopenvswitch.install     |  6 +++++-
 debian/ovn-common.install         |  1 +
 3 files changed, 16 insertions(+), 3 deletions(-)

Comments

0-day Robot Aug. 25, 2018, 12:56 a.m. UTC | #1 
Bleep bloop.  Greetings Han Zhou, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
ERROR: Author Han Zhou <hzhou8@ebay.com> needs to sign off.
WARNING: Unexpected sign-offs from developers who are not authors or co-authors or committers: Han Zhou <zhouhan@gmail.com>
Lines checked: 60, Warnings: 1, Errors: 1


Please check this out.  If you feel there has been an error, please email aconole@bytheb.org

Thanks,
0-day Robot
Han Zhou Aug. 25, 2018, 1:03 a.m. UTC | #2 
On Fri, Aug 24, 2018 at 5:56 PM 0-day Robot <robot@bytheb.org> wrote:
>
> Bleep bloop.  Greetings Han Zhou, I am a robot and I have tried out your
patch.
> Thanks for your contribution.
>
> I encountered some error that I wasn't expecting.  See the details below.
>
>
> checkpatch:
> ERROR: Author Han Zhou <hzhou8@ebay.com> needs to sign off.
> WARNING: Unexpected sign-offs from developers who are not authors or
co-authors or committers: Han Zhou <zhouhan@gmail.com>
> Lines checked: 60, Warnings: 1, Errors: 1
>
>
> Please check this out.  If you feel there has been an error, please email
aconole@bytheb.org
>
> Thanks,
> 0-day Robot

Sorry, it seems I messed up the other way this time. I will send v2 trying
to making the signed-off aligned with author ;)
diff mbox series

Patch

diff --git a/debian/libopenvswitch-dev.install b/debian/libopenvswitch-dev.install
index ca3d22c..bda5a2c 100644
--- a/debian/libopenvswitch-dev.install
+++ b/debian/libopenvswitch-dev.install
@@ -1,5 +1,13 @@ 
-usr/lib/*/lib*.so
-usr/lib/*/lib*.a
+usr/lib/*/libopenvswitch*.so
+usr/lib/*/libofproto*.so
+usr/lib/*/libovsdb*.so
+usr/lib/*/libsflow*.so
+usr/lib/*/libvtep*.so
+usr/lib/*/libopenvswitch*.a
+usr/lib/*/libofproto*.a
+usr/lib/*/libovsdb*.a
+usr/lib/*/libsflow*.a
+usr/lib/*/libvtep*.a
 usr/lib/*/pkgconfig
 include/*.h usr/include/openvswitch
 include/openflow/*.h usr/include/openvswitch/openflow
diff --git a/debian/libopenvswitch.install b/debian/libopenvswitch.install
index 3ddde58..ce9dc40 100644
--- a/debian/libopenvswitch.install
+++ b/debian/libopenvswitch.install
@@ -1 +1,5 @@ 
-usr/lib/*/lib*.so.*
+usr/lib/*/libopenvswitch*.so.*
+usr/lib/*/libofproto*.so.*
+usr/lib/*/libovsdb*.so.*
+usr/lib/*/libsflow*.so.*
+usr/lib/*/libvtep*.so.*
diff --git a/debian/ovn-common.install b/debian/ovn-common.install
index e95ee6c..90484d2 100644
--- a/debian/ovn-common.install
+++ b/debian/ovn-common.install
@@ -4,3 +4,4 @@  usr/bin/ovn-trace
 usr/bin/ovn-detrace
 usr/share/openvswitch/scripts/ovn-ctl
 usr/share/openvswitch/scripts/ovndb-servers.ocf
+usr/lib/*/libovn*.so.*