Thursday, 2019-01-31

*** tosky has quit IRC00:46
*** jamesmcarthur has quit IRC01:21
*** jamesmcarthur has joined #openstack-doc01:23
*** jamesmcarthur has quit IRC01:27
*** _fragatina has quit IRC01:56
*** jamesmcarthur has joined #openstack-doc01:58
*** jamesmcarthur has quit IRC01:58
*** jamesmcarthur has joined #openstack-doc02:22
*** dims has quit IRC02:38
*** jamesmcarthur has quit IRC02:47
*** dims has joined #openstack-doc02:55
*** _fragatina has joined #openstack-doc04:18
*** _fragatina has quit IRC04:19
*** _fragatina has joined #openstack-doc04:19
*** _fragatina has quit IRC04:21
*** _fragatina has joined #openstack-doc04:21
*** _fragatina has quit IRC04:27
*** _fragatina has joined #openstack-doc04:28
*** _fragatina has quit IRC04:31
*** _fragatina has joined #openstack-doc04:31
*** _fragatina has quit IRC04:32
*** _fragatina has joined #openstack-doc04:33
*** _fragatina has quit IRC04:41
*** _fragatina has joined #openstack-doc04:42
*** _fragatina has quit IRC04:42
*** _fragatina has joined #openstack-doc04:43
*** _fragatina has quit IRC04:44
*** _fragatina has joined #openstack-doc04:44
*** jamesmcarthur has joined #openstack-doc04:48
*** jamesmcarthur has quit IRC04:54
*** _fragatina has quit IRC05:11
*** _fragatina has joined #openstack-doc05:11
*** _fragatina has quit IRC05:16
*** _fragatina has joined #openstack-doc05:16
*** _fragatina has quit IRC05:19
*** _fragatina has joined #openstack-doc05:20
*** _fragatina has quit IRC05:38
*** _fragatina has joined #openstack-doc05:39
*** jamesmcarthur has joined #openstack-doc06:50
*** jamesmcarthur has quit IRC06:54
*** kopecmartin|off is now known as kopecmartin07:30
*** tosky has joined #openstack-doc08:34
*** jamesmcarthur has joined #openstack-doc08:51
*** jamesmcarthur has quit IRC08:55
*** jamesmcarthur has joined #openstack-doc10:52
*** jamesmcarthur has quit IRC10:56
*** pkovar has joined #openstack-doc11:54
*** pkovar has quit IRC12:03
*** pkovar has joined #openstack-doc12:20
*** jamesmcarthur has joined #openstack-doc12:52
*** jamesmcarthur has quit IRC12:57
*** pkovar has quit IRC12:58
*** pkovar has joined #openstack-doc13:03
*** jamesmcarthur has joined #openstack-doc13:15
*** jamesmcarthur has quit IRC13:20
*** jamesmcarthur has joined #openstack-doc13:46
*** jamesmcarthur has quit IRC13:51
*** pcaruana has quit IRC13:52
*** pcaruana has joined #openstack-doc14:02
*** dmacpher has quit IRC14:45
*** dmacpher has joined #openstack-doc15:22
*** eumel8 has joined #openstack-doc15:55
eumel8o/15:56
pkovar\o15:58
eumel8Hello Dr. Doc ;)15:58
*** jamesmcarthur has joined #openstack-doc16:00
*** _fragatina has quit IRC16:03
*** _fragatina has joined #openstack-doc16:04
*** _fragatina has quit IRC16:04
*** _fragatina has joined #openstack-doc16:05
*** _fragatina has quit IRC16:10
pkovarnothing in agenda for today's meeting.16:13
pkovarshould we cancel?16:13
eumel8I have something16:14
eumel8but not on agenda16:14
eumel8if nobody is there from i18n16:14
*** jamesmcarthur has quit IRC16:16
*** jamesmcarthur_ has joined #openstack-doc16:16
*** jamesmcarthur has joined #openstack-doc16:17
pkovaraha, okay16:19
*** jamesmcarthur_ has quit IRC16:20
pkovar>>>>>> #openstack-meeting16:34
*** efried has joined #openstack-doc16:43
efriedo/16:43
efriedI'm mucking with the contributor-guide and noticing some weirdnesses.16:43
efried1) Attempting to build the contributor-guide via `tox -e docs` is failing on some syntax stuff. Assuming it's just because we've made some common build tool stricter, I can go clean those up.16:44
efried2) Whenever I build, all the source rst files are getting *modified*. A block like http://paste.openstack.org/show/744325/ is being added, to each file, each time I build.16:46
efriedstephenfin, dhellmann: ^ (or feel free to point me to the right person)16:47
dhellmannefried : that looks like the translation stuff that was added to the build. I don't know how that works, but AJaeger or eumel8 might16:49
efriedack, thanks16:49
dhellmannefried : you might want to look at the tox file to see if there are other targets that don't run that step.16:50
efrieddhellmann: Doesn't look like it. But in any case that seems broken to me.16:50
dhellmannyeah, that seems very much designed to just run in the CI system and not locally16:51
eumel8efried: its more a feature then a bug. the rst files are modifified with the language header after translation build16:51
efriedeumel8: Fair enough, but a) shouldn't those headers go into a doc/build artifact rather than the source; and b) shouldn't the tool detect when the header is already there and not add it again?16:52
efrieda) is important for the sake of being able to develop actual changes to the source. The way it is now, I would have to finish making/testing my changes, then go in and edit every rst file to remove the headers before doing my commit.16:54
eumel8efried: a) no, thats a dynamic build dependly on the translated files or books. and b) don't know if that happen, I'll make a git checkout after tox build locally to get the normal files back16:55
efriedeumel8: Yes, there are ways to do a checkout or patch to reverse the header adds, but I have to be careful not to overwrite the actual changes I wanted to make when I do that.16:56
efriedI'll propose a patch with dhellmann's suggestion of a different tox target that doesn't add the headers.16:56
dhellmannit should be possible to do the data injection without modifying the rst files by creating a special directive16:57
dhellmannor using the sphinx features for inserting a common prefix/suffix to files as it builds them (in memory)16:57
efrieddhellmann: It's not quite common - would have to parameterize the basename of the source file - can that be done?16:58
dhellmann"not quite common"?16:58
dhellmannis it different in each file?16:58
efriedyes, if you look at the paste - the basename of the file is included in the links.16:58
efriedso it's the same *shape*, but the basename would have to be %s'd or similar.16:59
dhellmannah, I see16:59
dhellmannyeah, a directive then16:59
efriednot just the basename, actually, the path too.16:59
dhellmannyeah, I think all of that info would be available inside a directive17:00
efriedThis is going to be over my head. For now I'm just going to propose a change that a) adds a tox target that doesn't insert the headers; and b) fixes the build errors; so that I can c) get on with the actual content changes I was trying to make.17:00
dhellmannyeah, that's fine17:00
efriedwhom should I pester about the 'directive' thing?17:00
dhellmannAJaeger and eumel8 own this stuff afaik17:01
eumel8should be here: https://github.com/openstack/openstackdocstheme/blob/master/bin/docstheme-build-translated.sh17:02
dhellmannI guess another way to do this would have been to do it in the page templates17:03
dhellmannthat would keep it out of the content entirely17:03
*** kopecmartin is now known as kopecmartin|off17:04
dhellmannand wouldn't rely on us remembering to add a directive in every page17:04
efriedeumel8: I see `recover_rst_files` in that script, which at a glance looks like it should be restoring the unadulterated sources; but it's clearly not working.17:05
efriedyuh, git status is showing all the backup files still, which it wouldn't if that guy was working.17:06
*** _fragatina has joined #openstack-doc17:08
efriedoh. It's because it's not running :P17:09
efriedBecause the build is still failing.17:09
eumel8yes17:09
efriedShould put that in an exit trap.17:09
*** openstackgerrit has joined #openstack-doc17:31
openstackgerritEric Fried proposed openstack/openstackdocstheme master: Exit trap cleanup in docstheme-build-translated.sh  https://review.openstack.org/63428417:31
efrieddhellmann, eumel8: ^17:31
*** jamesmcarthur has quit IRC18:46
*** jamesmcarthur has joined #openstack-doc19:14
*** pkovar has quit IRC19:18
*** pcaruana has quit IRC19:30
*** jamesmcarthur has quit IRC19:36
*** jamesmcarthur has joined #openstack-doc19:47
openstackgerritMerged openstack/openstackdocstheme master: Exit trap cleanup in docstheme-build-translated.sh  https://review.openstack.org/63428419:49
AJaegerefried: stephenfin wanted to enhance sphinx to do this but hasn't had time yet. If you want to help, you're more than welcome19:52
AJaegerthanks, efried for your change19:52
*** jamesmcarthur has quit IRC20:42
*** jamesmcarthur has joined #openstack-doc20:44
*** jamesmcarthur has quit IRC20:49
*** jamesmcarthur has joined #openstack-doc20:51
*** efried has quit IRC21:02
*** efried has joined #openstack-doc21:02
efriedAJaeger: Roger that. It's way outside my area of expertise, but maybe with stephenfin's guidance I could make a start. Not going to be able to make it a high priority though :P21:03
*** jamesmcarthur has quit IRC21:04
*** efried has quit IRC21:16
*** efried has joined #openstack-doc21:17
*** jamesmcarthur has joined #openstack-doc21:51
*** jamesmcarthur has quit IRC22:08
*** jamesmcarthur has joined #openstack-doc22:26
openstackgerritEric Fried proposed openstack/contributor-guide master: Tutorial: changes in a series  https://review.openstack.org/63433322:28
efrieddhellmann, edmondsw: ^22:29
edmondswefried sweet, thanks22:31
*** jamesmcarthur has quit IRC22:41
*** jamesmcarthur has joined #openstack-doc23:22
efriedCan a guy edit .po files, or will they be blown away next time "Zanata" happens (whatever that is)?23:22
*** jamesmcarthur has quit IRC23:26

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!