sm-sorcery-bugs AT lists.ibiblio.org
Subject: Bugs for Sorcery are reported here
List archive
[SM-Sorcery-Bugs] [Bug 6841] Held spells do not register as installed in sorcery . . .
- 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.
-
[SM-Sorcery-Bugs] [Bug 6841] Held spells do not register as installed in sorcery . . .,
bugzilla-daemon, 05/08/2004
- <Possible follow-up(s)>
- [SM-Sorcery-Bugs] [Bug 6841] Held spells do not register as installed in sorcery . . ., bugzilla-daemon, 05/08/2004
- [SM-Sorcery-Bugs] [Bug 6841] Held spells do not register as installed in sorcery . . ., bugzilla-daemon, 05/08/2004
- [SM-Sorcery-Bugs] [Bug 6841] Held spells do not register as installed in sorcery . . ., bugzilla-daemon, 05/10/2004
Archive powered by MHonArc 2.6.24.