Skip to Content.
Sympa Menu

sm-grimoire - Re: [SM-Grimoire] Re: PERFORCE change 24280 for review

sm-grimoire AT lists.ibiblio.org

Subject: Discussion of Spells and Grimoire items

List archive

Chronological Thread  
  • From: "Sergey A. Lipnevich" <sergey AT optimaltec.com>
  • To: sm-grimoire AT lists.ibiblio.org
  • Subject: Re: [SM-Grimoire] Re: PERFORCE change 24280 for review
  • Date: Sat, 03 Jan 2004 20:47:15 -0500

Just wanted to note that this is very well said. Thank you Hamish!

Sergey.

Hamish Greig wrote:

that was a seperate issue as I understood it.
the first issue was if the change was as small as this cdrdao spell then it
should be a PREPARE query not a seperate spell that needs extra maintainance,
conflicts, provides and a knowledge of available spells.
the second issue that you are referrring to I understood was whether sorcery
needed to support cvs/stable version dependent spell scripts within the same
spell.
I had already stated I thought any restructuring or addditions in this respect
should wait until post 1.0 and I had also understood there was agreement that
unless the cvs version was warranted due to different DEPENDS, then it should
always be integrated. Therefore I had no reason to respond, nor did anyone
else ?

I think adding PROVIDES, CONFLICTS changing DEPENDS of other spells and
actually adding another spell to the grimoire is the less efficient method.
A query in PREPARE and a
"if ["$CVS" = "YES" ]; then VERSION=FOO-cvs ; else VERSION=FOO; fi"
in DETAILS would definitely be the quickest and simplest.
Especially if it is not a spell you will be maintaining, then it should be an
addition to the original spell. Adding spells means more maintenance, more
CONFLICTS and more PROVIDES. Rewriting spells as was discussed doesn't
introduce any of these overheads.
If I was wrong in my understanding of the previously raped and pillaged thread
(I have just reread it, and all it's child threads) then I would actually
like to readdress the issue( in a single thread ffs).





Archive powered by MHonArc 2.6.24.

Top of Page