Logs for #nikola for 2018-03-18

03:52:39 <neaj> @ChrisWarrick site build is green, thank you <3 
07:13:39 <neaj> I'm a bit confused about how multilingual sites are handled. looks like we get `index.html`, `th/index-th/index.html`, and `index-th/` (an empty directory).  
07:13:39 <neaj> `index.html` links to the empty directory (404).
07:34:27 <neaj> OK looks like that was some prime PEBKAC
08:03:14 <neaj> do I need to hardcode the language into `NAVIGATION_LINKS`? e.g. `{DEFAULT_LANG: (((('/slug/', "Slug"),), "Menu"), ), "th": (((('/th/slug/', "Slug Thai"),), "Menu Thai"), ) ...`
08:08:25 <berti_> uh, oh, force pushed branch... :-o
08:16:18 <berti> neaj: yes
08:39:00 <neaj> @berti augh :-p
08:39:33 <neaj> so far, navigation links seems to be nikola's weak point ... 
08:39:48 <berti> neaj: there are others
08:40:11 <berti> having contents and translations in the conf.py is not very nice
08:40:34 <berti> have a look at the translation strings for categories and tags
08:42:01 <berti> keeping everything in sync is a bit cumbersome
09:39:55 <berti> hm
09:40:21 <berti> after git pull upstream I got weired errors regarding the translations
09:41:29 <berti>     "title": self.site.config['TAG_PAGES_TITLES'].get(lang, {}).get(classification, self.site.MESSAGES[lang]["Posts about %s"] % classification),
09:41:29 <berti> KeyError: 'Posts about %s'
09:41:50 <berti> but anyway, got to investigate this later.
09:50:53 <gour> morning
09:52:19 <gour> why is e.g. foundation6 not listed under v8? superficial testing looks good here
09:58:56 <ChrisWarrick> gour: right now, v7 themes are compatible with v8
09:59:12 <ChrisWarrick> berti: have you got any custom translation files?
10:00:09 <ChrisWarrick> neaj: as I said on a mailing list: index.rst and index.th.rst, both should have the same slug (index), although it’s not necessary in most cases, so you can have slugs in your other language as well
10:00:13 <gour> ChrisWarrick: ok, i was asking since 'theme management' (command) differs in v8
10:00:23 <ChrisWarrick> gour: not really
10:01:35 <neaj> @ChrisWarrick yes, I ended up with different slugs before I understood the language system better, mainly because 'th 
10:01:45 <neaj> 'th' wasn't available when we started out
10:01:49 <neaj> all good now!
10:03:17 <gour> ChrisWarrick: the docs says: "nikola install_theme foundation6"
10:03:52 <ChrisWarrick> gour: They’re outdated. Fix upcoming.
10:04:28 <ChrisWarrick> ugh, rebuilding now will make it explode
10:04:40 <gour> any hint about this: https://pastebin.com/dWvxFknq or should i re-create my venv from the scratch?
10:06:29 <ChrisWarrick> gour: Oh. I just broke compatibility with old themes.
10:06:53 <ChrisWarrick> gour: which theme is that?
10:07:10 <gour> ChrisWarrick: heh...i can't be angry considering how much you do for Nikola...foundation6
10:08:06 <ChrisWarrick> gour: try reinstalling it first, I can’t see the offending code in it
10:08:21 <ChrisWarrick> or removing cache/.mako.tmp
10:08:55 <gour> i cleaned the cache, db, but didn't help
10:09:46 <ChrisWarrick> gour: figure out where base_helper.tmpl comes from
10:11:31 <KwBot> [nikola-themes] zouhx11 opened issue #144: [zen-ipython theme] search form doesn't work https://github.com/getnikola/nikola-themes/issues/144
10:12:01 <gour> ahh, it looks foundation6 was not installed in this instance
10:12:55 <ChrisWarrick> gour: then there is a bug in one of the built-in themes on master
10:14:18 <gour> ChrisWarrick: i rm-ed bootstrap4 and intsalled as well as referenced foundation6 - it's fine now
10:14:38 <ChrisWarrick> Ah. bootstrap4 in the themes repo is outdated and best left unused
10:15:36 <gour> does aiohttp improve stability or performance as well?
10:16:01 <ChrisWarrick> stability, yes
10:16:08 <ChrisWarrick> performance, it does not matter at all for `nikola auto`
10:17:42 <gour> so, live-reload  with 'auto' should be reliable now without requirement for old tricks?
10:17:57 <ChrisWarrick> it should be
10:26:58 <neaj> work in progress here: http://th.pycon.org/ :-]
10:40:23 <gour> ChrisWarrick: is theming in v8  going to be broken soon, iow. should i wait a bit with creating theme(s) for v8?
10:41:44 <ChrisWarrick> gour: They aren’t going to be broken too much, hopefully.  You may need very slight tweaks (like going from colorbox/flowr to baguetteBox/justified-layout), but those are minor.
10:42:07 <gour> ok, it sounds good
10:42:16 <ChrisWarrick> gour: The *box thing is done on master already, whereas j-l is on a branch, but it won’t matter at all unless you override gallery.tmpl.
13:41:58 -GitHub[nikola]:#nikola- [nikola] ralsina pushed 1 new commit to pymarkdown_options: https://git.io/vx3kE
13:41:58 -GitHub[nikola]:#nikola- nikola/pymarkdown_options 5e84a44 Roberto Alsina: MAke tests pass
13:49:02 -travis-ci:#nikola- getnikola/nikola#9729 (pymarkdown_options - 5e84a44 : Roberto Alsina): The build is still failing.
13:49:03 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/08f6a3f7cfd1...5e84a44a4569
13:49:03 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/355003885
14:03:42 <ChrisWarrick> gour_, ralsina: remember when I said I thought I had fixed the aiohttp issue?  Well, turns out I did.  On the bootstrap4 branch, in an unrelated commit.  https://github.com/getnikola/nikola/commit/e18550df3eaabc8c378ef30ff92feee935abe37c#diff-b92c90f8a590cf369d0adb27e559ee33
14:05:38 -GitHub[nikola]:#nikola- [nikola] Kwpolska force-pushed bootstrap4 from bb28175 to 8b52ea5: https://git.io/vAben
14:05:38 -GitHub[nikola]:#nikola- nikola/bootstrap4 64c4919 Chris Warrick: Introduce Bootstrap 4...
14:05:38 -GitHub[nikola]:#nikola- nikola/bootstrap4 b058357 Chris Warrick: Use new reST CSS in bs4 bundles...
14:05:38 -GitHub[nikola]:#nikola- nikola/bootstrap4 479381c Chris Warrick: Style sourcelinks in bs4 listing.tmpl...
14:13:08 -travis-ci:#nikola- getnikola/nikola#9731 (bootstrap4 - 8b52ea5 : Chris Warrick): The build failed.
14:13:09 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/bb281750a12e...8b52ea5adc12
14:13:09 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/355009735
14:22:35 -GitHub[nikola]:#nikola- [nikola] Kwpolska pushed 4 new commits to bootstrap4: https://git.io/vx3L7
14:22:35 -GitHub[nikola]:#nikola- nikola/bootstrap4 8bb1809 Chris Warrick: Update symlinked list and remove leftover .txt file...
14:22:35 -GitHub[nikola]:#nikola- nikola/bootstrap4 612ba50 Chris Warrick: Match header/menu layout with upstream template...
14:22:35 -GitHub[nikola]:#nikola- nikola/bootstrap4 b7e8b6c Chris Warrick: Update npm assets...
14:24:21 -GitHub[nikola]:#nikola- [nikola] Kwpolska opened pull request #2991: Initial Bootstrap 4 implementation (master...bootstrap4) https://git.io/vx3Lp
14:31:52 -travis-ci:#nikola- getnikola/nikola#9732 (bootstrap4 - 8e0c6e2 : Chris Warrick): The build is still failing.
14:31:53 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/8b52ea5adc12...8e0c6e2ad529
14:31:53 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/355013399
15:01:30 -GitHub[nikola]:#nikola- [nikola] Kwpolska pushed 2 new commits to bootstrap4: https://git.io/vx3qr
15:01:30 -GitHub[nikola]:#nikola- nikola/bootstrap4 6d64507 Chris Warrick: Remove duplicated import...
15:01:30 -GitHub[nikola]:#nikola- nikola/bootstrap4 8221dc4 Chris Warrick: Do not include a source link if it’s empty...
15:01:41 -GitHub[nikola]:#nikola- [nikola] ralsina pushed 1 new commit to pymarkdown_options: https://git.io/vx3qo
15:01:41 -GitHub[nikola]:#nikola- nikola/pymarkdown_options 3b695e5 Roberto Alsina: MAke tests pass
15:18:44 -GitHub[nikola]:#nikola- [nikola] Kwpolska pushed 1 new commit to master: https://git.io/vx3Ye
15:18:44 -GitHub[nikola]:#nikola- nikola/master 695e224 Chris Warrick: Merge pull request #2991 from getnikola/bootstrap4...
15:20:17 -GitHub[themes]:#nikola- [nikola-themes] Kwpolska pushed 2 new commits to master: https://git.io/vx3YZ
15:20:17 -GitHub[themes]:#nikola- nikola-themes/master ecd0017 Chris Warrick: Remove references to install_theme...
15:20:17 -GitHub[themes]:#nikola- nikola-themes/master b4c5753 Chris Warrick: Move bootstrap3 themes from core...
15:23:00 -travis-ci:#nikola- getnikola/nikola#9736 (pymarkdown_options - 3b695e5 : Roberto Alsina): The build was fixed.
15:23:01 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/5e84a44a4569...3b695e5f13bd
15:23:01 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/355023702
15:31:27 -travis-ci:#nikola- getnikola/nikola#9738 (master - 695e224 : Chris Warrick): The build was broken.
15:31:28 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/cc82f60a6c89...695e224f972a
15:31:28 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/355027814
15:35:05 -travis-ci:#nikola- Invariant build by Chris Warrick: The build passed. See https://travis-ci.org/getnikola/invariant-builds/builds/355031595.
15:49:48 -GitHub[nikola]:#nikola- [nikola] Kwpolska pushed 1 new commit to master: https://git.io/vx33L
15:49:48 -GitHub[nikola]:#nikola- nikola/master a4e358e Chris Warrick: Create bootblog4-jinja, make bootstrap4-jinja work...
16:21:31 <rokups> for some reason google refuses indexing site generated with nikola. claims site is marked "noindex".  i grepped entire output folder for all kinds of options that could prevent indexing and found nothing. maybe i dont know something? have any ideas what could be happening here?
16:31:11 <ChrisWarrick> rokups: grep -R noindex output/
16:33:28 <rokups> ChrisWarrick: did that, no such thing there
16:33:40 <rokups> i was wondering maybe there is some other thing that can cause same effect
16:33:52 <ChrisWarrick> rokups: how did you learn about the `noindex` thing?
16:34:16 <ChrisWarrick> https://support.google.com/webmasters/answer/93710?hl=en
16:34:18 <rokups> google search console has notifications about pages being not indexed due to that reason
16:34:38 <rokups> ofc "fetch as googlebot" does not have any noindex in site code either
16:35:26 <rokups> robots.txt is what nikola generates and doing robots check (google search console) returns no errors
16:35:31 <rokups> and side doesnt even have any meta tags
16:36:06 <gour> gitk
16:36:35 <ChrisWarrick> rokups: what is the page?
16:36:42 <gour> excuse me, wrong window
16:37:03 <rokups> ChrisWarrick: https://poema.lt/galerija
16:37:32 <ChrisWarrick> rokups: < X-Robots-Tag: none
16:37:40 <ChrisWarrick> rokups: your server is sending this. Probably the reason.
16:38:18 <rokups> oh, i misinterpreted it's meaning. thanks! will investigate
16:38:39 <ChrisWarrick> > Instead of a meta tag, you can also return an X-Robots-Tag header with a value of either noindex or none in your response. Here's an example of an HTTP response with an X-Robots-Tag instructing crawlers not to index a page:
16:38:53 <ChrisWarrick> from the Google page I linked above.  Cross out "Probably" on the previous line.
16:39:02 <ChrisWarrick> what did you expect that tag to do?
16:41:01 <rokups> no idea how it got there
16:41:24 <rokups> and maybe i thought "none" would mean "ignore robots.txt" idk
16:41:44 <rokups> "no robots allowed" now sounds more logical 
16:43:00 <ChrisWarrick> why would you override robots.txt with a http header?
16:43:08 <ChrisWarrick> also, the best approach to HTTP headers is to have as few as possible
16:51:54 <rokups> yeah now i know where that came from... i copied "very secure ssl settings for nginx" from somewhere. thats what happens when copying things mindlessly from internet.
17:05:06 <berti> ChrisWarrick: yes, my theme has an english-english and english-german translation
17:05:33 <ChrisWarrick> berti: your theme may lack some messages.  (Why do you not use the standard set?)
17:06:30 <berti> ChrisWarrick: German is a very beautiful language
17:06:49 <ChrisWarrick> berti: Ja, und?
17:06:58 <berti> and I hate the so-called denglisch used in most translations
17:07:42 <ChrisWarrick> berti: compare your translation files against https://github.com/getnikola/nikola/blob/master/nikola/data/themes/base/messages/messages_en.py
17:08:09 <berti> Is it required that the custon message file provides all translations? Having only a subset with new and changed translations worked very nice until yesterday...
17:09:10 <berti> ChrisWarrick: my translation is definitly not complete
17:09:39 <ChrisWarrick> berti: It is required.  Your fancy translation file was not used until yesterday. (Sorry!)
17:09:52 <ChrisWarrick> (And you didn’t notice!)
17:10:15 <berti> Huh? It was, definitly, I changed several entries. btw. V8 bleeding edge
17:11:00 <ChrisWarrick> the bug looked as if it was there for 4 years
17:11:34 <berti> No way. My translations were used.
17:11:47 <berti> On top of the existing ones
17:14:33 <berti> Just die a checkout on 9952944a9696a65d520e671950e2f40bdf15e672
17:15:15 <berti> and it works as it did before, Tags -> Schlagwörter, Imprint / Impressum
17:17:16 <berti> yepp, works very smooth, nikola default translation and my incomplete catalog on top of that
17:21:07 <berti> 08528e4231847cd16f90939520cfc48f40622e91 works, too
17:22:13 <berti> and 064d63f8f7dc4d3f8cd1dcfc2984132705f4066e breaks it
17:23:37 <ChrisWarrick> berti: I see what happened here.
17:26:03 <berti> ChrisWarrick: just had a look at the changes in this commit
17:28:10 <ChrisWarrick> berti: I have a better fix ready.
17:28:36 <berti> it not sure if this code is executed at all :-)
17:29:27 <ChrisWarrick> Loading messages from somewhere else than base is such a rare scenario that we’ve managed to swap a long-standing bug for a new one
17:30:35 <berti> funny things is that it worked before
17:31:11 <ChrisWarrick> The bug that I fixed yesterday was the inability to add a custom language
17:31:50 <berti> ok, it did work because i used an already existing language?
17:31:57 <ChrisWarrick> yes
17:34:49 -GitHub[nikola]:#nikola- [nikola] Kwpolska created fix-language-overrides (+1 new commit): https://git.io/vx3We
17:34:49 -GitHub[nikola]:#nikola- nikola/fix-language-overrides dd1ddd0 Chris Warrick: Restore ability to override messages partially...
17:34:50 <ChrisWarrick> berti: can you please test the commit ↑above↑?
17:40:07 <berti> yepp
17:43:09 -travis-ci:#nikola- getnikola/nikola#9740 (fix-language-overrides - dd1ddd0 : Chris Warrick): The build passed.
17:43:10 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/commit/dd1ddd0537b8
17:43:10 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/355065716
17:47:10 <berti> ChrisWarrick: works
17:47:16 <ChrisWarrick> ack
17:48:12 -GitHub[nikola]:#nikola- [nikola] Kwpolska pushed 1 new commit to master: https://git.io/vx3Wj
17:48:12 -GitHub[nikola]:#nikola- nikola/master dd1ddd0 Chris Warrick: Restore ability to override messages partially...
17:48:19 <berti> ChrisWarrick: is it expected to issue a clear warning, when a translation is incomplete?
17:48:55 <berti> ok, it does
17:49:33 <ChrisWarrick> that may be worth tweaking too
17:49:55 <berti> nice. AFAIR that does not work before
17:50:10 <berti> laways got weired errors and stacktraces
17:51:15 <berti> now it says "[2018-03-18T17:48:50Z] WARNING: Nikola: Incomplete translation for language 'de'." whene I simply delete a line from my message catalog
17:51:49 <berti> ChrisWarrick: thx
17:55:50 -travis-ci:#nikola- getnikola/nikola#9741 (master - dd1ddd0 : Chris Warrick): The build passed.
17:55:51 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/a4e358e2cfa9...dd1ddd0537b8
17:55:51 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/355068924
18:16:57 -GitHub[nikola]:#nikola- [nikola] Kwpolska pushed 1 new commit to master: https://git.io/vx38d
18:16:57 -GitHub[nikola]:#nikola- nikola/master f99d4a6 Chris Warrick: Hide “Incomplete language” message for overrides of complete languages...
18:17:02 <ChrisWarrick> berti: check out ↑
18:24:36 -travis-ci:#nikola- getnikola/nikola#9742 (master - f99d4a6 : Chris Warrick): The build passed.
18:24:37 -travis-ci:#nikola- Change view: https://github.com/getnikola/nikola/compare/dd1ddd0537b8...f99d4a66db88
18:24:37 -travis-ci:#nikola- Build details: https://travis-ci.org/getnikola/nikola/builds/355076369
18:49:42 -GitHub[themes]:#nikola- [nikola-themes] Kwpolska pushed 1 new commit to master: https://git.io/vx3BD
18:49:42 -GitHub[themes]:#nikola- nikola-themes/master deea889 Chris Warrick: Some compatibility fixes/workarounds...
18:49:55 -GitHub[themes]:#nikola- [nikola-themes] Kwpolska force-pushed master from deea889 to 0c5ca62: https://git.io/dkLWIA
18:49:55 -GitHub[themes]:#nikola- nikola-themes/master 0c5ca62 Chris Warrick: Some compatibility fixes/workarounds...
18:50:35 -GitHub[themes]:#nikola- [nikola-themes] Kwpolska force-pushed master from 0c5ca62 to 359a1ec: https://git.io/dkLWIA
18:50:35 -GitHub[themes]:#nikola- nikola-themes/master 359a1ec Chris Warrick: Some compatibility fixes/workarounds...
19:30:27 <berti> ChrisWarrick: hm, now missing translations a silently ignored
19:31:04 <berti> ChrisWarrick: is this the expected behaviour?
19:31:14 <ChrisWarrick> berti: The new expected behavior, yes
19:31:24 <ChrisWarrick> berti: (only in your case)
19:31:30 <berti> ok
19:47:37 <KwBot> [nikola-themes] Kwpolska closed issue #140: Some preview images are not representative of the theme https://github.com/getnikola/nikola-themes/issues/140
19:47:47 -GitHub[themes]:#nikola- [nikola-themes] Kwpolska pushed 1 new commit to master: https://git.io/vx3uc
19:47:47 -GitHub[themes]:#nikola- nikola-themes/master be97dd1 Chris Warrick: Switch screenshot pipeline to Puppeteer...
20:50:06 <polyzen> ChrisWarrick, do you plan on removing the python-nikola-doc splitpkgs
20:50:28 <ChrisWarrick> polyzen: one day, yes.  why do you ask?
20:50:48 <polyzen> because they're a pain in the butt :p
20:51:51 <polyzen> might be better off adding them as replaces to the nikola packages instead
20:52:14 <polyzen> replaces/replaced packages
21:11:15 <ChrisWarrick> polyzen: I expect to remove it whenever 7.8.13 is released. Which I plan to do on Tuesday.
21:11:31 <polyzen> /o/