You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Martin Furter <mf...@rola.ch> on 2009/06/17 22:26:30 UTC
[PATCH] Add URL to an error text in ra_neon
Hello
A friend got a strange error message while merging:
svn: 'http://subversion.tigris.org/xmlns/dav/md5-checksum' was not present
on the resource
Looking at the code I believe it didn't find the propert "md5-checksum" on
a file. The following untested patch adds the URL to the error text.
[[[
Add the URL to an error text.
* subversion/libsvn_ra_neon/props.c
(svn_ra_neon__get_one_prop) Add URL to the error text.
]]]
Index: subversion/libsvn_ra_neon/props.c
===================================================================
--- subversion/libsvn_ra_neon/props.c (revision 37338)
+++ subversion/libsvn_ra_neon/props.c (working copy)
@@ -625,8 +625,8 @@
{
/* ### need an SVN_ERR here */
return svn_error_createf(SVN_ERR_FS_NOT_FOUND, NULL,
- _("'%s' was not present on the resource"),
- name);
+ _("'%s' was not present on the resource '%s'"),
+ name, url);
}
*propval = value;
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2363009
Re: [PATCH] Add URL to an error text in ra_neon
Posted by Gavin Baumanis <ga...@thespidernet.com>.
On 21/06/2009, at 10:00 AM, Arfrever Frehtes Taifersar Arahesis wrote:
> 2009-06-18 00:26:30 Martin Furter napisał(a):
>> <snip>
>>
>> [[[
>> Add the URL to an error text.
>>
>> * subversion/libsvn_ra_neon/props.c
>> (svn_ra_neon__get_one_prop) Add URL to the error text.
>> ]]]
>>
>>
>
> Committed in r38126.
Thanks Arfrver,
I'll remove this one from the Watch-list too.
Gavin.
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2365114
Re: [PATCH] Add URL to an error text in ra_neon
Posted by Arfrever Frehtes Taifersar Arahesis <Ar...@GMail.Com>.
2009-06-18 00:26:30 Martin Furter napisał(a):
> Hello
>
> A friend got a strange error message while merging:
>
> svn: 'http://subversion.tigris.org/xmlns/dav/md5-checksum' was not present
> on the resource
>
> Looking at the code I believe it didn't find the propert "md5-checksum" on
> a file. The following untested patch adds the URL to the error text.
>
>
> [[[
> Add the URL to an error text.
>
> * subversion/libsvn_ra_neon/props.c
> (svn_ra_neon__get_one_prop) Add URL to the error text.
> ]]]
>
>
> Index: subversion/libsvn_ra_neon/props.c
> ===================================================================
> --- subversion/libsvn_ra_neon/props.c (revision 37338)
> +++ subversion/libsvn_ra_neon/props.c (working copy)
> @@ -625,8 +625,8 @@
> {
> /* ### need an SVN_ERR here */
> return svn_error_createf(SVN_ERR_FS_NOT_FOUND, NULL,
> - _("'%s' was not present on the resource"),
> - name);
> + _("'%s' was not present on the resource '%s'"),
> + name, url);
> }
>
> *propval = value;
Committed in r38126.
--
Arfrever Frehtes Taifersar Arahesis