Re: cpp branch

2017-03-01 Thread Geert Janssens
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

2017-03-01 Thread 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 separator, like say the semicolon, they start to work. I have looked at

Re: cpp branch

2017-02-21 Thread Geert Janssens
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

2017-02-21 Thread 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 messages, one for each line in the trace file. I was wondering if

Re: cpp branch

2017-02-20 Thread Geert Janssens
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

2017-02-17 Thread 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 with > it !! Well, they don't. I'm mixing two issues in one... > Some

Re: cpp branch

2017-02-14 Thread Geert Janssens
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

2017-02-13 Thread Robert Fewell
bar when in csv mode along > >>> with the renderer being dropped in the callback but I think you were > going > >>> to change that part so probably no issue. > >> > >> Bob, > > > > And just for the record my cpp branch fails on travis with an error

Re: cpp branch

2017-02-12 Thread John Ralls
in csv mode along >>> with the renderer being dropped in the callback but I think you were going >>> to change that part so probably no issue. >> >> Bob, > > And just for the record my cpp branch fails on travis with an error in the > test-import-map test. > I

Re: cpp branch

2017-02-12 Thread Robert Fewell
> so > > > > > > > > no issue, double clicking on the fixed width bar when in csv mode along > > > > with the renderer being dropped in the callback but I think you were > going > > > > to change that part so probably no issue. > > > > > > Bob

Re: cpp branch

2017-02-10 Thread Geert Janssens
> > > 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 callback but I think you were going > > to change that part so prob

Re: cpp branch

2017-02-10 Thread Geert Janssens
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

2017-02-10 Thread 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 load but that was down to you changing it, so > no issue, double clicking on the

Re: cpp branch

2017-02-10 Thread Robert Fewell
insdag 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. > >

Re: cpp branch

2017-02-09 Thread Geert Janssens
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 followi

Re: cpp branch

2017-01-17 Thread Geert Janssens
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 >

Re: cpp branch

2017-01-17 Thread 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 https://github.com/Bob-IT/gnucash/commits/fork_branch, I can import a csv file

Re: cpp branch

2017-01-10 Thread Geert Janssens
u're welcome :) I have just pushed the most recent state of my cpp branch (minus a final work in progress) to my github repository. You may want to rebase to that one. It should present a more or less usable csv transaction import assistant. Note I don't expect you to take the whole transaction rel

Re: cpp branch

2017-01-10 Thread Geert Janssens
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 gnuca