Skip to Content.
Sympa Menu

sm-discuss - Re: [SM-Discuss] PERFORCE change 71103 by Andrew Stitt for review

sm-discuss AT lists.ibiblio.org

Subject: Public SourceMage Discussion List

List archive

Chronological Thread  
  • From: Seth Alan Woolley <seth AT positivism.org>
  • To: sm-discuss AT lists.ibiblio.org
  • Subject: Re: [SM-Discuss] PERFORCE change 71103 by Andrew Stitt for review
  • Date: Tue, 6 Dec 2005 10:24:02 -0800

On Tue, Dec 06, 2005 at 06:30:06PM +0100, Arwed von Merkatz wrote:
> On Tue, Dec 06, 2005 at 09:01:10AM -0800, Seth Alan Woolley wrote:
> > All,
> >
> > Perhaps we should use this to do the gcc splitting. I think it would
> > make more sense than what we have now that doesn't actually work all
> > that well in corner cases.
>
> What doesn't work? The advantage with the current setup is that you
> don't have to build five compilers just because you want to add fortran
> to your existing 4 gcc compilers.

Here's an example other than compiler option changing requiring a
rebuild:

When a spell needs g++ but only works on gcc34, even though gcc 3.4.x is
what's in stable, it will install the null gcc34 spell, assume g++ 3.4.x
is installed, and succinctly fail.

So really there's one case where split spells work for c++ programs:

You never change your archspecs or gcc version between compiling gcc and
g++ and the spell works on gcc4 even if gcc 3.4 is installed. I can
work on an up_trigger to fix the first one if we want to keep them
split. The latter requires separate g++34 spells.

Is CCACHE not good enough to avoid recompiling all of gcc?

I'm just saying either the split spells need extra work or I'm delaying
releasing stable until it's fixed. I'm running into tons of problems
testing spells because I run into these issues whenever I go through a
chroot testing cycle. It's easily repeatable.

If we don't want to fix this issue but want stable released anyways, I'm
putting in a big release note about how you probably want to do: cast -c
g++ first thing after any update of gcc and for people to not expect g++
being split to actually use dependency resolving correctly.

All I'm saying is that it needs some more work that doesn't seem to be a
high priority for developers now. It's still causing problems for
people. And as has been pointed out, I probably shouldn't be putting in
fixes to bugs I find because I'll be the one responsible for testing
them anyways.

We either need them fixed or we need to revert to simpler behavior.

Seth

>
> > On Tue, Dec 06, 2005 at 03:50:01AM +0000, Perforce Review Daemon wrote:
> > > Change 71103 by andrew_stitt@afk-courier on 2005/12/06 03:45:34
> > >
> > > fix bug 6811, add support for sub-depends in sorcery
> > > see http://wiki.sourcemage.org/sub-dependencies for more information
> > >
> > > Affected files ...
> > >
> > > ... //sgl/sorcery/devel/ChangeLog#477 edit
> > > ... //sgl/sorcery/devel/etc/sorcery/state#8 edit
> > > ... //sgl/sorcery/devel/usr/sbin/cast#66 edit
> > > ... //sgl/sorcery/devel/usr/sbin/cleanse#40 edit
> > > ... //sgl/sorcery/devel/var/lib/sorcery/modules/build_api/common#21 edit
> > > ... //sgl/sorcery/devel/var/lib/sorcery/modules/libapi#32 edit
> > > ... //sgl/sorcery/devel/var/lib/sorcery/modules/libdepends#47 edit
> > > ... //sgl/sorcery/devel/var/lib/sorcery/modules/libdispel#17 edit
> > > ... //sgl/sorcery/devel/var/lib/sorcery/modules/libstate#26 edit
> > > ... //sgl/sorcery/devel/var/lib/sorcery/modules/libtablet#18 edit
> > >
> > > Differences ...
> > >
> > > ==== //sgl/sorcery/devel/ChangeLog#477 (text) ====
> > >
> > > @@ -1,3 +1,17 @@
> > > +2005-12-05 Andrew Stitt <astitt@...>
> > > + * Fixes bug 6811 for sub-depends support.
> > > + * etc/sorcery/state: Add line for sub-depends table.
> > > + * cast: load variable with provided sub-depends.
> > > + * cleanse: add code to clean up sub-depends table.
> > > + * build_api/common: Commit provided sub-depends to table.
> > > + * libapi: Add entries for sub_depends and is_depends_enabled functions
> > > + * libdepends: add code to process sub-depends
> > > + * libdispel: remove sub-depends when both spells are removed
> > > + * libstate: Add functions to interface with the sub-depends
> > > + table, also adjusted accessors for regular depends where they
> > > + were insufficient
> > > + * libtablet: Add routines to populate and access sub-depends info.
> > > +
> >
> > --
> > Seth Alan Woolley [seth at positivism.org], SPAM/UCE is unauthorized
> > Quality Assurance Team Leader & Security Team: Source Mage GNU/linux
> > Linux so advanced, it may as well be magic http://www.sourcemage.org
> > Key id 63C1E02F = E07A FB0E 5925 CE4A 6526 2AD5 1782 FEC2 63C1 E02F
>
>
>
> > _______________________________________________
> > SM-Discuss mailing list
> > SM-Discuss AT lists.ibiblio.org
> > http://lists.ibiblio.org/mailman/listinfo/sm-discuss
>
>
> --
> Arwed v. Merkatz Source Mage GNU/Linux
> developer
>
> http://www.sourcemage.org
> _______________________________________________
> SM-Discuss mailing list
> SM-Discuss AT lists.ibiblio.org
> http://lists.ibiblio.org/mailman/listinfo/sm-discuss
>

--
Seth Alan Woolley [seth at positivism.org], SPAM/UCE is unauthorized
Quality Assurance Team Leader & Security Team: Source Mage GNU/linux
Linux so advanced, it may as well be magic http://www.sourcemage.org
Key id 63C1E02F = E07A FB0E 5925 CE4A 6526 2AD5 1782 FEC2 63C1 E02F

Attachment: pgp7WguSxTZ8x.pgp
Description: PGP signature




Archive powered by MHonArc 2.6.24.

Top of Page