2017-03-23 GnuCash IRC logs

00:33:43 *** bertbob has quit IRC
00:55:19 *** bertbob has joined #gnucash
01:03:24 *** fiddlerwoaroof has quit IRC
01:04:45 *** fiddlerwoaroof has joined #gnucash
01:08:52 *** tuxd00d has quit IRC
01:30:08 *** O01eg has quit IRC
01:52:35 *** Mechtilde has joined #gnucash
02:14:04 *** fell_ has joined #gnucash
02:15:53 *** fell has quit IRC
02:40:55 *** Mechtilde has quit IRC
03:23:05 *** fell_ is now known as fell
03:23:23 *** gncbot sets mode: +o fell
03:40:51 *** fiddlerwoaroof has quit IRC
03:49:11 *** fiddlerwoaroof has joined #gnucash
04:11:41 *** mrklintscher has quit IRC
04:18:42 *** fiddlerwoaroof has quit IRC
04:21:04 *** gjanssens has joined #gnucash
04:21:05 *** ChanServ sets mode: +o gjanssens
04:28:24 *** mrklintscher has joined #gnucash
04:29:36 *** fiddlerwoaroof has joined #gnucash
05:03:50 *** fiddlerwoaroof has quit IRC
05:11:52 *** fiddlerwoaroof has joined #gnucash
05:49:43 *** fekepp has joined #gnucash
06:34:28 *** mlncn_ has joined #gnucash
06:48:55 *** Jimraehl1 has joined #gnucash
07:24:04 *** mlncn_ has quit IRC
07:47:57 *** Jimraehl1 has left #gnucash
07:49:56 *** rickoehn has joined #gnucash
07:56:28 *** mlncn_ has joined #gnucash
09:22:46 *** jonas has quit IRC
09:23:50 *** jonas has joined #gnucash
10:01:00 *** cg has quit IRC
10:36:32 *** benoitg has joined #gnucash
11:21:12 *** mrklintscher has joined #gnucash
11:39:12 <fell> gjanssens, jralls: it would be really nice, if one of you could write a few words about travis in the wiki.
12:02:48 *** mlncn_ has quit IRC
12:38:53 *** mlncn_ has joined #gnucash
12:51:04 *** Mechtilde has joined #gnucash
13:14:53 *** Mechtilde has quit IRC
13:17:12 *** Mechtilde has joined #gnucash
13:17:18 *** mlncn_ has quit IRC
13:52:52 *** fekepp has quit IRC
13:55:51 *** mlncn_ has joined #gnucash
14:00:13 *** mlncn_ has quit IRC
14:01:36 *** mrklintscher has quit IRC
14:22:31 *** mlncn_ has joined #gnucash
14:22:34 <jralls> fell: Where and for whom?
14:29:03 <fell> jralls: a new page somewhere linked in Developement / Testing.
14:29:19 <fell> For me ;-)
14:29:36 <fell> I get strange mails
14:29:59 <jralls> What are they asking about?
14:30:01 <fell> and do not know when and how to react.
14:30:27 <jralls> Ah, you mean telling you that some commit or other has broken travis?
14:31:21 <fell> Build: #941
14:31:22 <fell> Status: Fixed
14:32:04 <fell> because someone before had some error.
14:35:12 <jralls> If it's fixed then there's nothing to do. In that case it looks like a random failure happened on gjanssens's previous commit: Random because your commit couldn't have affected the test.
14:39:03 <jralls> Sorry, wrong predecessor, it was a failure from Bob Fewell's jqplot commit. But the test failure was something unrelated to that, too.
14:41:13 <fell> Because every commiter could get such mails, we should write a few words:
14:41:25 <fell> What is Tavis
14:41:38 <fell> What shoud I do if ...
14:41:51 <jralls> Agreed. I'll get something up shortly.
14:45:18 <jralls> I notice that we also need to update the Building page to talk about cmake, especially as we
14:45:50 <jralls> darn. Especially as we'll probably drop autotools pretty soon.
14:56:46 *** jchonig has quit IRC
15:01:16 *** fekepp has joined #gnucash
15:01:24 <fell> We are faster switching the tools than an old man like me can learn, how to use them. ;-)
15:03:24 <jralls> Eh? We've been using autotools for at least 15 years.
15:03:45 <jralls> Anyway, are you really that much older than I? I'm 60.
15:03:54 <fell> svn->git, ...
15:04:36 <fell> I hope I will become 60 in December
15:05:11 <jralls> I hope so too. So two geezers, one driving change and the other complaining of it. Sounds like a TV show. ;-)
15:08:42 <Mechtilde> and I#m the old wife between
15:09:00 *** frakturfreak has joined #gnucash
15:11:32 <fell> :-)
15:18:12 <gjanssens> LOL
15:18:12 <gncbot> gjanssens: Sent 1 day, 16 hours, and 3 minutes ago: <fell> Is there some reason, why default zoom has different tooltip texts in glade and prefs - except riling translators? ;-) I would prefer to use only the more informative from glade.
15:19:26 <gjanssens> fell: no good reason no. Probably sloppiness from my side
15:20:34 <fell> Sometimes it might be a question of layout or markup. That is the reason I asked.
15:21:39 <fell> I assume there is no easy way to put then in a const/var and use that in glade.
15:25:32 <gjanssens> Not that I know of. The "other" tooltip is coming from the preferences xml file. I didn't consider these are also parsed for translation.
15:26:08 <gjanssens> I'll copy the text from glade to the prefs file as well.
15:32:08 <gjanssens> Hmm turns out to be harder than expected...
15:33:05 <gjanssens> I'm having some issues getting the newlines interpreted the same in both files
15:34:05 <jralls> Maybe the preferences xml file should be in POTFILES.skip...
15:39:29 <gjanssens> I'm not sure. The strings in there are being used by dconf(-editor)
15:40:04 <gjanssens> I have no idea if this also applies to the other gsettings backends (plists or windows registry)
15:43:40 <gjanssens> Admittedly that is a rather technically adept user oriented tool.
15:43:42 <fell> Yes, Linux dconf-editor shows translations
15:45:58 <fell> I can remember questions like "How can I get more than 4 files in history" and then we send them to the editor.
15:47:43 <gjanssens> Nevermind. I've removed the manual line breaks and now both message id's are interpreted as identical.
15:48:09 <gjanssens> The tooltip still shows reasonable as gtk applies linewrapping by itself.
15:48:51 <fell> Nice
15:49:48 <jralls> Excellent. Thanks.
15:52:28 *** benoitg has quit IRC
15:54:29 <fell> How should I Interpret "Not enough information for Blank Transaction?" from src/gnome-utils/gnc-tree-control-split-reg.c:1725?
15:58:21 <jralls> The amplifying message at 1727 isn't enough?
15:58:35 <fell> The blank/empty transaction, I assume, not freeing the transaction
15:58:46 <gjanssens> Yes
15:59:01 <gjanssens> And this is only triggered if the blank transaction has no splits
15:59:34 <gjanssens> "Blank" transaction is the line for a new transaction in each register
16:00:08 <fell> I think, I will add a comment for other translators
16:00:26 <gjanssens> Once you start editing it it will track all parts of that new transaction until you commit it (by leaving the blank transaction in some way)
16:01:00 <gjanssens> From that point on the blank transaction becomes a normal transaction like any other and a new blank is created
16:01:25 <gjanssens> Apparently the code which can display this message guards against trying to save a blank transaction with no splits
16:02:45 <gjanssens> I'm not sure how one could get in this situation though
16:03:14 <fell> Oh, better you add the translator comment, if it is so complex.
16:05:31 <gjanssens> Oh, it doesn't have to be that complicated for translators. I was mostly thinking out loud
16:05:43 <jralls> It would get there by starting to edit and then pressing Enter.
16:06:11 <jralls> Especially in split view where splits aren't created automatically.
16:06:20 <gjanssens> How about: "This message will be presented when a user attempts to record a transaction without splits"
16:07:26 <fell> OK
16:07:49 <gjanssens> jralls: except it doesn't
16:08:00 <gjanssens> That will still create a single split with no amount
16:08:12 <gjanssens> At least on my local maint branch build
16:08:48 <gjanssens> I entered something bogus in the description field and then tried either of clicking in another transaction
16:08:50 <gjanssens> hitting enter
16:08:53 <gjanssens> hitting return
16:09:10 <gjanssens> all of them resulted in one split being added, in all view modes
16:09:39 <jralls> So maybe it's excessively defensive programming.
16:09:51 <gjanssens> Possibly
16:10:22 <gjanssens> Or a guard for something that no longer happens
16:12:03 <gjanssens> Oh, hold on! This code is part of the register2 implementation. It wouldn't be triggered by the traditional register
16:14:19 <gjanssens> And even there I can't trigger the dialog :(
16:15:04 <gjanssens> The same test as above prints a warning to the trace file about the transaction missing a currency
16:15:14 <fell> That is also funny: There are a bunch of string, which I could not find in the GUI, because I didn't configure with register2.
16:15:19 <gjanssens> So the test bails out before the number of splits is evaluated
16:16:01 <gjanssens> fell: I believe register2 is built by default, but only visible if you set the --extra command line parameter
16:16:08 *** Mechtilde has quit IRC
16:16:55 <gjanssens> That will add an extra menu called "Extensions" which will allow you to open an account in the register2 view
16:17:54 <jralls> It's high time we decide what to do with reg2.
16:18:02 <fell> I have that menu with Business and Register2
16:19:37 <jralls> We kept it in 2.6 so that Bob could continue to work on it, but then scared him off with talk of switching to Qt or wx without making it clear that that's years away.
16:19:57 <jralls> Maybe because we weren't clear on it ourselves.
16:20:03 <gjanssens> Probably
16:21:22 <gjanssens> I had meant to try and fix where Bob left off, but there's always been a more pressing matter...
16:22:20 <gjanssens> My port of the current register away from gnome-canvas to gtk3 has helped me a lot in understanding how that register works
16:22:33 <gjanssens> And that will be helpful when I revist the new register code.
16:23:38 <gjanssens> At this point I haven't decided yet which of the two is in better shape to bring up to current standards
16:24:07 <gjanssens> Both need a significant amount of work to get there.
16:24:37 <gjanssens> I wouldn't drop either of them until I (or someone else) can better judge this
16:25:38 <gjanssens> What I do know for sure is that refactoring the old register is not something that can be done for 2.8 still.
16:26:09 <gjanssens> It really needs to be taken completely apart.
16:26:34 <gjanssens> It has a gui toolkit independent mvc model and then a gtk layer is bolted on top of this
16:26:46 <gjanssens> IMO that makes things horribly complex
16:27:32 <gjanssens> callback functions are passed around indirectly between the two parts and it's currently quite a challenge to follow code flows
16:27:59 <gjanssens> I haven't looked at the register2 code in years so memory is faint
16:28:18 <gjanssens> I do remember in the end it got stuck on performance issues
16:29:22 <gjanssens> But I suspect it taps into the same complex toolkit independent mvc which means lots of data is copied around from that environment to the GtkTreeView/Model environment
16:29:35 <jralls> don't forget that there is also code in there that really belongs in engine. I'm thinking in particular of price-manipulation code, but there's other stuff too.
16:29:40 <gjanssens> I would guess this is part of the performance problems
16:29:57 <gjanssens> Yep
16:30:36 <gjanssens> And table management in c is needlessly complex
16:30:53 <gjanssens> I'm thinking c++ vectors all the time when I wade through it...
16:31:17 <gjanssens> Anyway, IMO too big a project to still start for 2.8
16:31:26 <jralls> I think of them a lot when working on our code.
16:31:42 <gjanssens> So I'd carry both registers around for the next develpment cycle
16:32:20 <gjanssens> yes, much of our current code would benefit from them
16:32:22 <jralls> I agree completely that it's too big a project. I suggest instead that we move reg2 into a feature branch so it's still available to developers but not to users.
16:32:37 <gjanssens> I'd rather not
16:32:55 <gjanssens> It would cause even more bit rot than it's getting now already
16:33:00 <jralls> And so that translators aren't burdened with strings that no user ever sees.
16:33:23 <gjanssens> That's a more interesting argument
16:33:29 <jralls> I don't think bit rot matters at this point. It will require a refactor and rewrite anyway.
16:34:14 <jralls> The first refactor is to pull out all of the code Bob copied to new files thinking that the old reg would just go away.
16:34:47 <gjanssens> That's a (yet another) job I already started a while back :)
16:35:11 <gjanssens> And haven't found time to finish so far
16:35:41 <jralls> I sympathize. There's too much that needs to be done.
16:36:05 <gjanssens> Oh well, no complaints
16:36:37 <gjanssens> I'm really eager to get the gtk3 port running
16:37:03 <gjanssens> Then it will probably be msys2/mingw64 (to get gtk3 on windows)
16:37:35 <gjanssens> After that I think I'll revisit the csv importer for the remaining loose ends
16:37:35 <jralls> Right. I think that's the short-term priority for both of us.
16:38:26 <gjanssens> And if there's still time after that, deduplicating the register code
16:38:42 <jralls> And I've got some cleanup in the SQL backend to switch from gnc-numeric to GncNumeric and to write some more tests.
16:39:01 <gjanssens> Ok
16:39:13 <jralls> And when all that's done it will probably be December...
16:39:32 <gjanssens> Indeed
16:39:48 <fell> Which Year? ;-)
16:41:10 <jralls> This one, I hope. We need all of that to release 2.8.
16:44:05 <fell> I should not read here while committing
16:44:40 <fell> Fix typo had de.po attached :-(
16:45:12 <jralls> lmat: Have you had a chance to look at https://bugzilla.gnome.org/show_bug.cgi?id=761682?
16:53:34 <gjanssens> fell: yep :)
16:55:11 <gjanssens> fell: regarding https://bugzilla.gnome.org/show_bug.cgi?id=759844 did you have a chance to check if you have an unused build dir in your source dir ?
16:55:55 <fell> I believe, I have only one build dir.
16:56:10 <gjanssens> Hmm, curious
16:56:29 <fell> I had made a git xclean before
16:57:07 <fell> But I saw, some intltool files were truncated to size 0
16:58:12 <fell> But I wanted finish my de.po review before digging further.
16:59:07 <gjanssens> Sure, no rush
17:00:21 <gjanssens> jralls: another topic - trunk branch
17:00:31 <gjanssens> We recently decided dropping it
17:00:35 <jralls> Nuke it.
17:00:38 <gjanssens> Do you still remember how it's set up ?
17:00:50 <gjanssens> It's an alias IIRC
17:01:06 <gjanssens> Can one just run git branch -d on it ?
17:02:01 <jralls> Of course, but that will only remove it from your local repo. To kill it in code you need to git push origin :trunk
17:02:19 <gjanssens> Obviously.
17:02:39 <gjanssens> I just wanted to be sure that wouldn't nuke master with it
17:02:48 <gjanssens> Oh well if it does that's easily repaired...
17:03:20 <jralls> But origin/HEAD points to it and it needs to be aimed at origin/master or much will break.
17:04:17 <gjanssens> Well I just discovered *that* is why I can't push nothing to trunk
17:04:49 <jralls> http://stackoverflow.com/questions/8839958/how-does-origin-head-get-set
17:09:12 <gjanssens> Thanks for the pointer jralls. Let's see if I can get it moved (unless you are already at it)
17:09:58 <jralls> Nope, go for it. While you do that I'll have a look at the gitolite repo to make sure there's nothing there that needs to be changed.
17:10:19 <gjanssens> Ok
17:12:25 <jralls> I don't see anything in gitolite.
17:18:13 <gjanssens> Good
17:18:30 <gjanssens> code however refuses to delete the trunk branch
17:19:04 <gjanssens> "By default, deleting the current branch is denied" and lots of other error text
17:19:23 <gjanssens> This is so even after running git remote set-head origin master
17:20:23 <gjanssens> remote: error: You can set 'receive.denyDeleteCurrent' configuration variable to
17:20:23 <gjanssens> remote: error: 'warn' or 'ignore' in the remote repository to allow deleting the
17:20:25 <gjanssens> remote: error: current branch, with or without a warning message.
17:20:52 <gjanssens> So I guess this needs intervention by... warlord !!!
17:20:57 <gjanssens> :D
17:21:32 <gjanssens> Interestingly github did allow me to drop the trunk branch
17:21:50 <gjanssens> I expect it will be automatically recreated there upon the next push to code
17:22:08 <gjanssens> jralls: unless you have other ideas still ?
17:26:42 <gjanssens> warlord: are you around ?
17:27:11 <jralls> I'm trying out http://stackoverflow.com/questions/13949093/git-change-default-branch-gitolite.
17:29:54 <jralls> Which isn't working. Do you remember what gitolite version is installed on code?
17:30:12 <gjanssens> Unfortunately no
17:30:18 *** mlncn_ has quit IRC
17:31:45 <gjanssens> jralls: most likely version 3.3
17:32:11 <gjanssens> I have a local gitolite setup from the same time period for experimentation
17:32:13 <jralls> OK. That *should* be new enough.
17:32:14 <gjanssens> That's still 3.3
17:32:41 <gjanssens> On comment on the SO article says it required 3.5
17:34:17 <jralls> Not exactly. He said a different version of the command was needed on 3.5.3. Sending the 'info' command tells me that the new code has gitolite 3.6.6-1
17:36:04 <fell> Does anybody see, why
17:36:05 <fell> "There are problems with the import settings!\n"
17:36:06 <fell> "The date format could be wrong or there are not enough columns set..."
17:36:14 <fell> should be c-format?
17:36:33 <jralls> Oh, wait. I didn't understand what symbolic-refs was telling me the first time: refs/heads/master is a symbolic ref to trunk, not the other way around.
17:37:22 <jralls> fell: You mean not translated? Are they displayed to the UI or written to tracefile?
17:38:16 <fell> in gnucash.pot assistant-csv-trans-import.c:1426 gets the #, c-format flag
17:38:27 <gjanssens> jralls: should I then try to remove master first and recreate it on the same commit as trunk points at ?
17:39:34 <fell> above is the msgid
17:40:28 <jralls> gjanssens: I was going to try to delete master and rename trunk to master directly with git pass-through.
17:41:59 <jralls> sorry, gitolite pass-through. i.e. ssh git@code symbolic-ref -d gnucash refs/heads/master. I don't think that you can do that with a git command to the remote.
17:42:24 <gjanssens> jralls: oh. I just attempted to delete master already
17:42:35 <gjanssens> I hope I didn't mess things up then
17:42:46 <jralls> Did it work? I got an "invalid perm" error.
17:43:04 <gjanssens> It gave errors as well, but it seems it worked after all.
17:43:13 <gjanssens> Here's the full output:
17:43:26 <gjanssens> remote: fatal: Invalid revision range 4f5658fc7a3ed81bbd5b30238e535fa19f60af42..0000000000000000000000000000000000000000
17:43:28 <gjanssens> remote: fatal: Invalid revision range 4f5658fc7a3ed81bbd5b30238e535fa19f60af42..0000000000000000000000000000000000000000
17:43:29 <gjanssens> remote: *** Mirror changes to origin (usually github)...
17:43:31 <gjanssens> remote: To ssh://github.com/Gnucash/gnucash.git
17:43:32 <gjanssens> remote: 1cbb25d..41a346c origin/maint -> origin/maint
17:43:34 <gjanssens> remote: ! [remote rejected] master (refusing to delete the current branch: refs/heads/master)
17:43:35 <gjanssens> remote: error: failed to push some refs to 'ssh://git@github.com/Gnucash/gnucash.git'
17:43:37 <gjanssens> To code.gnucash.org:gnucash
17:43:38 <gjanssens> - [deleted] master
17:43:46 <gjanssens> Note github refused to delete master as it's the HEAD branch there
17:44:02 <gjanssens> but he last line suggests master has been removed on code
17:44:14 <gjanssens> So you can try to rename trunk now via gitolite
17:45:51 <jralls> Have been. gitolite keeps saying "unknown command branch gnucash -m trunk master" (or with -m gnucash, I tried both ways).
17:47:51 <gjanssens> I'm making a fresh clone of gnucash at code to see the actual state now
17:50:15 <gjanssens> Ok, I managed to delete the master link from code and push trunk as master branch
17:50:24 <gjanssens> Now to change the refs
17:51:40 <gjanssens> Uh no
17:51:45 *** bertbob has quit IRC
17:51:56 <gjanssens> master is still a symbolic ref to trunk :(
17:53:14 *** rickoehn has quit IRC
17:54:47 <jralls> Sigh. I was just trying that too. I think we'll need warlord to do it from the shell.
17:55:02 <jralls> I need to go run some errands.
17:57:08 *** mrklintscher has joined #gnucash
17:58:00 <gjanssens> ok I may be to bed when you return
17:58:25 <jralls> fell: https://www.gnu.org/software/gettext/manual/html_node/c_002dformat-Flag.html. The msgid has printf format codes in it.
17:59:40 <jralls> gjanssens, Considering that it's 230 there, I expect you to go to bed before that! Good night! Fell, you too!
18:00:32 <fell> Good luck with your errands!
18:02:32 *** mlncn_ has joined #gnucash
18:06:39 *** bertbob has joined #gnucash
18:12:21 <gjanssens> jralls: Don't know if it was something you did or I, but it seems to be done now. No more trunk branch, and master is current HEAD
18:12:22 <fell> Hm, I do not see a '%' in the string, but it is part of a g_strdup_printf
18:13:01 <gjanssens> To get rid of it in my local repos I still had to run git fetch -p
18:13:17 <gjanssens> Without -p the deleted branches aren't checked apparently
18:13:55 <fell> Probably you should add it to wiki:git.
18:17:25 <gjanssens> Tested a push to master, which seems to work fine
18:17:50 <gjanssens> fell: remind of of that if I haven't done so in a couple of days
18:17:58 <gjanssens> I've got to leave now
18:18:19 <fell> I will try ..
18:18:42 <fell> Good night, gjanssens!
18:19:17 <gjanssens> Good night fell!
18:19:25 *** gjanssens has quit IRC
18:20:02 *** mrklintscher has quit IRC
18:46:36 *** fekepp has quit IRC
19:07:12 *** frakturfreak has quit IRC
19:41:01 *** dgtlmoon has joined #gnucash
19:54:15 *** mlncn_ has quit IRC
20:15:08 *** mlncn_ has joined #gnucash
20:42:18 *** Heisenburg has joined #gnucash
21:06:21 *** benoitg has joined #gnucash
21:10:39 *** fekepp has joined #gnucash
21:42:22 *** To7 has joined #gnucash
22:43:50 *** xmaka has quit IRC
22:44:29 *** xmaka has joined #gnucash
22:49:44 *** boson has quit IRC
22:55:42 *** boson has joined #gnucash
23:19:24 <fell> We use the term "Billing" for a section about wage calculation In dialog-employee and as adress type in business-reports/receivables.scm.
23:21:10 <fell> I think, we should use different terms. THey may need different translations.
23:32:43 <fell> ...For other Business partners we use billing information. I will adjust it.
23:39:56 *** storyjesse has joined #gnucash