In fact, given that this revision was broken (hopefully r28077 is more
correct), yeah, tests would be nice :-)
--dave
On Nov 27, 2007 10:09 AM, David Glasser <glasser@davidglasser.net> wrote:
> I'd love to see a test for the problem that this revision fixes, but
> I'm not familiar enough with svn log -g (the only command that tests a
> codepath with LIKE) to reproduce it myself (let alone write a
> programmatic test). Anybody want to give it a shot?
>
> --dave
>
> On Nov 27, 2007 10:05 AM, <glasser@tigris.org> wrote:
> > Author: glasser
> > Date: Tue Nov 27 10:05:54 2007
> > New Revision: 28073
> >
> > Log:
> > We make at least one LIKE query against paths, to find paths that are
> > children of a given path. This really really really needs to be
> > case-sensitive!
> >
> > * subversion/libsvn_fs_util/sqlite-util.c
> > (schema_create_sql): Make all LIKEs case-sensitive when the database
> > is created.
> >
> >
> > Modified:
> > trunk/subversion/libsvn_fs_util/sqlite-util.c
> >
> > Modified: trunk/subversion/libsvn_fs_util/sqlite-util.c
> > URL: http://svn.collab.net/viewvc/svn/trunk/subversion/libsvn_fs_util/sqlite-util.c?pathrev=28073&r1=28072&r2=28073
> > ==============================================================================
> > --- trunk/subversion/libsvn_fs_util/sqlite-util.c (original)
> > +++ trunk/subversion/libsvn_fs_util/sqlite-util.c Tue Nov 27 10:05:54 2007
> > @@ -93,6 +93,8 @@
> > /* USER_VERSION 1 */
> > "PRAGMA auto_vacuum = 1;"
> > APR_EOL_STR
> > + "PRAGMA case_sensitive_like=on;"
> > + APR_EOL_STR
> > "CREATE TABLE mergeinfo (revision INTEGER NOT NULL, mergedfrom TEXT NOT "
> > "NULL, mergedto TEXT NOT NULL, mergedrevstart INTEGER NOT NULL, "
> > "mergedrevend INTEGER NOT NULL, inheritable INTEGER NOT NULL);"
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: svn-unsubscribe@subversion.tigris.org
> > For additional commands, e-mail: svn-help@subversion.tigris.org
> >
> >
>
>
>
> --
> David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
>
--
David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Nov 27 19:30:23 2007