Open MPI logo

Open MPI Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Development mailing list

Subject: Re: [OMPI devel] [OMPI svn] svn:open-mpi r29633 - in trunk: . contrib contrib/dist/linux contrib/dist/linux/debian contrib/dist/linux/debian/source
From: Ralph Castain (rhc_at_[hidden])
Date: 2013-11-07 10:04:53


I'm sorry, but I just don't believe this belongs in our code base. If you want to create Debian packages (and I don't understand why), then do so in your own branch of the repo and maintain this stuff there. It just doesn't belong in our shared repository.

Please remove it, or explain why this should be here.
Ralph

On Nov 7, 2013, at 6:19 AM, svn-commit-mailer_at_[hidden] wrote:

> Author: miked (Mike Dubman)
> Date: 2013-11-07 09:19:27 EST (Thu, 07 Nov 2013)
> New Revision: 29633
> URL: https://svn.open-mpi.org/trac/ompi/changeset/29633
>
> Log:
> Move debian/ into contrib
> Refs: #3887
> reviewer=ompi-rm1.7
>
> Added:
> trunk/contrib/dist/linux/compile_debian_mlnx_example.in
> trunk/contrib/dist/linux/debian/
> trunk/contrib/dist/linux/debian/changelog.in
> trunk/contrib/dist/linux/debian/compat
> trunk/contrib/dist/linux/debian/control.in
> trunk/contrib/dist/linux/debian/rules.in
> trunk/contrib/dist/linux/debian/source/
> trunk/contrib/dist/linux/debian/source/format
> Text files modified:
> trunk/configure.ac | 11 ++++++++++-
> trunk/contrib/Makefile.am | 2 ++
> trunk/contrib/dist/linux/compile_debian_mlnx_example.in | 15 +++++++++++++++
> trunk/contrib/dist/linux/debian/changelog.in | 5 +++++
> trunk/contrib/dist/linux/debian/compat | 1 +
> trunk/contrib/dist/linux/debian/control.in | 15 +++++++++++++++
> trunk/contrib/dist/linux/debian/rules.in | 17 +++++++++++++++++
> trunk/contrib/dist/linux/debian/source/format | 1 +
> 8 files changed, 66 insertions(+), 1 deletions(-)
>
> Modified: trunk/configure.ac
> ==============================================================================
> --- trunk/configure.ac Thu Nov 7 04:28:43 2013 (r29632)
> +++ trunk/configure.ac 2013-11-07 09:19:27 EST (Thu, 07 Nov 2013) (r29633)
> @@ -1342,6 +1342,12 @@
>
> contrib/Makefile
>
> + contrib/dist/linux/compile_debian_mlnx_example
> + contrib/dist/linux/debian/changelog
> + contrib/dist/linux/debian/rules
> + contrib/dist/linux/debian/control
> + contrib/dist/linux/debian/copyright:LICENSE
> +
> test/Makefile
> test/event/Makefile
> test/asm/Makefile
> @@ -1350,7 +1356,10 @@
> test/support/Makefile
> test/threads/Makefile
> test/util/Makefile
> -])
> +],[
> + chmod +x contrib/dist/linux/debian/rules
> + chmod +x contrib/dist/linux/compile_debian_mlnx_example
> + ])
>
> OPAL_CONFIG_FILES
> m4_ifdef([project_orte], [ORTE_CONFIG_FILES])
>
> Modified: trunk/contrib/Makefile.am
> ==============================================================================
> --- trunk/contrib/Makefile.am Thu Nov 7 04:28:43 2013 (r29632)
> +++ trunk/contrib/Makefile.am 2013-11-07 09:19:27 EST (Thu, 07 Nov 2013) (r29633)
> @@ -33,6 +33,8 @@
> EXTRA_DIST = \
> dist/make_dist_tarball \
> dist/linux/openmpi.spec \
> + dist/linux/compile_debian_mlnx_example.in \
> + dist/linux/debian \
> dist/macosx-pkg/buildpackage.sh \
> dist/macosx-pkg/ReadMe.rtf \
> platform/optimized \
>
> Added: trunk/contrib/dist/linux/compile_debian_mlnx_example.in
> ==============================================================================
> --- /dev/null 00:00:00 1970 (empty, because file is newly added)
> +++ trunk/contrib/dist/linux/compile_debian_mlnx_example.in 2013-11-07 09:19:27 EST (Thu, 07 Nov 2013) (r29633)
> @@ -0,0 +1,15 @@
> +INSTALL_DIR=${INSTALL_DIR:-/usr/mpi/gcc}
> +PREFIX=${INSTALL_DIR}/openmpi-_at_OMPI_MAJOR_VERSION_at_.@OMPI_MINOR_VERSION_at_.@OMPI_RELEASE_VERSION@
> +
> +MXM_PATH=${MXM_PATH:-/opt/mellanox/mxm}
> +FCA_PATH=${FCA_PATH:-/opt/mellanox/fca}
> +KNEM_PATH=${KNEM_PATH:-/opt/knem-1.0.90mlnx2}
> +
> +[ -d $MXM_PATH ] && WITH_MXM="--with-mxm=$MXM_PATH"
> +[ -d $FCA_PATH ] && WITH_FCA="--with-fca=$FCA_PATH"
> +[ -d $KNEM_PATH ] && WITH_KNEM="--with-knem=$KNEM_PATH"
> +
> +CONFIG_ARGS=${CONFIG_ARGS:-"--prefix=$PREFIX --libdir=$OMPI_PREFIX/lib64 --with-platform=contrib/platform/mellanox/optimized $WITH_MXM $WITH_FCA $WITH_KNEM"}
> +export CONFIG_ARGS
> +cp -a contrib/dist/linux/debian .
> +dpkg-buildpackage -us -uc
>
> Added: trunk/contrib/dist/linux/debian/changelog.in
> ==============================================================================
> --- /dev/null 00:00:00 1970 (empty, because file is newly added)
> +++ trunk/contrib/dist/linux/debian/changelog.in 2013-11-07 09:19:27 EST (Thu, 07 Nov 2013) (r29633)
> @@ -0,0 +1,5 @@
> +openmpi (@OMPI_MAJOR_VERSION_at_.@OMPI_MINOR_VERSION_at_.@OMPI_RELEASE_VERSION@) unstable; urgency=low
> +
> + * Initial release.
> +
> + -- Aleksey Senin <alekseys_at_[hidden]> Thu, 02 May 2013 10:55:55 +0300
>
> Added: trunk/contrib/dist/linux/debian/compat
> ==============================================================================
> --- /dev/null 00:00:00 1970 (empty, because file is newly added)
> +++ trunk/contrib/dist/linux/debian/compat 2013-11-07 09:19:27 EST (Thu, 07 Nov 2013) (r29633)
> @@ -0,0 +1 @@
> +8
>
> Added: trunk/contrib/dist/linux/debian/control.in
> ==============================================================================
> --- /dev/null 00:00:00 1970 (empty, because file is newly added)
> +++ trunk/contrib/dist/linux/debian/control.in 2013-11-07 09:19:27 EST (Thu, 07 Nov 2013) (r29633)
> @@ -0,0 +1,15 @@
> +Source: openmpi
> +Section: net
> +Priority: extra
> +Homepage: http://www.open-mpi.org/
> +Maintainer: http://www.open-mpi.org
> +
> +Package: openmpi
> +Depends: ${shlibs:Depends}, ${misc:Depends}
> +Architecture: all
> +Description: Open MPI
> + Open MPI is a project combining technologies and resources from
> + several other projects (FT-MPI, LA-MPI, LAM/MPI, and PACX-MPI) in
> + order to build the best MPI library available.
> + This package contains all the tools necessary to compile, link, and run
> + Open MPI jobs.
>
> Added: trunk/contrib/dist/linux/debian/rules.in
> ==============================================================================
> --- /dev/null 00:00:00 1970 (empty, because file is newly added)
> +++ trunk/contrib/dist/linux/debian/rules.in 2013-11-07 09:19:27 EST (Thu, 07 Nov 2013) (r29633)
> @@ -0,0 +1,17 @@
> +#!/usr/bin/make -f
> +
> +# export DH_VERBOSE=1
> +
> +DPKG_EXPORT_BUILDFLAGS = 1
> +include /usr/share/dpkg/buildflags.mk
> +
> +%:
> + dh $@ --parallel
> +
> +override_dh_auto_clean:
> +
> +override_dh_auto_configure:
> + dh_auto_configure -- $(CONFIG_ARGS)
> +
> +override_dh_auto_test:
> +
>
> Added: trunk/contrib/dist/linux/debian/source/format
> ==============================================================================
> --- /dev/null 00:00:00 1970 (empty, because file is newly added)
> +++ trunk/contrib/dist/linux/debian/source/format 2013-11-07 09:19:27 EST (Thu, 07 Nov 2013) (r29633)
> @@ -0,0 +1 @@
> +3.0 (native)
> _______________________________________________
> svn mailing list
> svn_at_[hidden]
> http://www.open-mpi.org/mailman/listinfo.cgi/svn