Skip to Content.
Sympa Menu

sm-discuss - Re: [SM-Discuss] python 2.5 upgrade

sm-discuss AT lists.ibiblio.org

Subject: Public SourceMage Discussion List

List archive

Chronological Thread  
  • From: Flavien Bridault <vlaaad AT sourcemage.org>
  • To: sm-discuss AT lists.ibiblio.org
  • Subject: Re: [SM-Discuss] python 2.5 upgrade
  • Date: Wed, 08 Nov 2006 22:50:55 +0100

Le mercredi 08 novembre 2006 à 16:01 +0100, Jaka Kranjc a écrit :
> On Wednesday 08 November 2006 14:59, Flavien Bridault wrote:
> > Ok it seems my methode catches everything. So before committing the
> > stuff, I would like to be sure that I can remove all the TRIGGERS that
> > contains "on_cast python cast_self" ? The UP_TRIGGERS research does find
> > these spells, but I just wanted to know if those TRIGGERS were only
> > intended for minor updates (2.4->2.5) or was it also for maintenance
> > releases (2.5->2.5.1) or anything else ?
> I wouldn't remove them, they will always trigger, not just now. UP_TRIGGERS
> would catch them only in this version-change case. Unless you made it more
> general, making more work for yourself. The stuff there now works, albeit
> all
> the if checks you posted should be changed to use is_depends_enabled.

Well after discussion on IRC, it appears that no one really knows why
those TRIGGERS are here, except that *sometimes* they are useful. I have
the feeling that they have been added to handle minor versions upgrade,
as I am trying to do here. So in that case we have a duplication.
However the UP_TRIGGERS is more efficient as it manages to trigger all
needed spells and it explicits the case where this is needed, so here
the minor version upgrade.

To verify this, I :
- removed all TRIGGERS on_cast python in the grimoire,
- downgraded python to 2.4.3, and so triggered all python dependent
spells with the UP_TRIGGERS,
- verified that most of the previous spells work,
- upgraded python to 2.4.4, so in that case the UP_TRIGGERS did
nothing,
- verified that most of the previous spells still work, and they do,
- upgraded python 2.5, and so triggered all python dependent spells
with the UP_TRIGGERS,
- verified that most of the previous spells still work, and they do,

So my proposal would be to remove all these TRIGGERS and to replace them
by a single UP_TRIGGERS in the python spell, unless someone gives me a
good reason. ;-) For instance if the TRIGGERS were there for an another
purpose. I will wait until tomorrow evening before applying this.
--
Flavien Bridault

Disk, Graphics, Graphics-libs sections Guru
Source Mage GNU/Linux - http://www.sourcemage.org

irc: vlaaad
jabber: vlaaad AT amessage.be

Attachment: signature.asc
Description: Ceci est une partie de message numériquement signée




Archive powered by MHonArc 2.6.24.

Top of Page