[ovs-dev] [PATCH v2] debian: Add network integration scripts.
Gurucharan Shetty
gshetty at nicira.com
Fri May 4 23:57:55 UTC 2012
On Fri, May 4, 2012 at 1:54 PM, Ben Pfaff <blp at nicira.com> wrote:
> On Fri, May 04, 2012 at 01:32:34PM -0700, Gurucharan Shetty wrote:
>> diff --git a/debian/openvswitch-switch.postinst b/debian/openvswitch-switch.postinst
>> index 7b9d7bc..22ce434 100755
>> --- a/debian/openvswitch-switch.postinst
>> +++ b/debian/openvswitch-switch.postinst
>> @@ -33,6 +33,11 @@ case "$1" in
>> fi
>> done
>> fi
>> +
>> + ln -s /usr/share/openvswitch/scripts/ifupdown.sh \
>> + /etc/network/if-pre-up.d/openvswitch || true
>> + ln -s /usr/share/openvswitch/scripts/ifupdown.sh \
>> + /etc/network/if-post-down.d/openvswitch || true
>> ;;
>
> I just noticed this. .deb files can contain symlinks. Is there a
> reason to call ln in the postinst and rm in the prerm instead?
There is no reason. For some reason my openvswitch-switch.link file
contents are not used. The following patch works. May be there is a
better way to do this?
--- a/debian/rules
+++ b/debian/rules
@@ -111,6 +111,9 @@ install-arch: build-arch
$(MAKE) -C _debian DESTDIR=$(CURDIR)/debian/tmp install
cp debian/openvswitch-switch.template
debian/openvswitch-switch/usr/share/openvswitch/switch/default.template
dh_install -s
+ dh_link --package=openvswitch-switch \
+ usr/share/openvswitch/scripts/ifupdown.sh
etc/network/if-pre-up.d/openvswitch \
+ usr/share/openvswitch/scripts/ifupdown.sh
etc/network/if-post-down.d/openvswitch
# Must not depend on anything. This is to be called by
# binary-arch/binary-indep
More information about the dev
mailing list