2020-10-13 GnuCash IRC logs

00:27:40 *** phebus has quit IRC
00:45:14 *** phebus has joined #gnucash
00:45:15 *** ChanServ sets mode: +v phebus
00:55:59 *** Mechtilde has joined #gnucash
01:30:05 *** jervin has quit IRC
01:32:56 *** sbluhm has joined #gnucash
01:32:56 *** ChanServ sets mode: +v sbluhm
01:38:41 *** Mechtilde has quit IRC
01:47:55 *** fell has joined #gnucash
01:47:55 *** ChanServ sets mode: +o fell
01:49:40 *** fell_laptop has quit IRC
01:56:16 *** vuamw has joined #gnucash
01:57:22 *** frakturfreak has quit IRC
02:07:01 <CDB-Man_> @tell chris well I've managed to figure out my issue with compensatory notional distributions, and the new file has been uploaded into https://bugs.gnucash.org/show_bug.cgi?id=797796
02:07:01 <gncbot> CDB-Man_: The operation succeeded.
02:11:47 *** frakturfreak has joined #gnucash
02:11:47 *** ChanServ sets mode: +v frakturfreak
02:14:17 *** David has quit IRC
02:14:22 *** David has joined #gnucash
02:39:22 *** bertbob has quit IRC
02:40:16 *** vuamw has quit IRC
02:48:37 *** bertbob has joined #gnucash
02:48:38 *** ChanServ sets mode: +v bertbob
03:10:54 *** vuamw has joined #gnucash
03:14:19 *** gjanssens has joined #gnucash
03:14:19 *** gncbot sets mode: +o gjanssens
03:14:19 *** ChanServ sets mode: +o gjanssens
03:19:50 *** gjanssens has quit IRC
03:23:32 *** bertbob has quit IRC
03:23:50 *** gjanssens has joined #gnucash
03:23:50 *** ChanServ sets mode: +o gjanssens
03:25:27 *** Mechtilde has joined #gnucash
03:27:48 *** bertbob has joined #gnucash
03:27:48 *** ChanServ sets mode: +v bertbob
03:42:04 *** Hamaryns has joined #gnucash
03:42:04 *** ChanServ sets mode: +v Hamaryns
04:05:38 *** fell has quit IRC
04:07:06 *** Aussie_matt has quit IRC
04:17:11 *** sbluhm has quit IRC
04:26:39 *** hussam has quit IRC
04:33:23 *** sbluhm has joined #gnucash
04:50:38 *** sbluhm has quit IRC
04:57:47 *** Hamaryns has quit IRC
05:01:27 *** sbluhm has joined #gnucash
05:01:27 *** ChanServ sets mode: +v sbluhm
05:14:13 *** Mechtilde has quit IRC
05:15:33 *** Hamaryns has joined #gnucash
05:15:33 *** ChanServ sets mode: +v Hamaryns
05:21:56 *** hussam has joined #gnucash
05:21:56 *** ChanServ sets mode: +v hussam
05:31:57 *** fell has joined #gnucash
05:31:57 *** ChanServ sets mode: +o fell
05:32:47 *** User has joined #gnucash
05:40:30 *** Mechtilde has joined #gnucash
06:04:23 *** Aussie_matt has joined #gnucash
06:06:28 *** fell has quit IRC
06:07:34 *** fell has joined #gnucash
06:07:34 *** ChanServ sets mode: +o fell
06:12:31 *** halipaul has quit IRC
06:13:41 *** fell has quit IRC
06:14:47 *** fell has joined #gnucash
06:14:47 *** ChanServ sets mode: +o fell
06:27:33 *** Hamaryns has quit IRC
06:35:27 *** fell_laptop has joined #gnucash
06:35:27 *** ChanServ sets mode: +o fell_laptop
06:35:27 *** fell has quit IRC
06:37:57 *** David has quit IRC
06:38:03 *** David has joined #gnucash
06:39:41 *** halipaul has joined #gnucash
06:43:43 *** halipaul has quit IRC
06:43:45 *** fell_laptop has quit IRC
06:43:48 *** halipaul has joined #gnucash
06:44:02 *** fell_laptop has joined #gnucash
06:44:02 *** ChanServ sets mode: +o fell_laptop
07:03:07 *** Herbert has joined #gnucash
07:05:36 *** storyjesse has joined #gnucash
07:10:20 *** Hamaryns has joined #gnucash
07:10:20 *** ChanServ sets mode: +v Hamaryns
07:39:10 *** sbluhm1 has joined #gnucash
07:39:45 *** chris has joined #gnucash
07:39:45 *** ChanServ sets mode: +v chris
07:39:50 *** gncbot sets mode: +o chris
07:40:27 *** sbluhm has quit IRC
07:56:38 <chris> CDB-Man: I'll check your XLS&gnucash in due course. Free time is now a precious resource :)
07:56:39 *** hussam has quit IRC
07:56:44 <chris> ^CDB-Man_
07:57:39 <chris> CDB-Man_ eventually I'd think a wiki entry like https://wiki.gnucash.org/wiki/Quickstart_Australian_BAS would be invaluable for various stock activity
07:59:21 *** David has quit IRC
07:59:45 *** David has joined #gnucash
08:34:05 *** parsnip has quit IRC
08:44:42 *** fell_laptop has quit IRC
08:46:56 *** Aussie_matt has quit IRC
08:51:47 *** fell_laptop has joined #gnucash
08:51:47 *** ChanServ sets mode: +o fell_laptop
08:55:38 *** halipaul has quit IRC
09:03:32 *** halipaul has joined #gnucash
09:15:41 *** Jimraehl1 has joined #gnucash
09:21:18 *** fell_laptop is now known as fell
09:31:47 *** Hamaryns has quit IRC
09:42:44 *** Hamaryns has joined #gnucash
09:42:44 *** ChanServ sets mode: +v Hamaryns
10:30:55 *** sbluhm1 has quit IRC
10:31:29 *** hussam has joined #gnucash
10:31:29 *** ChanServ sets mode: +v hussam
10:37:09 *** GSmiley has joined #gnucash
10:37:54 *** GSmiley has quit IRC
10:46:10 *** jervin has joined #gnucash
10:49:53 *** sbluhm has joined #gnucash
10:49:53 *** ChanServ sets mode: +v sbluhm
10:57:50 *** jervin has quit IRC
11:00:11 *** Mechtilde has quit IRC
11:15:13 *** Hamaryns has quit IRC
11:31:14 *** guak has joined #gnucash
11:35:44 *** ArtGravity has joined #gnucash
11:35:44 *** ChanServ sets mode: +v ArtGravity
11:43:54 *** sbluhm has quit IRC
11:44:08 <chris> jralls: I've bisected 797959 to the culprit c4d9ca7bc
11:46:09 *** ArtGravity has quit IRC
11:49:41 <fell> Jralls, we have in gnucash/gtkbuilder/assistant-stock-split.glade <property name="title" translatable="yes">Cash in Lieu</property> and <property name="text" translatable="yes">Cash In Lieu</property>. Can I unify them? In which direction?
11:51:09 <chris> 797959 on windows it seems
11:52:14 <jralls> chris, Cool, I was just looking at that in the debugger and got a backtrace. It's in the comma-removal routine in gnc_price_cell_modify_verify.
11:53:25 <jralls> fell, I don't know. Aren't the two properties for different things?
11:54:58 <jralls> chris, which routine was added by that commit.
11:57:21 <chris> uhh gnc_price_cell_modify_verify was modified
11:57:36 <chris> don't know much more than the exact commit tbh
12:04:30 <jralls> Take a look at https://github.com/Gnucash/gnucash/blob/maint/gnucash/register/register-core/pricecell.c#L110 ff. Do you see my mistake? Hint: g_utf8_strchr returns NULL if it doesn't find an instance of thousands_sep.
12:05:44 <chris> nullptr dereference from for conditional
12:06:16 <fell> Yes, the window title becomes "Cash in Lieu" and the default transaction memo "Cash In Lieu". What are the English conventions for both?
12:08:23 <jralls> fell, sorry, didn't notice the case difference for 'in'. The American convention would be "Cash In Lieu" for the title and "Cash in lieu" for the memo; British would use "Cash in lieu" for both.
12:09:17 <jralls> I guess I'd split the difference and go for Cash in Lieu for both to avoid the broken translation.
12:10:50 <fell> The other option would be to add translator comments.
12:12:41 <fell> But that doeesn't solve the lowercase L
12:14:00 <jralls> fell, sorry, what do you mean?
12:15:20 <fell> in Lieu vs. in lieu
12:20:01 <jralls> No, got that part, it's the only L. Making it lowercase in the memo would make it more correct but would break the translations. I suppose you could change the title to Cash In Lieu and translate it to Cash in lieu in en_GB.po (which would be technically correct).
12:21:06 <jralls> But changing the memo to Cash in lieu will cause it to be untranslated for the TP translations in 4.3.
12:22:50 <jralls> chris, no, there's no dereference in the for statement's condition. NULL tests as false so if c == NULL the condition will fail and the loop will quit.
12:26:19 <chris> this routine is imho complicated... it ensures string's commas are removed isn't it? i'd process chars sequentially, add-to-new-string if not "," :-o
12:26:56 <chris> for a lisper, pointer arithmetic is 'out of band'
12:27:19 <chris> ahh we need to process utf8
12:28:23 <jralls> That's what it does, new_newval being the new string being added to. Yes, UTF8, but aside from g_utf8_pointer_to_offset everything would be the same for ASCII.
12:28:49 <chris> (string-filter (lambda (c) (not (eqv? c #\,))) oldstring) bingo
12:31:58 <jralls> Sure, but in C you have to write it out, there is no string-filter function. Also thousands sep is locale-dependent, you can't hard-code the comma.
12:33:18 <chris> gtg now !
12:36:28 *** storyjesse has quit IRC
12:41:08 <fell> "Cash in Lieu" was introduced after we got the indian translations (and many others). It will not become more worse for a few TP updates. I think i will add comments for future translation updates, use the correct strings, msgmerge but remove the then existing fuzzy flags. After that, translators can adjust the case. That seems less annoying than english default text.
12:41:15 <chris> s/c/*c in 3rd for arg
12:45:07 <jralls> fell, Aside from the two TP projects that actually update frequently that will be fine.
12:45:20 *** sbluhm has joined #gnucash
12:46:00 <jralls> chris, nope. g_utf8_strchr returns a char* not a char.
13:27:20 *** sbluhm has quit IRC
13:30:10 *** sbluhm has joined #gnucash
13:30:10 *** ChanServ sets mode: +v sbluhm
13:30:41 *** Mechtilde has joined #gnucash
15:05:10 *** Mechtilde has quit IRC
15:16:42 *** fell has quit IRC
15:17:01 *** fell has joined #gnucash
15:17:01 *** ChanServ sets mode: +o fell
15:36:23 *** fell_laptop has joined #gnucash
15:36:23 *** ChanServ sets mode: +o fell_laptop
15:36:44 *** fell has quit IRC
15:48:42 *** jw4 has quit IRC
15:49:14 *** jw4 has joined #gnucash
15:57:30 *** fell_laptop is now known as fell
15:58:31 <fell> chris, from I18nspector: W: fr.po: invalid-last-translator 'Christopher Lam'
15:59:48 <fell> Formally you must provide an email, but as dou did not intended to become resposibke for fr.po, you should revert the line.
16:04:48 *** fell_laptop has joined #gnucash
16:04:48 *** fell has quit IRC
16:04:48 *** ChanServ sets mode: +o fell_laptop
16:21:16 *** gjanssens has quit IRC
16:22:14 *** fell_laptop has quit IRC
16:30:54 *** fell_laptop has joined #gnucash
16:30:54 *** ChanServ sets mode: +o fell_laptop
16:38:56 *** rIMpossible has joined #gnucash
16:38:56 *** ChanServ sets mode: +v rIMpossible
16:43:39 *** rIMpossible has quit IRC
16:45:10 *** rIMpossible has joined #gnucash
16:45:10 *** ChanServ sets mode: +v rIMpossible
16:45:12 *** rIMpossible has quit IRC
16:45:48 *** rIMpossible has joined #gnucash
16:45:48 *** ChanServ sets mode: +v rIMpossible
17:02:28 *** User has quit IRC
17:18:27 *** KaiForce has quit IRC
17:33:54 *** sbluhm has quit IRC
17:37:48 *** o01eg has quit IRC
17:42:35 <warlord> jralls, quick question -- if a github CI test fails (due to a test inf. failure), is there a way to kick off a new test without pushing a new commit?
17:43:58 <jralls> Yes, follow the link to the travisCI build and click the restart button. When it asks you to authenticate you can sign on with github.
17:51:32 <warlord> I should amend to say "this is on a PR to someone else's project".
17:54:46 <jralls> It might still work if they're using travisCI. I don't know how restrictive they are about who gets the restart button.
17:56:27 *** Aussie_matt has joined #gnucash
18:21:25 *** vuamw has quit IRC
18:21:37 <warlord> jralls, I PMed you the PR#
18:21:42 *** fell_laptop is now known as fell
18:22:25 <fell> Has anybody ever seen: /usr/bin/xgettext: warning: a fallback ITS rule file '/usr/share/gettext-0.21/its/metainfo.its' is used; it may not be in sync with the upstream
18:29:01 <jralls> warlord, Huh, I didn't know that Github had added their own CI. I don't know how to work it, but I guess from the response you got there's no restart button.
18:31:41 <jralls> fell: https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1399
18:34:16 <jralls> fell Added to gettext by https://github.com/autotools-mirror/gettext/commit/71a3e81eafa9a70dacf403bb97fbf1a739283ccd
18:45:28 <fell> So, if I understand it right, my current its package has (still) not installed the metainfo rule, but gettext-0.21 has it already.
18:53:35 <warlord> jralls, I just needed to push an empty commit.
19:00:18 <jralls> fell, from the bug reports linked to that commit it looks more like it has to do with appstream than with its.
19:27:42 <chris> unusual how the comma bug crashes only on windows
19:31:57 <fell> chris, that was not the important line in fr.po
19:32:15 <jralls> chris, that's not unusual for use-after-free bugs. It only crashes if the freed memory gets written over somehow; it's quite possible that on linux and macos the immediate reallocation of new_newval gets the same memory.
19:32:53 <jralls> In which case it would mostly work.
19:33:14 <fell> in de.po the line is: "Last-Translator: Christian Stimming <christian@cstimming.de>\n"
19:38:40 <fell> Chris in fr.po it was:"Last-Translator: La Boussole <yoann@laboussole.coop>\n"
19:39:58 <fell> and the next was "Language-Team: français <yoann@laboussole.coop>\n"
19:42:05 <fell> but that would be wrong (same email as last translator), so I would set it to "Language-Team: NONE\n"
19:49:32 <jralls> fell, chris, gjanssens had an interesting approach for Language-Team in nl.po: "GnuCash Dutch Community <gnucash-nl@gnucash.org>".
19:50:09 <fell> That is copied from gnucash-de. ;-)
19:50:36 <jralls> Sort of. Geert has a nicer name. ;-)
19:51:26 <fell> Right, another improvement.
19:54:00 <fell> Chris, gnucash-fr@gnucash.org looks nicer than NONE
19:54:12 <jralls> But in general not a bad idea for the cases where we have a corresponding mailing list and it's not with TP.
19:57:17 *** Wilco has quit IRC
19:59:55 <jralls> Italian is already that way. Spanish is still set to the TP team although they cut Javier loose a couple of years ago. pt_BR has a mailing list but says "Brazilian Portuguese" with no email. It also hasn't been updated since 2014. Portuguese Portuguese has the TP team though AFAIK Pedro hasn't ever had anything to do with them. That one should be None.
20:02:32 *** Wilco has joined #gnucash
20:34:44 *** guak has quit IRC
20:37:04 *** guak has joined #gnucash
20:37:53 *** guak has quit IRC
20:55:47 <fell> jralls: you accepted ar from TP, but it is marked GC. And I wonder, how their mailto:(nothing) should work.
21:40:16 *** jervin has joined #gnucash
21:45:09 *** jervin has quit IRC
21:45:22 *** jervin has joined #gnucash
22:22:20 *** jervin has joined #gnucash
22:40:51 <chris> fell: may be best you fix fr.po
22:41:50 <fell> In the last moment …, but I will do it. ;-)
23:36:15 *** David has quit IRC