06:59:35 .merge+ #1080 06:59:35 Added 06:59:49 .merges 06:59:49 -xmr-pr- #1027 #1074 #1075 #1080 18:38:24 .merge+ #1079 18:38:25 Added 18:41:18 So, today i found out that 1/4 of the repository is made by duplicate screenshots -_-. I'm not gonna lie, i screamed a lot: https://github.com/monero-project/monero-site/issues/1082 18:44:00 hmmmm 18:44:39 ErCiccione[m]: removing it won't change the repo size 18:45:37 it will remove it out of the working directory, but it'll remain in .git as it's historical 18:45:38 -xmr-pr- erciccione opened pull request #1083: Update screenshots of user guide 'verification-windows-beginner.html' ... 18:45:38 -xmr-pr- > https://github.com/monero-project/monero-site/pull/1083 18:45:38 -xmr-pr- erciccione opened issue #1082: Fix insane number of duplicated png files used for user guides 18:45:38 -xmr-pr- > https://github.com/monero-project/monero-site/issues/1082 18:46:07 so you might as well not bother 18:46:39 yes sure, the git repo is gone at this point. But would be 120MB less heavy on people's hard drive, no? 18:46:45 It fixes runaway repo growth though. 18:47:04 nope 18:48:01 moneromooo: sure, but only if we're adding new languages or new user guides 18:48:26 I don't disagree with finding a way to tell jekyll "if this png doesn't exist translated use the default one" 18:49:23 also one nice thing ErCiccione[m] is that if these pngs have the same hash then it'll only store one blob for the history 18:49:27 and not hundreds of them 18:49:33 O yeah i got it now. The thing is that right now to change one user guide's PNGs we have to change all screenshots for all languages. That will make the history heavier with each PR and doesn't make any sense 18:49:42 Oh, that's a good point. I forgot about this... 18:50:46 you might be able to do it in Liquid, ErCiccione[m] 18:51:31 yeah that make sense. But still, changing more than 100 pictures when changing one user guide is insane. I think i will put all png's in the resources folder and create a folder for the languages only if the screenshots get updated 18:51:50 liquid doesn't handle files very well, needs some tweaks 18:52:44 basically we have to tell it manually where to go get the files. The "if file exists" thing doesn't work with liquid afaik 18:53:43 https://stackoverflow.com/questions/16528783/check-for-existence-of-file-using-jekyll 18:55:06 looping through site.static_files is inefficient, but given that we jekyll build infrequently I don't think it's a problem 18:55:08 else you have to do it in Ruby :-P 18:55:10 Yeah i already read that for another PR i was working on (the video in the homepage. That's the only way to do it, but i remember being problematic for some reason 18:55:34 are there any other changes to the _config/nginx.conf other than the version numbers? 18:56:51 asymptotically: that config file is quite old, may be very different now. I think binaryfate said he will upload an updated one 18:59:21 anyway. I feel better knowing that the size is either not affected or there is nothing to do about it anymore. Thanks fluffypony :P 18:59:42 not sure why the `rewrite ^ /cli/montello.tar.bz2 redirect;` goes from https to http 19:00:07 Will change it anyway, but i can remove the "priority" label 19:00:39 we have the technology to nuke all of the images :D it'll just break all of the open PRs and upset anyone that has branched off 19:04:00 I don't think we have open PRs touching images of the user guides. Anyway it will require time, but imo is really necessary. 19:08:53 Now i remember what was the issue with looping through site.static_files for the promo video. Could be not relevant in this case, which would make things easier. 19:20:14 I updated the issue