[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