Re: Assertion in svn_uri_is_canonical

2013-08-22 Thread Branko Čibej
On 23.08.2013 06:51, Branko Čibej wrote: > On 23.08.2013 01:28, Branko Čibej wrote: >> On 23.08.2013 01:04, Philip Martin wrote: >>> Branko Čibej writes: >>> On 22.08.2013 22:40, Branko Čibej wrote: > On 22.08.2013 18:11, kmra...@rockwellcollins.com wrote: >> Passing an invalid URL to

Re: Assertion in svn_uri_is_canonical

2013-08-22 Thread Branko Čibej
On 23.08.2013 01:28, Branko Čibej wrote: > On 23.08.2013 01:04, Philip Martin wrote: >> Branko Čibej writes: >> >>> On 22.08.2013 22:40, Branko Čibej wrote: On 22.08.2013 18:11, kmra...@rockwellcollins.com wrote: > Passing an invalid URL to svn co causes an abort and core dump. This

Re: Assertion in svn_uri_is_canonical

2013-08-22 Thread Branko Čibej
On 23.08.2013 01:04, Philip Martin wrote: > Branko Čibej writes: > >> On 22.08.2013 22:40, Branko Čibej wrote: >>> On 22.08.2013 18:11, kmra...@rockwellcollins.com wrote: Passing an invalid URL to svn co causes an abort and core dump. This fails with all protocols http, svn, file.

Log-addressing branch ready for review

2013-08-22 Thread Stefan Fuhrmann
Hi all, As of r1516665, work on this branch has been completed. Please review. See the BRANCH-README for the list of major changes. If there are no objections, I will merge the code in the week of Sep 23th. -- Stefan^2.

Re: Assertion in svn_uri_is_canonical

2013-08-22 Thread Philip Martin
Philip Martin writes: > What is the canonical form of this? > > scheme://./ > > Should we drop '.' to give: > > scheme:/// > > or do we have to retain it as > > scheme://./ > > and change svn_uri_is_canonical to allow a hostname '.'? Another problem is scheme://[.] which we

Re: Assertion in svn_uri_is_canonical

2013-08-22 Thread Philip Martin
Branko Čibej writes: > On 22.08.2013 22:40, Branko Čibej wrote: >> On 22.08.2013 18:11, kmra...@rockwellcollins.com wrote: >>> Passing an invalid URL to svn co causes an abort and core dump. This >>> fails with all protocols >>> http, svn, file. It occurs with all versions I tested (1.7.5, 1.7.

1.8.3 up for testing/signing

2013-08-22 Thread Ben Reser
The 1.8.3 release artifacts are now available for testing/signing. Please get the tarballs from https://dist.apache.org/repos/dist/dev/subversion and add your signatures there. I plan to try and release on August 29th so please try and get your votes/signatures in place by August 27th. Thanks!

Re: Error compiling with serf 1.3.1 on Windows

2013-08-22 Thread Johan Corveleyn
On Tue, Aug 20, 2013 at 1:07 AM, Johan Corveleyn wrote: > On Mon, Aug 19, 2013 at 11:03 PM, Greg Stein wrote: >> On Mon, Aug 19, 2013 at 3:39 PM, Johan Corveleyn wrote: >>> When I try to build trunk (@now) with serf 1.3.1 on Win XP (with VS >>> 2010), I get the following error: >>> >>> [[[ >>> N

1.7.13 up for testing/signing

2013-08-22 Thread Ben Reser
The 1.7.13 release artifacts are now available for testing/signing. Please get the tarballs from https://dist.apache.org/repos/dist/dev/subversion and add your signatures there. I plan to try and release on August 29th so please try and get your votes/signatures in place by August 27th. Thanks!

Re: Assertion in svn_uri_is_canonical

2013-08-22 Thread Branko Čibej
On 22.08.2013 22:40, Branko Čibej wrote: > On 22.08.2013 18:11, kmra...@rockwellcollins.com wrote: >> Passing an invalid URL to svn co causes an abort and core dump. This >> fails with all protocols >> http, svn, file. It occurs with all versions I tested (1.7.5, 1.7.11, >> 1.7.12, 1.8.1, and the

Re: Assertion in svn_uri_is_canonical

2013-08-22 Thread Branko Čibej
On 22.08.2013 18:11, kmra...@rockwellcollins.com wrote: > Passing an invalid URL to svn co causes an abort and core dump. This > fails with all protocols > http, svn, file. It occurs with all versions I tested (1.7.5, 1.7.11, > 1.7.12, 1.8.1, and the now defunct 1.8.2) > It occurs with multiple s

Re: [PATCH] add 'svn:use-commit-times' property

2013-08-22 Thread Daniel Shahaf
Masaru Tsuchiyama wrote on Thu, Aug 22, 2013 at 23:20:14 +0900: > Could you comment the updated patch? As I said, I review the style but not the actual feature. Other devs are far more familiar than I am with the client layer so I prefer to let someone else do the review. Indeed Philip has now c

Re: [PATCH] add command line options to mailer.py

2013-08-22 Thread Daniel Shahaf
Masaru Tsuchiyama wrote on Thu, Aug 22, 2013 at 23:10:01 +0900: > Hello > > Does anybody comment my patch? > FWIW: Saw the patch, it seems like it might be a good idea, but haven't had time to look into it in more detail.

Assertion in svn_uri_is_canonical

2013-08-22 Thread kmradke
Passing an invalid URL to svn co causes an abort and core dump. This fails with all protocols http, svn, file. It occurs with all versions I tested (1.7.5, 1.7.11, 1.7.12, 1.8.1, and the now defunct 1.8.2) It occurs with multiple subcommand (ls, info, etc.) It happens on both unix and windows

Re: [PATCH] add 'svn:use-commit-times' property

2013-08-22 Thread Masaru Tsuchiyama
Philip Martin wrote: masaru tsuchiyama writes: + if (!fb->adding_file) +{ + if (fb->use_commit_times && fb->changed_date) +{ + SVN_ERR(svn_io_set_file_affected_time(fb->changed_date, +fb->local_abspath, +

Re: [PATCH] print debug information when subprocess.Popen subprocess.Popen

2013-08-22 Thread Daniel Shahaf
On Thu, Aug 22, 2013 at 11:22:59PM +0900, Masaru Tsuchiyama wrote: > Daniel Shahaf wrote: > >On Sat, Aug 03, 2013 at 03:52:49PM +0300, Daniel Shahaf wrote: > >>Masaru Tsuchiyama wrote on Sat, Aug 03, 2013 at 16:06:06 +0900: > >>>If I run the these scripts at Release\subversion\tests\cmdline, I don'

Re: [PATCH] add 'svn:use-commit-times' property

2013-08-22 Thread Philip Martin
masaru tsuchiyama writes: > + if (!fb->adding_file) > +{ > + if (fb->use_commit_times && fb->changed_date) > +{ > + SVN_ERR(svn_io_set_file_affected_time(fb->changed_date, > +fb->local_abspath, > +

Re: [PATCH] fix exception of gen-make.py using --with-libintl [Re: svn commit: r1504501 - in /subversion/trunk: build.conf build/generator/gen_win.py build/generator/gen_win_dependencies.py gen-make.p

2013-08-22 Thread Masaru Tsuchiyama
Hi. Does anybody comment my patch? -- Masaru Tsuchiyama Masaru Tsuchiyama wrote: Hi. Does anybody comment my patch? (2013/07/31 21:53), Masaru Tsuchiyama wrote: Hi. Running gen-make.py with --with-libintl option raises AttributeError exception when using svn-win32-libintl.zip at http://su

Re: [PATCH] print debug information when subprocess.Popen subprocess.Popen

2013-08-22 Thread Masaru Tsuchiyama
Daniel Shahaf wrote: On Sat, Aug 03, 2013 at 03:52:49PM +0300, Daniel Shahaf wrote: Masaru Tsuchiyama wrote on Sat, Aug 03, 2013 at 16:06:06 +0900: If I run the these scripts at Release\subversion\tests\cmdline, I don't get a file-not-found error. So perhaps the right fix is not your patch, b

Re: [PATCH] add 'svn:use-commit-times' property

2013-08-22 Thread Masaru Tsuchiyama
Hello Daniel Could you comment the updated patch? > I prefer that someone else review the semantics, i.e., the actual > feature being added. Does anybody comment for the feature? -- Masaru Tsuchiyama masaru tsuchiyama wrote: > You still have several places where your code exceeds 8

Re: [PATCH] create LC_MESSAGES directories and copy subversion.mo to the directories at compile time

2013-08-22 Thread Masaru Tsuchiyama
Does anybody comment my patch? -- Masaru Tsuchiyama Masaru Tsuchiyama wrote: Hi I attach a patch to create LC_MESSAGES directories and copy subversion.mo files to the directories and do some extra works. [[[ * build/generator/gen_win.py (POFile.__init__) : set class member 'lang' to 'bas

Re: [PATCH] fix compiler error on VS2008 "CVTRES : fatal error CVT1103: cannot read file"

2013-08-22 Thread Masaru Tsuchiyama
Hello Does anybody comment my patch? -- Masaru Tsuchiyama Masaru Tsuchiyama wrote: > Hi > > I attach a patch to fix the following compiler error on VS2008 > "CVTRES : fatal error CVT1103: cannot read file". > > I got the error on tests projects. > > The projects use the same IntermediateDir

Re: [PATCH] add command line options to mailer.py

2013-08-22 Thread Masaru Tsuchiyama
Hello Does anybody comment my patch? -- Masaru Tsuchiyama Masaru Tsuchiyama wrote: > Hello > > I attach a patch to implement command line options for smtp servers. > > [[[ > add command line options to mailer.py > > * tools/hook-scripts/mailer/mailer.py >() : import OptionParser. >(

Re: Long Pathes on Windows and SQLite

2013-08-22 Thread Branko Čibej
On 22.08.2013 14:26, Markus Schaber wrote: > Hi, > > Some days ago, I stumbled across a problem, and could analyze it a little bit > those days. > > Windows has a general limit on the path length of 260 characters on some > APIs, while others allow a length of about 32k characters (with some > r

Long Pathes on Windows and SQLite

2013-08-22 Thread Markus Schaber
Hi, Some days ago, I stumbled across a problem, and could analyze it a little bit those days. Windows has a general limit on the path length of 260 characters on some APIs, while others allow a length of about 32k characters (with some restrictions). Most (if not all) of the SVN codebase seems

Re: [Issue 4416] New - Anonymous checkout of public directory hosted bypre-1.8 fails if repo root is not public

2013-08-22 Thread Ivan Zhakov
On Thu, Aug 22, 2013 at 12:12 PM, Bert Huijben wrote: > This is most likely related to obtaining inherited properties,.. > > Could be a huge regression in soms setups. > The original report was on svn-users@s.a.o: http://svn.haxx.se/users/archive-2013-08/0334.shtml -- Ivan Zhakov CTO | VisualSVN

RE: [Issue 4416] New - Anonymous checkout of public directory hosted bypre-1.8 fails if repo root is not public

2013-08-22 Thread Bert Huijben
This is most likely related to obtaining inherited properties,.. Could be a huge regression in soms setups. Bert -Original Message- From: "marktsuch...@tigris.org" Sent: ‎22-‎8-‎2013 07:09 To: "iss...@subversion.tigris.org" Subject: [Issue 4416] New - Anonymous checkout of public direc