-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Julian Foad wrote:
[..]
> I looked for assignments to the "activity_coll" field, and found it in
> the function end_element(), where it copies it from the "cdata" field. I
> think that is where the canonicalization should be done.
My bad, misinterpreted `end_element()' as neon's method!
> And here:
>
>> * props.c
>> (svn_ra_neon__get_baseline_info, svn_ra_neon__get_one_prop): Same.
>
> The function ..._get_baseline_info() gets its bc_url from a hash created
> by the function svn_ra_neon__get_baseline_props(), so I think that's
> where the change needs to be.
>
> The function "get_one_prop" is not specifically for getting a baseline
> URL, it is a generic function for getting any property, so I think you
> should not be changing that function.
Tracing these back further, the value gets set in `end_element()'. This
fixes one more instance of non-canonical URL in `get_version_url()',
thus fixing all instances of non-canonical URLs AFAICS. Thank you for
the feedback, Julian. Attaching the updated patch herewith.
[[[
Log:
Ensure the URLs are always canonical.
[ in subversion/libsvn_ra_neon ]
* util.c
(svn_ra_neon__request_get_location): Canonicalize the 'BASE URL' as
per the rule.
* props.c
(end_element): Same.
* options.c
(end_element): Same.
Found by: stsp
Suggested by: stsp, julianfoad
Patch by: Kannan R <kannanr_at_collab.net>
]]]
If the above patch seems fine, the patch to upgrade ..add_component() to
..add_component2() can be found in [1].
[1]-http://mail-archives.apache.org/mod_mbox/subversion-dev/200912.mbox/%3C4B2B12EA.5000208@collab.net%3E
- --
Thanks & Regards,
Kannan
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQEVAwUBS0XWUHlTqcY7ytmIAQIRnwf+LBmMhAKRPwCbWO2tmtzkhuQ9Bx7P1wlK
lGFhyhUf82InW6uEYqMP9UaihxPBeSDr5Cqec92HN8xV/O1j0366pBdSJ/1sXvrn
hg8U3toh9GSbIm5FqQ9WDWgOjYPG0VSr9pz/rFi+6fL9DkrSWqfcfHLZOJ4YIYQW
jPpLhRFVuypQDxpeoOpSuh3PHPg+zovaEsGCNepfqpwiVITvK3B74r8zq/sYd6SL
MF56QKDjgfFJAbD8W9SlA+y6aYJQm1ZxUMV9B2FdBSx8hv+i7SM3U6UTinQizUs8
8zutd2q7i7TA1jyqyZGRKx+4pProUbeIO1mUsRJrpwjaYlzssjRzRA==
=8XvR
-----END PGP SIGNATURE-----
Received on 2010-01-07 13:47:45 CET