Skip to Content.
Sympa Menu

sm-sorcery-bugs - [SM-Sorcery-Bugs] [Bug 6841] Held spells do not register as installed in sorcery . . .

sm-sorcery-bugs AT lists.ibiblio.org

Subject: Bugs for Sorcery are reported here

List archive

Chronological Thread  
  • From: bugzilla-daemon AT metalab.unc.edu
  • To: sm-sorcery-bugs AT lists.ibiblio.org
  • Subject: [SM-Sorcery-Bugs] [Bug 6841] Held spells do not register as installed in sorcery . . .
  • Date: Sat, 8 May 2004 10:19:10 -0400

http://bugs.sourcemage.org/show_bug.cgi?id=6841





------- Additional Comments From vlmarek AT volny.cz 2004-05-08 10:19 -------
<neuron> Oadae: the bug is in /var/lib/sorcery/modules/libstate, line 252 :)
...

[15:36] <alley_cat> seems to be by design, but ask andrew about it
[15:36] <alley_cat> you can use if spell_installed gcc || spell_held gcc
[15:36] <neuron> It should be more bit array oriented maybe
[15:38] lejatorn [~lejatorn AT regulus3.student.uu.se] has left #smgl-dump []
[15:43] <Oadae> alley_cat, most likely every spell in the grimoires that use
spell_installed should be modified to use spell_held.
[15:43] <Oadae> jason@jason:~$ grep -r 'spell\_installed' /devel/* | wc -l
[15:43] <Oadae> 250
[15:43] <Oadae> jason@jason:~$ grep -r 'spell\_installed' /devel/* | grep
'spell\_held' | wc -l
[15:43] <Oadae> 0
[15:43] <neuron> Fix real_spell_installed function instead seems to be much
more
easier to me
[15:44] <neuron> And in future authors don't have to remember to use
spell_held also
[15:44] <neuron> And it's faster too :)
[15:44] <alley_cat> yep
[15:45] <Oadae> I agree.
[15:45] <alley_cat> just wondering if somewhere sorcery uses
real_spell_installed itself and assumes it returns false for held spells ...
[15:46] <neuron> no
[15:47] <Oadae> Doesn't look like it's used at all.
[15:47] <alley_cat> it does use spell_installed || spell_held in some places
[15:48] <alley_cat> but in other places it clearly uses spell_installed in a
way
that'll break with the current one
[15:48] <neuron> Moreover, spell cannot be held without being installed :)
[15:48] <neuron> Or can ?
[15:48] <alley_cat> [ "$CCACHE" == "on" ] && spell_installed ccache
[15:48] <alley_cat> so people with ccache held won't ever have it used :P
[15:49] <neuron> sssssh :)
[15:49] <alley_cat> same for distcc and netselect
[15:50] <alley_cat> that stuff definitely needs a cleanup, with
spell_installed
being true for both installed and held
[15:50] <alley_cat> as held implies installed




------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.




Archive powered by MHonArc 2.6.24.

Top of Page