On Nov 7, 2013, at 8:54 AM, Mike Dubman <mi...@dev.mellanox.co.il> wrote:

> how it is differ from other related stuf kept in svn to support:
> 
> - rpm based distros? (dist/linux/)
> - macos (contrib/dist/macosx-pkg/)
> - __debian_stuff_could_be_here_as_well

That's a fair point - I'll let the folks who added those directories chime in 
as to their views on the difference, if any - I think some of those may be 
historical and quite stale (i.e., they may only be there because we forgot they 
existed)

> 
> no prob to remove it, i think it is handy, logical  and helps to at least one 
> project member and most likely to others (even ones in denial).
> 
> We use it to package OMPI into MOFED for debian.
> Let me know if it helps or we should still remove it.
> 
> 
> 
> 
> 
> On Thu, Nov 7, 2013 at 5:04 PM, Ralph Castain <r...@open-mpi.org> wrote:
> 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-mai...@open-mpi.org 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-@OMPI_MAJOR_VERSION@.@OMPI_MINOR_VERSION@.@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@.@OMPI_MINOR_VERSION@.@OMPI_RELEASE_VERSION@) 
> > unstable; urgency=low
> > +
> > +  * Initial release.
> > +
> > + -- Aleksey Senin <aleks...@mellanox.com>  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
> > s...@open-mpi.org
> > http://www.open-mpi.org/mailman/listinfo.cgi/svn
> 
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/devel
> 
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/devel

Reply via email to