Re: cpp branch

Op woensdag 1 maart 2017 15:31:03 CET schreef Robert Fewell: > Geert, > > Stumbled upon this problem and can not seem to narrow it down further. > > If you have a comma separated file and load it, the column header combo > boxes do not work, unable to change. if you select an additional

Re: cpp branch

Geert, Stumbled upon this problem and can not seem to narrow it down further. If you have a comma separated file and load it, the column header combo boxes do not work, unable to change. if you select an additional separator, like say the semicolon, they start to work. I have looked at

Re: cpp branch

Op dinsdag 21 februari 2017 11:33:08 CET schreef Robert Fewell: > Geert, > > Just an observation, when I do an import and select the date column first > with an invalid default format, the view goes red and the error message > tells me what is wrong. At the same time I ended up with 20 WARN

Re: cpp branch

Geert, Just an observation, when I do an import and select the date column first with an invalid default format, the view goes red and the error message tells me what is wrong. At the same time I ended up with 20 WARN messages, one for each line in the trace file. I was wondering if

Re: cpp branch

Op vrijdag 17 februari 2017 22:09:00 CET schreef Geert Janssens: > Op zondag 12 februari 2017 11:50:35 CET schreef Robert Fewell: > > Geert, > > > > OK I got your latest changes and all built and ran OK. Needed to reread > > your comment about dates as I could not see what gnc_numeric had to do >

Re: cpp branch

Op zondag 12 februari 2017 11:50:35 CET schreef Robert Fewell: > Geert, > > OK I got your latest changes and all built and ran OK. Needed to reread > your comment about dates as I could not see what gnc_numeric had to do with > it !! Well, they don't. I'm mixing two issues in one... > Some

Re: cpp branch

Op maandag 13 februari 2017 10:52:27 CET schreef Robert Fewell: > I added the following to travis.yml... > > after_failure: "cat src/engine/test/test-import-map.log" > > This forced a rerun of my copy, still no failure. > > Bob > Bob, Thanks for your additional testing. I added this line

Re: cpp branch

I added the following to travis.yml... after_failure: "cat src/engine/test/test-import-map.log" This forced a rerun of my copy, still no failure. Bob On 13 February 2017 at 01:25, John Ralls wrote: > > > On Feb 10, 2017, at 1:10 PM, Geert Janssens

Re: cpp branch

> On Feb 10, 2017, at 1:10 PM, Geert Janssens > wrote: > > Op vrijdag 10 februari 2017 14:51:55 CET schreef Geert Janssens: >> Op vrijdag 10 februari 2017 11:24:00 CET schreef Robert Fewell: >>> Geert, >>> >>> Looks like I did too much housekeeping and removed the

Re: cpp branch

Geert, OK I got your latest changes and all built and ran OK. Needed to reread your comment about dates as I could not see what gnc_numeric had to do with it !! Some changes I think required. 1. The initial text on the assistant page. 2. If you have selected an account in the combo and then

Re: cpp branch

Op vrijdag 10 februari 2017 14:51:55 CET schreef Geert Janssens: > Op vrijdag 10 februari 2017 11:24:00 CET schreef Robert Fewell: > > Geert, > > > > Looks like I did too much housekeeping and removed the lot. If I remember > > the only real issues I found > > > > were the settings save and

Re: cpp branch

Bob, I managed to work around the segfault I had left and pushed that commit as well in case you want to try it. While doing so I noticed three more issues I need to fix: - newlines are not properly stripped from csv type data (they are for fixed width type data) - column splitting misbehaves

Re: cpp branch

Op vrijdag 10 februari 2017 11:24:00 CET schreef Robert Fewell: > Geert, > > Looks like I did too much housekeeping and removed the lot. If I remember > the only real issues I found > were the settings save and load but that was down to you changing it, so > no issue, double clicking on the

Re: cpp branch

Geert, Looks like I did too much housekeeping and removed the lot. If I remember the only real issues I found were the settings save and load but that was down to you changing it, so no issue, double clicking on the fixed width bar when in csv mode along with the renderer being dropped in the

Re: cpp branch

Op dinsdag 17 januari 2017 13:34:44 CET schreef Robert Fewell: > Geert, > > The boost error was due to me not building boost with icu support. > > I have pulled your cpp branch and tried to build which was OK but had some > errors when I used it. > With the following changes at >

Re: cpp branch

Op dinsdag 17 januari 2017 13:34:44 CET schreef Robert Fewell: > Geert, > > The boost error was due to me not building boost with icu support. > I'm glad you managed to get past this. > I have pulled your cpp branch and tried to build which was OK but had some > errors when I used it. > With

Re: cpp branch

Geert, The boost error was due to me not building boost with icu support. I have pulled your cpp branch and tried to build which was OK but had some errors when I used it. With the following changes at https://github.com/Bob-IT/gnucash/commits/fork_branch, I can import a csv file with no

Re: cpp branch

Op dinsdag 10 januari 2017 15:38:27 CET schreef Robert Fewell: > Geert, > > Thank you for the comments, especially the date ones which now make sense. > I will have a look at my changes and why I made them as it may relate to > changes I will need to the price importer. > You're welcome :) I

Re: cpp branch

Bob, Op vrijdag 6 januari 2017 12:55:34 CET schreef Robert Fewell: > Geert, > > Sorry for the direct email, was not sure how to advise of my efforts with > your cpp branch. > While the branch is not committed to the official gnucash repo yet, it's still code related to gnucash so feel free to