summaryrefslogtreecommitdiff
path: root/debian/rules
Commit message (Collapse)AuthorAgeFilesLines
* Edit Open vSwitch license info so that GitHub recognizes it.Andrea Kao2018-04-161-2/+1
| | | | | | | | | | | | | | | | | | | GitHub uses a library called Licensee to identify a project's license type. It shows this information in the status bar and via the API if it can unambiguously identify the license. This commit creates a LICENSE file that stores the full text of the Apache license. It also removes the COPYING file and transfers its contents to a new "License" section in the README. Collectively, these changes allow Licensee to successfully identify the license type of Open vSwitch's codebase as Apache. Submitted-at: https://github.com/openvswitch/ovs/pull/224 Signed-off-by: Andrea Kao <eirinikos@gmail.com> [blp@ovn.org removed references to COPYING and updated Makefile.am] Signed-off-by: Ben Pfaff <blp@ovn.org> Acked-by: Aaron Conole <aconole@redhat.com>
* debian: Use debhelper for DEB_UPSTREAM_VERSIONClint Byrum2017-07-071-8/+8
| | | | | | | | debhelper provides helpers to get parts of the version number, including this part which was until now figured out in an open-coded way. Signed-off-by: Clint Byrum <clint@fewbar.com> Signed-off-by: Ben Pfaff <blp@ovn.org>
* debian; Avoid installing ovs-vswitchd.conf.db manpage as "db" language.Ben Pfaff2017-03-171-0/+3
| | | | | | | | | | | Usually, when the name of a manpage has a two-letter extension, it means that the manpage is written in the language designated by that language code. Reported-by: Michael Stapelberg <stapelberg@debian.org> Reported-at: https://bugs.debian.org/850631 Signed-off-by: Ben Pfaff <blp@ovn.org> Tested-by: nickcooper-zhangtonghao <nic@opencloud.tech>
* openvswitch: FTBFS with dpkg-buildpackage.nickcooper-zhangtonghao2016-12-231-1/+4
| | | | | | | | | | | | | | | | The debian packages are ready. This patch fixes the bug #831924 reported at debian bug tracking system. With this patch, openvswitch-2.6.1 will be upload to the Debian archive. If we build the packages with "dpkg-buildpackage --target binary-indep", an error state arises. debian/rules should be modified so that the build-indep and binary-indep target generates the architecture independent packages. If there are things not be handled properly,let me know. Reported-at: https://people.debian.org/~lucas/logs/2016/07/20/openvswitch_2.5.1~pre+git20160626-2_unstable_archallonly.log Signed-off-by: nickcooper-zhangtonghao <nic@opencloud.tech> Signed-off-by: Ben Pfaff <blp@ovn.org>
* debian, rhel: Ship ovs shared libraries and header filesEdwin Chiu2016-06-071-1/+3
| | | | | | | | | | Compile and package ovs shared libraries and create new header package for debian (openvswitch-dev) and rhel (openvswitch-devel). VMware-BZ: #1556299 Signed-off-by: Edwin Chiu <echiu@vmware.com> Co-authored-by: Harold Lim <haroldl@vmware.com> Signed-off-by: Ben Pfaff <blp@ovn.org>
* debian: Fix treatment of upstream version that contains hyphens.Simon Horman2016-04-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | The Debian Policy Manual (https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Version) says that the upstream_version may contain only alphanumerics and the characters . + - : ~ (full stop, plus, hyphen, colon, tilde) and should start with a digit. Currently, the upstream_version is defined in the debian/rules file: DEB_UPSTREAM_VERSION=$(shell dpkg-parsechangelog | sed -rne 's,^Version: ([0-9]:)*([^-]+).*,\2,p') The version number is taken from the dpkg-parsechangelog printout then the first part of the version number which does not contain hyphen is filtered out with sed. However the Debian Policy Manual says that hyphen is allowed in the upstream_version. This is not a problem with current vanilla OVS debian version. But, if a postfix string including a hyphen is added to the upstream_version then installation of datapath-dkms package will fail. Reported-by: Zoltán Balogh <zoltan.balogh@ericsson.com> Tested-by: Zoltán Balogh <zoltan.balogh@ericsson.com> Signed-off-by: Ben Pfaff <blp@ovn.org>
* tests: Add ability to automatically rerun failed tests.Ben Pfaff2016-02-241-2/+1
| | | | | | | | A lot of packaging was doing this already, so this simplifies their implementation. Signed-off-by: Ben Pfaff <blp@ovn.org> Acked-by: Andy Zhou <azhou@ovn.org>
* debian: Add a package for OVN central components.Gurucharan Shetty2016-01-051-0/+3
| | | | | Signed-off-by: Gurucharan Shetty <guru@ovn.org> Acked-by: Ben Pfaff <blp@ovn.org>
* debian: Add a package for host components.Gurucharan Shetty2016-01-051-0/+3
| | | | | Signed-off-by: Gurucharan Shetty <guru@ovn.org> Acked-by: Ben Pfaff <blp@ovn.org>
* debian: Rationalize packaging using new debhelper.James Page2015-07-131-113/+44
| | | | | | | | | | | Newer debhelper versions support overrides, making rules files simpler and easier to extend. Requires some changes in how files are installed (_debian build folder dropped). Signed-off-by: James Page <james.page@ubuntu.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: Remove obsolete manual setting of CFLAGS and warnings from rules.Ben Pfaff2015-06-101-8/+1
| | | | | | | | Setting CFLAGS by hand before invoking dpkg-buildflags is ineffective, because dpkg-buildflags overrides it. Reported-by: Andrey Korolyov <andrey@xdel.ru> Signed-off-by: Ben Pfaff <blp@nicira.com>
* configure: Stop avoiding -Wformat-zero-length.Ben Pfaff2015-06-101-7/+0
| | | | | | | | | | | | | | | | | | | Debian likes to enable -Wformat-zero-length, even over our code trying to disable it. It isn't too hard to make our code warning-free against this option, so this commit both stops disabling it and fixes the warnings. The first fix is to change set_subprogram_name() to take a plain string instead of a format string, and to adjust its few callers. This fixes one warning since one of those callers passed in an empty string. The second fix is to remove a test for ovs_scan() against an empty string. I couldn't find a way to avoid a warning for this test, and it isn't too valuable in any case. This allows us to drop filtering for -Wformat from the Debian rules file, so this commit removes it. Signed-off-by: Ben Pfaff <blp@nicira.com>
* dkms.conf.in: Install all kernel modules.Alex Wang2015-04-181-1/+4
| | | | | | | | | With the latest change of separating vports into their own modules, we need to update the dkms.conf.in and make dkms install all vport modules. So, this commit modifies the debian/rules to read all kernel module names and sets the dkms.conf correctly. Signed-off-by: Alex Wang <alexw@nicira.com>
* debian: Fix cross build.Ed Swierk2014-08-261-1/+1
| | | | | | | | | | Cross-building openvswitch with debuild -aARCH (or equivalent) fails because the target architecture is not getting passed to configure. Thus binaries like ovs-appctl get built using the build host architecture. Signed-off-by: Ed Swierk <eswierk@skyportsystems.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: Really fix packaging given new dot-files.Ben Pfaff2014-08-261-1/+1
| | | | | | | | commit a5c1096abb1 (debian: Fix debian packaging build failures due to dot-files in tree.) contained an error that I've made before and should have noticed. This commit should really fix the problem. Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: Fix debian packaging build failures due to dot-files in tree.Ben Pfaff2014-08-261-1/+1
| | | | | | | | | | openvswitch/* no longer matches all the files since commit 826bc7b61345a5d (build: Add travis continuous integration support) added files with names beginning with ".". VMware-BZ: #1298353 Signed-off-by: Ben Pfaff <blp@nicira.com> Acked-by: Justin Pettit <jpettit@nicira.com>
* debian: Avoid -Wformat-zero-length warnings.Ben Pfaff2014-07-221-0/+7
| | | | | | | | | | Debian puts an extra "-Wformat" in the CFLAGS following OVS's own "-Wformat -Wno-format-zero-length", which therefore overrides -Wno-format-zero-length, so this commit adds an extra -Wno-format-zero-length to avoid those false positives. Signed-off-by: Ben Pfaff <blp@nicira.com> Acked-by: Justin Pettit <jpettit@nicira.com>
* debian: Re-run failed unit tests once.Gurucharan Shetty2014-06-241-1/+2
| | | | | | | | | | | There are always a few unit tests that have some race conditions that cause them to occasionally fail. It is probably a good idea to re-run the failed tests once while building deb packages. Signed-off-by: Gurucharan Shetty <gshetty@nicira.com> Co-authored-by: Keith Amidon <kamidon@nicira.com> Signed-off-by: Keith Amidon <kamidon@nicira.com> Acked-by: Ben Pfaff <blp@nicira.com>
* ovsdbmonitor: Remove.Ben Pfaff2014-01-101-1/+0
| | | | | | | ovsdbmonitor was poorly maintained and not widely used. Acked-by: Flavio Leitner <fbl@redhat.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: Fix build with old versions of dpkg-buildflags.Ben Pfaff2013-08-131-2/+7
| | | | | | | | | dpkg-buildflags has not always supported --export=configure, but commit 6c2d4c8780 (debian: Apply hardening options to build.) used it unconditionally, causing the build to fail on old Debian distributions. This fixes the problem. Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: Apply hardening options to build.Ben Pfaff2013-08-131-0/+1
| | | | | | | | | | | Debian now encourages building every program with various GCC hardening options. This commit implements that recommendation for Open vSwitch. See https://wiki.debian.org/Hardening for details. Found by lintian. Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: build with debugging symbolBen Pfaff2013-04-151-0/+1
| | | | | Signed-off-by: Zang MingJie <zealot0630@gmail.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: Add network integration scripts.Gurucharan Shetty2012-05-041-0/+1
| | | | | | | | Add scripts that will allow Open vSwitch bridges and ports to be configured through /etc/network/interfaces. This patch follows a very similar style as OVS network integration for rhel. Signed-off-by: Gurucharan Shetty <gshetty@nicira.com>
* configure: Remove --with-build-number.Ben Pfaff2012-03-191-14/+3
| | | | | | | | | | From early days, Nicira used the --with-build-number option to configure to stamp our internal builds. We've since switched to another scheme, so this option is obsolete. Good riddance. Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: Use a different way to avoid failing install without kernel module.Ben Pfaff2012-03-161-2/+1
| | | | | | | | | | | | | | | | | | The dh_installinit --error-handler option makes a lot of sense, but after playing with it for a while I could not figure out a nice way to use it only for openvswitch-switch without either duplicating the dh_installinit fragments in postinst and prerm (the actual bug that was reported) or omitting them for some package. Also, we forgot to write the error handler function for the prerm. This commit switches to a different way to avoid failing the install when the kernel module is not available, without using --error-handler. CC: 663051@bugs.debian.org Reported-by: Thomas Goirand <zigo@debian.org> Reviewed-by: Simon Horman <horms@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: Avoid unit test failure when doing "unofficial" builds.Ben Pfaff2012-03-121-1/+7
| | | | | | | | | | | | | The configure option --with-build-number=0 is interpreted differently in different places. The configure script itself accepts 0 as an actual build number and puts '#define BUILDNR "+build0"' into config.h. The code in python/automake.mk treats 0 as "no build number" and puts 'BUILDNR = ""' into version.py. This commit avoids the problem by not passing 0 as a build number. Reviewed-by: Simon Horman <horms@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: Remove some useless files from the dkms pacakge.Thomas Goirand2012-03-121-0/+10
| | | | | | | | | | | | This commit removes useless files from the dkms package that caused lintian warnings. (Many of the other files in the dkms package are also useless but do not cause lintian warnings so they are less important.) Reviewed-by: Simon Horman <horms@verge.net.au> Signed-off-by: Thomas Goirand <zigo@debian.org> Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: Clean .pyc files in "clean" target.Thomas Goirand2012-03-121-0/+1
| | | | | | Reviewed-by: Simon Horman <horms@verge.net.au> Signed-off-by: Thomas Goirand <zigo@debian.org> Signed-off-by: Ben Pfaff <blp@nicira.com>
* debian: Strip epoch from version number used in directory names.Ben Pfaff2011-11-101-1/+1
| | | | | | | | This doesn't fix a visible bug, since there's no epoch in the Open vSwitch version used in Debian, but some Nicira internal build scripts were inserting an epoch so it was visible in our builds. Reported-by: Edwin Chiu <echiu@nicira.com>
* debian: Install ovs-vswitchd.conf.db(5) manpage.Ben Pfaff2011-11-041-1/+1
| | | | | | | | | | | | This manpage wasn't getting installed. This fixes it. The --language=C option to dh_installman is necessary to keep that script from thinking that the ".db" suffix indicates a translation into the "db" language (which doesn't actually exist) and therefore installing it into the wrong directory with the .db suffix stripped. Bug #8138. Reported-by: Ethan Jackson <ethan@nicira.com>
* Debian: fail gracefully if modules can't be loaded on installSimon Horman2011-09-121-1/+2
| | | | | | | | | | | | | | | | | | By registering an error-handler for the init script used in openvswitch-switch.postinst and detecting if module insertion fails, it is possible to avoid failure to install in the case where the openvswitch_mod module is not available. This is done without altering the behaviour that the start target of the openvswitch-switch init script will fail if module insertion fails. This patch also adds a friendly hint as as to why starting openvswitch-switch has failed if it is due to failure to insert the openvswtich_mod. This message is displayed as necessary both on package install and other calls to the start target of the init script. [Ben Pfaff fixed up == to = in postinst]
* debian: Transition to dh_python2Simon Horman2011-08-261-1/+1
| | | | | | | | | | dh_pysupport that the packaging used until now is deprecated, with dh_python2 as its successor. This commit removes the PYTHONPATH setting from debian/openvswitch-ipsec.init because it is not needed, as the Python packaging is public. In fact, the Python packaging was public, unintentionally, before, so the PYTHONPATH could have been removed earlier.
* debian: Apply Ubuntu patch to add DKMS support.Chuck Short2011-08-251-0/+15
| | | | | | | | | | | | | | | | | | | | | | | I tested that installing openvswitch-datapath-dkms worked OK on my own Debian machine. The bulk of this patch is taken from downstream Ubuntu DKMS support written by Chuck Short <zulcss@ubuntu.com>, version 1.2.0-1ubuntu1. I made the following changes: * Update debian/.gitignore. * Update debian/automake.mk. * Correct description in debian/control (it was a cut-and-paste from the openvswitch-datapath-source description without editing). * Fix up for --with-l26 to --with-linux and datapath/linux-2.6 to datapath/linux transitions. CC: Chuck Short <zulcss@ubuntu.com> CC: Dave Walker <DaveWalker@ubuntu.com> Acked-by: Simon Horman <horms@verge.net.au>
* debian: Fix bug from commit 211b05b5 "debian: Modernize use of dh_install."Ben Pfaff2011-07-261-0/+1
| | | | | | | | | | | | Commit 211b05b5 "debian: Modernize use of dh_install" caused build failures in clean environments because debian/ovsdbmonitor.install now installed files that were never created in such an environment. It didn't cause a build failure in my own testing because I still had old files around. This fixes the problem. It seems to be necessary to run "make install" once for each of binary-arch and binary-indep, because dh_prep always deletes debian/tmp. It is logically not necessary but I don't see a clean and robust way to avoid it.
* debian: Modernize use of dh_install.Ben Pfaff2011-07-261-3/+3
| | | | | | | | | | | | | Originally I intended this as just a cleanup, but as a side effect it also installs some files from the install tree in debian/tmp instead of from _debian. This should avoid a reported problem in which ovs-bugtool was being created in the source directory instead of the build directory (I still don't see why this happened). Reported-by: Sébastien RICCIO <sr@swisscenter.com> Tested-by: Sébastien RICCIO <sr@swisscenter.com> Acked-by: Simon Horman <horms@verge.net.au> CC: Simon Horman <horms@verge.net.au>
* debian: Remove obsolete reference to var/log/core.Ben Pfaff2011-07-061-1/+1
| | | | | Commit 14c3b136 "debian: Remove corekeeper package" removed the var/log/core directory from the Debian packaging, but I missed this part.
* Refactor initscripts into distro-independent and distro-specific pieces.Ben Pfaff2011-06-171-1/+1
| | | | This should make it easier to add OVS support to new distributions.
* debian: Restart daemons only after upgrade is complete.Ben Pfaff2011-03-141-1/+1
| | | | | | | | | | This yields less downtime during an upgrade and allows an SSH session or other network access to dpkg to continue during the upgrade to, e.g., answer questions about conffiles from dpkg. (Open vSwitch doesn't have any conffiles but other packages being upgraded in the same dpkg run might.) Bug #542.
* debian: Run tests in parallel too.Ben Pfaff2011-03-141-3/+6
| | | | | Passing e.g. -j4 in TESTSUITEFLAGS runs 4 tests in parallel, which speeds up the testsuite.
* debian: Remove openvswitch-pki-server package.Ben Pfaff2011-01-041-1/+0
| | | | | | | The functionality provided by this package was used only by openvswitch-switch-config, which was removed some time ago. If I had been more careful at the time, I would have removed this package at the same time.
* debian: Fix parallel builds.Ben Pfaff2010-11-161-0/+7
| | | | | | | | | | | | | Some of the Debian automatic builds are failing apparently because the build-arch and build-indep targets are being called in parallel and they are both attempting to rebuild tests/testsuite at the same time, which doesn't work. We could make the OVS makefiles handle this particular case, but in general it's not a good idea to independently invoke a single makefile multiples times in parallel, so this commit avoids that problem. This also dumps the DEB_BUILD_OPTIONS variable to the log, because there doesn't appear to be any way to infer it from the automatic builders' log files.
* debian: On "make check" failure dump the testsuite log.Ben Pfaff2010-11-161-1/+4
| | | | | | The Debian automatic build machines don't save any files from the build, but they do preserve the build log, so dumping it to stdout is the only reasonable way to get test failure information.
* debian: Remove corekeeper package.Ben Pfaff2010-11-101-1/+0
| | | | | The corekeeper package has little or nothing to do with Open vSwitch itself, so remove it.
* debian: Add openvswitch-python packageSajjad Lateef2010-09-201-0/+1
| | | | | | | | | | | | | This installs the Python runtime bindings for Open vSwitch database into /usr/share/python-support/openvswitch-python/ovs and /usr/share/python-support/openvswitch-python/ovs/db Updated FSF address in copyright file Minor Whitespace re-formatting Removed prerm, preinst, postinst files for openvswitch-python
* Remove openvswitch-switch-config package.Ben Pfaff2010-08-301-2/+0
| | | | | | | | This package was never used by many users, has not worked properly for a while, and causes lintian warnings, so remove it. CC: Simon Horman <horms@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Remove ezio-term and ovs-switchui utilities.Ben Pfaff2010-08-231-1/+0
| | | | | | | These utilities were useful when Nicira was building switches with 16x2 LCD front panel displays, but they aren't useful for other environments and even Nicira does not use that kind of switch any longer. So remove them and all the build infrastructure on which they depended.
* Cleanly separate IDL annotations from OVSDB schema information.Ben Pfaff2010-01-261-4/+0
| | | | | | | | | | | | | | | | | Until now, the OVSDB IDL annotations have been glommed together with the schema information in a single file, and then we've used ovsdb-idlc to extract the schema from that file. This commit reverses the process: the schema and the annotations are stored separately and then glommed together as necessary at build time. This new arrangement has a few advantages: - We can now easily have multiple different sets of IDL annotations for a single OVSDB schema. For example, some users may not need access to columns that other users do. - Bugs in ovsdb-idlc cannot screw up the underlying schema (as shown by a recent commit).
* debian: Tolerate varying locations of vswitch-idl.ovsschema.Ben Pfaff2009-12-171-0/+4
| | | | | The Debian packaging as well as the testsuite was picky about where the ovsschema file appeared. This should fix the problem.
* debian: Break rules for datapath module out of debian/rules.Ben Pfaff2009-12-151-42/+5
| | | | | | | | | debian/rules included makefiles from /usr/share/modass/include. Unfortunately these makefiles set some environment variables to values that we do not want in the general Debian build, e.g. on this machine they set CC to gcc-4.1. It appears that it is generally good practice to break out the kernel module rules from the general-purpose rules anyhow, so this commit does so.
* debian: Honor "noopt", "parallel=<n>" build options.Ben Pfaff2009-06-091-1/+12
|