Re: [PATCH] Issue #4711 invalid xml file produced by svn log --xml -g --search

2022-01-02 Thread Daniel Sahlberg
Den fre 24 dec. 2021 kl 20:44 skrev Daniel Sahlberg < daniel.l.sahlb...@gmail.com>: > Sorry for the long delay but I finally found some time to circle back to > this. > > Den tors 26 aug. 2021 kl 17:21 skrev Daniel Shahaf >: > >> > I'm leaning towards setting this as a milestone=2.0 issue and for

Re: [PATCH] Issue #4711 invalid xml file produced by svn log --xml -g --search

2021-12-24 Thread Daniel Sahlberg
Sorry for the long delay but I finally found some time to circle back to this. Den tors 26 aug. 2021 kl 17:21 skrev Daniel Shahaf : > > I'm leaning towards setting this as a milestone=2.0 issue and for the > time > > leaving it alone not to risk destabilising anything. > > That's an option. Or w

Re: [PATCH] Issue #4711

2021-08-26 Thread Daniel Shahaf
Daniel Sahlberg wrote on Thu, Aug 26, 2021 at 14:44:04 +0200: > Den lör 21 aug. 2021 kl 05:18 skrev Daniel Shahaf : > > > Daniel Sahlberg wrote on Fri, 20 Aug 2021 10:30 +00:00: > > > Den fre 20 aug. 2021 kl 12:11 skrev Daniel Shahaf < > > d...@daniel.shahaf.name>: > > > > Daniel Sahlberg wrote on

Re: [PATCH] Issue #4711

2021-08-26 Thread Daniel Sahlberg
Den lör 21 aug. 2021 kl 05:18 skrev Daniel Shahaf : > Daniel Sahlberg wrote on Fri, 20 Aug 2021 10:30 +00:00: > > Den fre 20 aug. 2021 kl 12:11 skrev Daniel Shahaf < > d...@daniel.shahaf.name>: > > > Daniel Sahlberg wrote on Thu, Aug 19, 2021 at 23:23:49 +0200: > > > > [[[ > > > ⋮ > > > > > -

Re: [PATCH] Issue #4711

2021-08-20 Thread Daniel Shahaf
Daniel Sahlberg wrote on Fri, 20 Aug 2021 10:30 +00:00: > Den fre 20 aug. 2021 kl 12:11 skrev Daniel Shahaf : > > Daniel Sahlberg wrote on Thu, Aug 19, 2021 at 23:23:49 +0200: > > > [[[ > > ⋮ > > > > > > r4 | dsahlberg | 2021

Re: [PATCH] Issue #4711

2021-08-20 Thread Daniel Sahlberg
Den fre 20 aug. 2021 kl 12:11 skrev Daniel Shahaf : > Daniel Sahlberg wrote on Thu, Aug 19, 2021 at 23:23:49 +0200: > > Current output. Again XML indented for readability. It is the same WC as > > before, but with an additional revision 7 where I reverse merge what was > > merged in r6: > > At som

Re: [PATCH] Issue #4711

2021-08-20 Thread Daniel Shahaf
Daniel Sahlberg wrote on Thu, Aug 19, 2021 at 23:23:49 +0200: > Current output. Again XML indented for readability. It is the same WC as > before, but with an additional revision 7 where I reverse merge what was > merged in r6: At some point it becomes a good idea to post your test data so others

Re: [PATCH] Issue #4711

2021-08-19 Thread Daniel Sahlberg
Den tors 19 aug. 2021 kl 14:51 skrev Daniel Shahaf : > Daniel Sahlberg wrote on Wed, Aug 18, 2021 at 07:54:47 +0200: > > Den tis 17 aug. 2021 kl 05:36 skrev Daniel Shahaf < > d...@daniel.shahaf.name>: > > > subversion/svn/schema/log.rnc doesn't mention reverse-merge at all. > > > (I was looking to

Re: [PATCH] Issue #4711

2021-08-19 Thread Daniel Shahaf
Daniel Shahaf wrote on Thu, 19 Aug 2021 12:51 +00:00: > Daniel Sahlberg wrote on Wed, Aug 18, 2021 at 07:54:47 +0200: > > The larger question is "do we want it?". > I think we should include it, because: > > - The API provides it. The CLI be to the API as the plain log output is > to the XML o

Re: [PATCH] Issue #4711

2021-08-19 Thread Daniel Shahaf
Daniel Sahlberg wrote on Wed, Aug 18, 2021 at 07:54:47 +0200: > Den tis 17 aug. 2021 kl 05:36 skrev Daniel Shahaf : > > > Daniel Sahlberg wrote on Sun, Aug 15, 2021 at 22:11:29 +0200: > > > Den lör 14 aug. 2021 kl 13:21 skrev Daniel Shahaf < > > d...@daniel.shahaf.name>: > > > > Specifically, I wa

Re: [PATCH] Issue #4711

2021-08-17 Thread Daniel Sahlberg
Den tis 17 aug. 2021 kl 05:36 skrev Daniel Shahaf : > Daniel Sahlberg wrote on Sun, Aug 15, 2021 at 22:11:29 +0200: > > Den lör 14 aug. 2021 kl 13:21 skrev Daniel Shahaf < > d...@daniel.shahaf.name>: > > > Specifically, I was pointing out site/tools/upcoming.py as a tool that > > > synthesizes pla

Re: [PATCH] Issue #4711

2021-08-16 Thread Daniel Shahaf
Daniel Sahlberg wrote on Sun, Aug 15, 2021 at 22:11:29 +0200: > Den lör 14 aug. 2021 kl 13:21 skrev Daniel Shahaf : > > > Daniel Sahlberg wrote on Fri, Aug 13, 2021 at 16:55:38 +0200: > > > danielsh had two comments on IRC, > > > https://colabti.org/irclogger/irclogger_log/svn-dev?date=2021-08-09,

Re: [PATCH] Issue #4711

2021-08-15 Thread Daniel Sahlberg
Den lör 14 aug. 2021 kl 13:21 skrev Daniel Shahaf : > Daniel Sahlberg wrote on Fri, Aug 13, 2021 at 16:55:38 +0200: > > danielsh had two comments on IRC, > > https://colabti.org/irclogger/irclogger_log/svn-dev?date=2021-08-09, > which > > I will try to address here. I'm taking the liberty of copyi

Re: [PATCH] Issue #4711

2021-08-14 Thread Daniel Shahaf
Daniel Sahlberg wrote on Fri, Aug 13, 2021 at 16:55:38 +0200: > danielsh had two comments on IRC, > https://colabti.org/irclogger/irclogger_log/svn-dev?date=2021-08-09, which > I will try to address here. I'm taking the liberty of copying from the IRC > log to get some context. Thanks, and sorry f

Re: [PATCH] Issue #4711

2021-08-13 Thread Daniel Sahlberg
danielsh had two comments on IRC, https://colabti.org/irclogger/irclogger_log/svn-dev?date=2021-08-09, which I will try to address here. I'm taking the liberty of copying from the IRC log to get some context. Den mån 9 aug. 2021 kl 03:12 skrev Daniel Sahlberg < daniel.l.sahlb...@gmail.com>: > Hi,

[PATCH] Issue #4711

2021-08-08 Thread Daniel Sahlberg
Hi, I've been researching issue #4711 [1] and, as requested by danielsh in JIRA, I'm turning to dev@ to discuss a possible solution. As already found by danielsh, the problem lies in the filtering code in svn_cl__log_entry_receiver_xml. If the log entry doesn't match the search pattern, the