<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "http://bugs.gentoo.org/bugzilla.dtd">

<bugzilla version="2.22.7"
          urlbase="http://bugs.gentoo.org/"
          maintainer="bugzilla@gentoo.org"
>

    <bug>
          <bug_id>217854</bug_id>
          
          <creation_ts>2008-04-15 20:31 0000</creation_ts>
          <short_desc>sci-libs/hdf5-1.6.6 - use package.use.mask instead of putting arch in DEPEND</short_desc>
          <delta_ts>2008-06-13 13:25:04 0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>Gentoo Linux</product>
          <component>Applications</component>
          <version>2008.0_beta1</version>
          <rep_platform>Sparc</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>jer@gentoo.org</reporter>
          <assigned_to>sci@gentoo.org</assigned_to>
          <cc>sparc@gentoo.org</cc>

      

      
          <long_desc isprivate="0">
            <who>jer@gentoo.org</who>
            <bug_when>2008-04-15 20:31:10 0000</bug_when>
            <thetext>It&apos;s better to add &apos;sci-libs/hdf5 mpi&apos; to sparc profiles (so that emerge -p hdf5 would show the USE flag as masked), than to put this sort of stuff in the ebuilds (where USE=mpi would appear useful but is silently ignored).

Index: hdf5-1.6.6.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/sci-libs/hdf5/hdf5-1.6.6.ebuild,v
retrieving revision 1.5
retrieving revision 1.6
diff -u -B -r1.5 -r1.6
--- hdf5-1.6.6.ebuild   9 Apr 2008 01:34:59 -0000       1.5
+++ hdf5-1.6.6.ebuild   15 Apr 2008 14:06:33 -0000      1.6
@@ -1,6 +1,6 @@
 # Copyright 1999-2008 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/sci-libs/hdf5/hdf5-1.6.6.ebuild,v 1.5 2008/04/09 01:34:59 jer Exp $
+# $Header: /var/cvsroot/gentoo-x86/sci-libs/hdf5/hdf5-1.6.6.ebuild,v 1.6 2008/04/15 14:06:33 markusle Exp $

 inherit eutils fixheadtails flag-o-matic fortran toolchain-funcs

@@ -10,12 +10,12 @@

 LICENSE=&quot;NCSA-HDF&quot;
 SLOT=&quot;0&quot;
-KEYWORDS=&quot;~amd64 ~hppa ~ppc ~ppc64 ~x86&quot;
+KEYWORDS=&quot;~amd64 ~hppa ~ppc ~ppc64 ~x86 ~sparc&quot;
 # need to update szip to get alpha, ia64, etc back in here,
 IUSE=&quot;cxx debug fortran mpi ssl szip threads tools zlib &quot;

-DEPEND=&quot;mpi? ( &gt;=sys-cluster/mpich2-1.0.6
-               net-fs/nfs-utils )
+DEPEND=&quot;!sparc? ( mpi? ( &gt;=sys-cluster/mpich2-1.0.6
+               net-fs/nfs-utils ) )
        ssl? ( dev-libs/openssl )
        szip? ( sci-libs/szip )
        zlib? ( sys-libs/zlib )
[...]</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>dberkholz@gentoo.org</who>
            <bug_when>2008-04-16 00:41:06 0000</bug_when>
            <thetext>Did you try testing to see if it works at this point, instead of masking it?</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>markusle@gentoo.org</who>
            <bug_when>2008-04-16 14:08:03 0000</bug_when>
            <thetext>It would indeed have been better to fix this in the sparc
profile itself. Please feel free to do this. I am always very 
hesitant with messing around in profiles of arches I don&apos;t 
know anything about, which is probably why I decided to 
go the ebuild route. For future reference, is changing arch
profiles in this way an ok thing for me to do?

In any case, if at all possible, it would be best to keyword 
sys-cluster/mpich2 and also sys-cluster/openmpi (which will
likely be the future mpi workhorse) on sparc.

Thanks for your patience.

Best,
Markus </thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>jer@gentoo.org</who>
            <bug_when>2008-04-16 14:33:21 0000</bug_when>
            <thetext>Should be really easy to fix. hdf5 depends on virtual/mpi which gives 4 alternatives - if any of those MPI implementations works, then only that implementation should be keyworded for sparc. I&apos;ll give openmpi a whirl.

Also note that only arch team members should normally edit arch profiles, and that your hesitation isn&apos;t out of place at all.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>jer@gentoo.org</who>
            <bug_when>2008-04-16 14:41:09 0000</bug_when>
            <thetext>OK, sys-cluster/lam-mpi and sys-cluster/mpich2 are keyworded already (even though for the latter, sparc and ia64 keywords were dropped - I do not know whether this was done in an appropriate manner). As it is now, virtual/mpi should already work for sparc or it shouldn&apos;t be used at all - if a new-style virtual doesn&apos;t implement compatible (drop in replacement) alternatives then it shouldn&apos;t be used.

Besides, if openmpi is going to be the best of four alternatives in the future, then the order of the alternative packages in virtual/mpi-1.0.ebuild should probably be changed:

RDEPEND=&quot;|| (
        sys-cluster/mpich
        sys-cluster/lam-mpi
        sys-cluster/openmpi
        sys-cluster/mpich2
    )&quot;</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>jsbronder@gentoo.org</who>
            <bug_when>2008-04-16 14:47:25 0000</bug_when>
            <thetext>
&gt; Besides, if openmpi is going to be the best of four alternatives in the future,
&gt; then the order of the alternative packages in virtual/mpi-1.0.ebuild should
&gt; probably be changed:

We&apos;re working on it, I&apos;m adding sparc to the keywording request :)

See bug #216120
</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>fmccor@gentoo.org</who>
            <bug_when>2008-04-16 16:06:17 0000</bug_when>
            <thetext>openmpi is now keyworded ~sparc.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>markusle@gentoo.org</who>
            <bug_when>2008-04-17 13:11:36 0000</bug_when>
            <thetext>Thanks much, Ferris! hdf5 currently hard depends
on mpich2. I&apos;ll have to check if it can handle
openmpi so we can depend on it and revive the mpi 
useflag on sparc.

cheers,
Markus</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>armin76@gentoo.org</who>
            <bug_when>2008-06-05 17:09:03 0000</bug_when>
            <thetext>Whats up here?</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>markusle@gentoo.org</who>
            <bug_when>2008-06-06 14:36:25 0000</bug_when>
            <thetext>Unfortunately, I don&apos;t currently have the time
to investigate this in any depth. In the meantime,
would it be possible for the spark team to mask USE=&apos;mpi&apos; in 
their profile so we can remove it from DEPEND until 
somebody has the time to investigate the mpi situation.

Thanks much.
Markus</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>armin76@gentoo.org</who>
            <bug_when>2008-06-10 14:05:08 0000</bug_when>
            <thetext>Done :)</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>markusle@gentoo.org</who>
            <bug_when>2008-06-11 09:36:36 0000</bug_when>
            <thetext>(In reply to comment #10)
&gt; Done :)
&gt; 

Thank you very much! However, I think you masked
mpi globally in use.mask rather than only for hdf5
in package.use.mask. The mpi useflag itself should
be fine since sys-cluster/openmpi is keyworded on
sparc; only mpich2 which hdf5-1.6.6. depends on 
is missing.

Thanks,
Markus</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>bluebird@gentoo.org</who>
            <bug_when>2008-06-12 21:16:58 0000</bug_when>
            <thetext>Right, virtual/mpi can be provided by openmpi and lam-mpi on sparc.
I moved it to package.use.mask</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>markusle@gentoo.org</who>
            <bug_when>2008-06-13 13:25:04 0000</bug_when>
            <thetext>Great and thank you very much! I&apos;ve removed sparc from
DEPEND in the ebuild.

Thanks,
Markus</thetext>
          </long_desc>
      
    </bug>

</bugzilla>