You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Karl Fogel <kf...@newton.ch.collab.net> on 2002/09/19 17:45:40 UTC

Re: (possible) bug report.

Can you give the exact recipe we need to reproduce this bug?
(See the HACKING file for bug reporting guidelines.)

Thanks -- if this is a corruption bug, it's very important that we
track it down.

-Karl


Leandro Lucarella <ll...@mecon.gov.ar> writes:
> Hi! I'm using svn and I have a problem with one particular file and two
> particular lines on it.
> The lines are:
> $selected   = '<a href="%s/%s/" onmouseout="MM_swapImgRestore()" onmouseover="MM_displayStatusMsg(\'HORAS EXTRAS\'); MM_swapImage(\'he%s\',\'\',\'%s/imagenes/he%s_f4.gif\',1); return document.MM_returnValue"><img name="he%s" src="%s/imagenes/he%s_f3.gif" border="0" alt="%s"></a>';
> $unselected = '<a href="%s/%s/" onmouseout="MM_swapImgRestore()" onmouseover="MM_displayStatusMsg(\'HORAS EXTRAS\'); MM_swapImage(\'he%s\',\'\',\'%s/imagenes/he%s_f2.gif\',1); return document.MM_returnValue" onclick="MM_swapImage(\'he%s\',\'\',\'%s/imagenes/he%s_f3.gif\',1)"><img name="he%s" src="%s/imagenes/he%s.gif" border="0" alt="%s"></a>';
> 
> When I upload this two lines in the file, svn 'convert' them to this:
> border="0" alt="%s"></a>';
> es/he%s_f3.gif\',1)"><img name="he%s" src="%s/imagenes/he%s.gif" border="0" alt="%s"></a>';
> 
> I'm using keyword expansion ("URL Author Date Rev"). svn version is a
> snapshot from Debian GNU/Linux unstable (subversion-snapshot):
> Subversion Client, version 0.14.0 (dev build)
>    compiled Aug 16 2002, 14:35:50
> 
> I'll attach the original file and the checked out so you can see them...
> 
> PS: Please Cc me, I'm not on the list.
> 
> -- 
> Leandro Lucarella @ MEcon                       .-----------------------------.
>                                                /  Powered by Debian GNU/Linux |
> .-----------------------------------------------------------------------------|
> |  GPG Key:               http://bal748.mecon.ar/~luca/public_key.gpg         |
> |  GPG Key Fingerprint:   716E 4896 93B2 640D 917D  96D8 4B16 3177 C864 D90B  |
> `-----------------------------------------------------------------------------'
> Donde las dan las toman, y callar es bueno.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: (possible) bug report.

Posted by Karl Fogel <kf...@newton.ch.collab.net>.
Philip Martin <ph...@codematters.co.uk> writes:
> I've done it.

Wow!  Nice job.

>  1. create repository
>  2. checkout wc
>  3. svn add original file
>  4. svn ps svn:keywords

Hmmm.  What effect is step 4 supposed to have -- does it set  the
property to the empty string, or unset it, or...?  (Overall, this is a
bug in any case, but not sure what this step means precisely.)

>  5. svn ci
>  6. rm wc
>  7. svn co
>  8. compare original file to wc version
> 
> They don't match :-(

Zounds!

I've filed new issue 

   http://subversion.tigris.org/issues/show_bug.cgi?id=903

for this; we'll get it fixed before the upcoming 0.14.3 interim
release.

(Philip, Ben just told me you've posted a new recipe now; I'll make
sure that gets into the issue after I've sent this mail :-) ).

-K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: (possible) bug report.

Posted by Philip Martin <ph...@codematters.co.uk>.
Karl Fogel <kf...@newton.ch.collab.net> writes:

> Can you give the exact recipe we need to reproduce this bug?
> (See the HACKING file for bug reporting guidelines.)
> 
> Thanks -- if this is a corruption bug, it's very important that we
> track it down.

I've done it.

 1. create repository
 2. checkout wc
 3. svn add original file
 4. svn ps svn:keywords
 5. svn ci
 6. rm wc
 7. svn co
 8. compare original file to wc version

They don't match :-(

-- 
Philip Martin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: (possible) bug report.

Posted by Karl Fogel <kf...@newton.ch.collab.net>.
Leandro Lucarella <ll...@mecon.gov.ar> writes:
> > It's resolved now; see issue #903 for details.
> 
> Nice, really fast!!!! 8)

:-) Let me put it this way:

If you so much as *think* the words "data corruption" anywhere near
this list, people will stop whatever they're doing and start
debugging.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: (possible) bug report.

Posted by Leandro Lucarella <ll...@mecon.gov.ar>.
Karl Fogel, el jueves 19 de septiembre a las 14:03 me escribiste:
> Leandro Lucarella <ll...@mecon.gov.ar> writes:
> > Wow! This is new... now I can't even commit the changes because of a
> > segmentation fault... (I never had a segmentation fault before).
> 
> Yup -- that's consistent with what Mike found to be the cause of the bug.
> 
> It's resolved now; see issue #903 for details.

Nice, really fast!!!! 8)

> Thanks for the report!

Glad to help :)

-- 
Leandro Lucarella @ MEcon                       .-----------------------------.
                                               /  Powered by Debian GNU/Linux |
.-----------------------------------------------------------------------------|
|  GPG Key:               http://bal748.mecon.ar/~luca/public_key.gpg         |
|  GPG Key Fingerprint:   716E 4896 93B2 640D 917D  96D8 4B16 3177 C864 D90B  |
`-----------------------------------------------------------------------------'
Proteger el gusto es matar el arte.
		-- Anónimo. 

Re: (possible) bug report.

Posted by Karl Fogel <kf...@newton.ch.collab.net>.
Leandro Lucarella <ll...@mecon.gov.ar> writes:
> Wow! This is new... now I can't even commit the changes because of a
> segmentation fault... (I never had a segmentation fault before).

Yup -- that's consistent with what Mike found to be the cause of the bug.

It's resolved now; see issue #903 for details.

Thanks for the report!

-K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: (possible) bug report.

Posted by Leandro Lucarella <ll...@mecon.gov.ar>.
Karl Fogel, el jueves 19 de septiembre a las 13:14 me escribiste:
> Leandro Lucarella <ll...@mecon.gov.ar> writes:
> > recipe is the repository?
> 
> No -- rather, it's a series of steps to take to make the bug happen.
> 
> > I've taked a quick look the file and just saw instructions to send a
> > patch, but nothing about bug reporting...
> > Can you tell me in a few simple steps what do I have to do to give you
> > the necesary information to track the bug?
> 
> I'm sorry -- I meant the file BUGS, not HACKING.
> 
> Looks like Philip Martin has reproduced it, though, so no need now.
> (Does his recipe match yours?)

Arrhhh, I've just tried to make the simplest test:
luca@bal748h:~$ mkdir test
luca@bal748h:~$ svnadmin create test.svn
luca@bal748h:~$ cd test
luca@bal748h:~/test$ cp ../file.php .
luca@bal748h:~/test$ cat file.php
$selected = '<a href="%s/%s/" onmouseout="MM_swapImgRestore()" onmouseover="MM_displayStatusMsg(\'HORAS EXTRAS\'); MM_swapImage(\'he%s\',\'\',\'%s/imagenes/he%s_f4.gif\',1); return document.MM_returnValue"><img name="he%s" src="%s/imagenes/he%s_f3.gif" border="0" alt="%s"></a>';
luca@bal748h:~/test$ cd ..
luca@bal748h:~$ svn import file:///home/luca/test.svn test
Adding         test/file.php
Transmitting file data .
Committed revision 1.
svn co file:///home/luca/test.svn/ test2
luca@bal748h:~$ cd test2
luca@bal748h:~/test2$ cat file.php 
$selected = '<a href="%s/%s/" onmouseout="MM_swapImgRestore()" onmouseover="MM_displayStatusMsg(\'HORAS EXTRAS\'); MM_swapImage(\'he%s\',\'\',\'%s/imagenes/he%s_f4.gif\',1); return document.MM_returnValue"><img name="he%s" src="%s/imagenes/he%s_f3.gif" border="0" alt="%s"></a>';
<<<<<<<<<<<<<< for now is working great, here's come the ugly part >>>>>>>>>>>>>>>>>>>
luca@bal748h:~/test2$ svn ps svn:keywords "URL Author Date Rev" file.php 
property `svn:keywords' set on 'file.php'
luca@bal748h:~/test2$ svn ci
Segmentation fault

Wow! This is new... now I can't even commit the changes because of a
segmentation fault... (I never had a segmentation fault before).

luca@bal748h:~/test2$ vi file.php
luca@bal748h:~/test2$ cat file.php 
$selected =
'<a href="%s/%s/" onmouseout="MM_swapImgRestore()" onmouseover="MM_displayStatusMsg(\'HORAS EXTRAS\'); MM_swapImage(\'he%s\',\'\',\'%s/imagenes/he%s_f4.gif\',1); return document.MM_returnValue"><img name="he%s" src="%s/imagenes/he%s_f3.gif" border="0" alt="%s"></a>';
<<<<<<<<<<<<<<<<<<< Added an extra \n before '$selected =' >>>>>>>>>>>>>>>>>>>>>>>>>>>>
luca@bal748h:~/test2$ svn ci       
Sending        file.php
Transmitting file data .
Committed revision 2.
luca@bal748h:~/test2$ cat file.php 
$selected =
'<a href="%s/%s/" onmouseout="MM_swapImgRestore()" onmouseover="MM_displayStatusMsg(\'HORAS EXTRAS\'); MM_swapImage(\'he%s\',\'\',\'%s/imagenes/he%s_f4.gif\',1); return document.MM_returnValue"><img name="he%s" src="%s/imagenes/he%s_f3.gif" border="0" alt="%s"></a>';

It worked.

-- 
Leandro Lucarella @ MEcon                       .-----------------------------.
                                               /  Powered by Debian GNU/Linux |
.-----------------------------------------------------------------------------|
|  GPG Key:               http://bal748.mecon.ar/~luca/public_key.gpg         |
|  GPG Key Fingerprint:   716E 4896 93B2 640D 917D  96D8 4B16 3177 C864 D90B  |
`-----------------------------------------------------------------------------'
Juventud, ¿sabes que la tuya no es la primera generación que anhela una vida
plena de belleza y libertad?
		-- Albert Einstein. (1879-1955) Físico alemán. 

Re: (possible) bug report.

Posted by Karl Fogel <kf...@newton.ch.collab.net>.
Leandro Lucarella <ll...@mecon.gov.ar> writes:
> recipe is the repository?

No -- rather, it's a series of steps to take to make the bug happen.

> I've taked a quick look the file and just saw instructions to send a
> patch, but nothing about bug reporting...
> Can you tell me in a few simple steps what do I have to do to give you
> the necesary information to track the bug?

I'm sorry -- I meant the file BUGS, not HACKING.

Looks like Philip Martin has reproduced it, though, so no need now.
(Does his recipe match yours?)

> I hope it helps... It's tiny little way to say thanks for such a
> wonderful CVS replacement :)

It will be more wonderful when it doesn't have this bug :-).

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: (possible) bug report.

Posted by Leandro Lucarella <ll...@mecon.gov.ar>.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Karl Fogel, el jueves 19 de septiembre a las 12:45 me escribiste:
> Can you give the exact recipe we need to reproduce this bug?

recipe is the repository?

> (See the HACKING file for bug reporting guidelines.)

I've taked a quick look the file and just saw instructions to send a
patch, but nothing about bug reporting...
Can you tell me in a few simple steps what do I have to do to give you
the necesary information to track the bug?

> Thanks -- if this is a corruption bug, it's very important that we
> track it down.

I hope it helps... It's tiny little way to say thanks for such a
wonderful CVS replacement :)

- -- 
Leandro Lucarella @ MEcon                       .-----------------------------.
                                               /  Powered by Debian GNU/Linux |
.-----------------------------------------------------------------------------|
|  GPG Key:               http://bal748.mecon.ar/~luca/public_key.gpg         |
|  GPG Key Fingerprint:   716E 4896 93B2 640D 917D  96D8 4B16 3177 C864 D90B  |
`-----------------------------------------------------------------------------'
Abre la boca que te va la sopa.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)

iD8DBQE9ihb97T/vBXlJp7MRAjQCAJ92opl2h2XCCkfqJkyG7tMWV3zpGACdEPCM
/i6QWIebOD4CQ/2D+fB5LoY=
=gFqb
-----END PGP SIGNATURE-----

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org