Bug 758 – casting a spell does not cast dependencies

Bug 758 - casting a spell does not cast dependencies

Summary:

casting a spell does not cast dependencies

Status:	CLOSED FIXED

Product:	Sorcery
Classification:	Unclassified
Component:	Cast
Version:	pre-1.10
Platform:	Other other

Importance:	P2 normal
Assigned To:	Nick Jennings

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2002-08-25 17:15 UTC by Jason Flatt
Modified:	2003-02-04 14:15 UTC (History)
CC List:	4 users (show)

See Also:

Attachments
Patch as applied to cvs. (566 bytes, patch) 2002-09-02 00:46 UTC, Nathan Doss	Details \| Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Jason Flatt 2002-08-25 17:15:50 UTC

When casting a spell, I am prompted to install dependencies and optional 
dependencies, however the dependencies are not summoned, nor are they cast.

Comment 1 Dufflebunk 2002-08-25 17:58:11 UTC

Seems to be fixed in latest CVS.

Comment 2 Rasmus Ory Nielsen 2002-08-28 07:04:51 UTC

I just tried to 'dispel nasm' and after that 'cast -r -c lilo'.
Nasm was not installed again, so the compile failed :-(

BTW, devel grimoire and devel sorcery.

Comment 3 Ryan Abrams 2002-09-01 02:00:34 UTC

not running devel sorcery atm, so i cant say for sure, but in 0.8.0.1, gnome-2-desktop-
profile runs, and cast's NOTHING. It says the queue is empty (which is a pretty big hint as 
to the problem)

Will try again with devel

Comment 4 Ryan Abrams 2002-09-01 02:47:41 UTC

Just pulled from cvs and did it again. BUSTED. try gnome-2-desktop-profile. It. does. not. 
work.

Big bad bug. Please fix. Please?

Comment 5 Chris Brien 2002-09-01 21:47:12 UTC

*** Bug 796 has been marked as a duplicate of this bug. ***

Comment 6 Nathan Doss 2002-09-02 00:46:53 UTC

Created attachment 233 [details]
Patch as applied to cvs.

This patch fixes part of the dependencies problems.  There appear to be other
problems.

Comment 7 Nathan Doss 2002-09-02 03:01:29 UTC

I've checked in some changes for some other bugs that I believe fix this bug 
as well.  I'll wait for someone to verify that everthing works before 
resolving this as fixed.

Comment 8 Jon Svendsen 2002-09-02 14:27:47 UTC

I'm gonna try to do some testing on this later tonight.

Btw, I really think that the resolution of this bug, as well as bug 765,
calls for another 0.8.0.x release.

Comment 9 Nick Jennings 2002-09-02 14:31:38 UTC

I've said no more 0.8.0.x releases. This is an unstable branch of sorcery, and
it was made clear from the begining. 

Besides 0.8.1 is scheduled for sept. 10th I'm sure people can wait until then if
they are unwilling to track devel.

Comment 10 Jon Svendsen 2002-09-02 14:38:22 UTC

I can live with that, but I still think it seems a little strange to (willingly)
leave the version of sorcery we ARE, after all, marketing as "the one you choose
if you want to be on the safe side" completely and utterly broken for well over
a week.

Then again, people can use the testing/stable grimoires and get sorcery 0.1.x
(can't they?), so it would appear that I am on drugs. ;)

Comment 11 Jon Svendsen 2002-09-02 19:07:41 UTC

The patch does work, reintroducing both fixed and optional dependencies. wohoo!
resolving should be ok.

It hasn't made it into -devel.tar.bz2 yet though. How often is that pulled?

Comment 12 Nathan Doss 2002-09-02 20:33:21 UTC

Looking at the ftp site, the files have a timestamp of midnight.  Since the 
server's in North Carolina/USA, I think the tarball's pulled at GMT-5?

Comment 13 Nathan Doss 2002-09-02 21:56:25 UTC

  
Based on various irc comments, it appears this bug is now fixed in cvs.

Comment 14 erics 2003-02-04 12:51:21 UTC

Bugzilla cleanup for sorcery...

Comment 15 erics 2003-02-04 14:15:12 UTC

Closing out sorcery bugs...