Bugs for Sorcery are reported here

[bugzilla-daemon AT bugs.sourcemage.org]

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





------- Additional Comments From baux AT sourcemage.org  2004-12-31 04:02 -------
(In reply to comment #7)
> Magically makes me nervous because i dont want to have to learn how this all
> magically works when it breaks :-/

it isn't really magic, its sufficient to read it bettet :)
in any case, when (and if) it breaks I'll repair it 

> 
> What if the password ends in a / or some other field ends in a / theres 
> still
> too much fuzziness in this for me to be comfortable having to maintain it.

why should you maintain it... I'll maintain it if you don't understand it 

> Perhaps using some other unique seperation character or making all the 
> optional
> fields manditory fields and instead saying that if they are empty, they dont
exist?

unique separator should be compliant to standard uri description (like : or /)
and I use them. The others (not compliant to standard uri) cannot be or sould
not be used.

> 
> I think it would be best to do away with the patch-level field althogether 
> since
> one could just as easily include it as part of the revision (and thats all 
> the
> code does with it anyway). Arch has to parse it anyway so why not let it do 
> the
> hard work rather than us having to do it also. Besides sorcery doesnt really
> care what the patch level is.

not true... if is used to track spell version (have a look to emacs-planner 
spell)

> 
> There appears to be a bug as well: you dont handle the case of unpacking to 
> a
> pre-existing tarball without a patch level. If you remove the patch level
> requirement and fold it into the revision then the problem goes away.

this is a 'filosofical' question... if a spell doens't request for a specific
patch level could mean which want the latest... so get it 

> Re-assigning, please hand back to me when you're done.

here I'm... if you like url_tla as is (and I'll maintain it in future) ok...
if not... make it better becouse as is I think it's ok

thanks 

-- 
Configure bugmail: http://bugs.sourcemage.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.