Bugs for Sorcery are reported here

[bugzilla-daemon AT bugs.sourcemage.org]

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


Fredrik Carlström <fredrik.carlstrom AT tele2.se> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |fredrik.carlstrom AT tele2.se




--- Comment #7 from Fredrik Carlström <fredrik.carlstrom AT tele2.se>  
2008-11-06 12:45:46 ---
> The bigger problem with the function is that it doesn't compare
> patchlevels

God point.

I'm not certain what you mean by that it is a problem that it only looks at 
one
cache. As long as there is just one saved cache per version, I don't see any
reason to check another.

It has happened for me personally that I've edited the files in 
$DEPENDS_CONFIG
directly. In these cases this check makes sense. Of course there is the
possibility to say "do never cast without -c or -r", but imo it's a bug when
user's choices are ignored and not only in case of an conflict.

If the uncommitted requested subdependencies can be just a subset of the
provided ones without any changes in the configuration, then I agree. 
Otherwise
I think the default should be to not resurrect unless all settings agree, at
least not without asking the user. This does also apply to CFLAGS and other
options.

I wasn't aware of the function mk_tmp_dirs. Thanks for pointing it out :).


-- 
Configure bugmail: http://bugs.sourcemage.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.