)]}'
{"elements/yum-minimal/pre-install.d/03-yum-cleanup":[{"author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"change_message_id":"2c3af07444ca26019042abb9190a7bfa0eeaed7d","unresolved":false,"context_lines":[{"line_number":26,"context_line":"# Fedora 24 has a much better way to handle just installing some"},{"line_number":27,"context_line":"# languages; see list mail.  We should support that at the right time."},{"line_number":28,"context_line":"if [[ $DISTRO_NAME \u003d\u003d \"fedora\" \u0026\u0026 $DIB_RELEASE -gt 23 ]]; then"},{"line_number":29,"context_line":"    echo \"locales have moved into packages for \u003e Fedora 23!\""},{"line_number":30,"context_line":"    echo \"This needs to be cleaned up\""},{"line_number":31,"context_line":"    echo \"See https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/message/K56XYH5AK4SBFQD36IFS2KDWUN2NY44E/\""},{"line_number":32,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"1a122d0e_1224740b","line":29,"updated":"2016-04-17 16:10:32.000000000","message":"I am not sure what a user it supposed to do about this message? If were just hoping to warn a user that there are known bugs relating to locales on f24 and greater maybe we should create an LP bug and print out a message to that effect \"Warning: There are known locale bugs with diskimage-builder and fedora 24 and greater. See \u003clp bug\u003e for more info.\"","commit_id":"d6ac3f13b9c9f9ea33fa6c048094f663f22e23f8"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e7a08ee9224f587fd58c96b790a74cfbd1cd06c0","unresolved":false,"context_lines":[{"line_number":26,"context_line":"# Fedora 24 has a much better way to handle just installing some"},{"line_number":27,"context_line":"# languages; see list mail.  We should support that at the right time."},{"line_number":28,"context_line":"if [[ $DISTRO_NAME \u003d\u003d \"fedora\" \u0026\u0026 $DIB_RELEASE -gt 23 ]]; then"},{"line_number":29,"context_line":"    echo \"locales have moved into packages for \u003e Fedora 23!\""},{"line_number":30,"context_line":"    echo \"This needs to be cleaned up\""},{"line_number":31,"context_line":"    echo \"See https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/message/K56XYH5AK4SBFQD36IFS2KDWUN2NY44E/\""},{"line_number":32,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"1a122d0e_15a459a4","line":29,"in_reply_to":"1a122d0e_1224740b","updated":"2016-04-18 05:06:08.000000000","message":"actually I missed an \"die\" message there that I meant to put in.  I know it\u0027s wrong, but I haven\u0027t fixed it (yet).  If someone beats me to it, great.\n\nlp bug, mail link, bugzilla -- I don\u0027t think it matters but OK","commit_id":"d6ac3f13b9c9f9ea33fa6c048094f663f22e23f8"},{"author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"change_message_id":"2c3af07444ca26019042abb9190a7bfa0eeaed7d","unresolved":false,"context_lines":[{"line_number":42,"context_line":"# now the archive has everything in it, and is about 100MiB.  Strip it"},{"line_number":43,"context_line":"# to just en_US (basically, this is the locale we support if you ssh"},{"line_number":44,"context_line":"# in, other than POSIX)"},{"line_number":45,"context_line":"localedef --delete-from-archive \\"},{"line_number":46,"context_line":"          $(localedef --list-archive | grep -v \u0027^en_US\u0027 | xargs)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"# This removes the locales from the archive index but doesn\u0027t rebuild"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"1a122d0e_521eecba","line":45,"updated":"2016-04-17 16:10:32.000000000","message":"Can we make this grep -v a var and change it to grep -ev - then a user can specify DIB_LOCALES\u003d\u0027*\u0027 and actually get all the locales.","commit_id":"d6ac3f13b9c9f9ea33fa6c048094f663f22e23f8"},{"author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"change_message_id":"5bb833788d9f00c0c718f47d9294462fa94bc9c8","unresolved":false,"context_lines":[{"line_number":42,"context_line":"# now the archive has everything in it, and is about 100MiB.  Strip it"},{"line_number":43,"context_line":"# to just en_US (basically, this is the locale we support if you ssh"},{"line_number":44,"context_line":"# in, other than POSIX)"},{"line_number":45,"context_line":"localedef --delete-from-archive \\"},{"line_number":46,"context_line":"          $(localedef --list-archive | grep -v \u0027^en_US\u0027 | xargs)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"# This removes the locales from the archive index but doesn\u0027t rebuild"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"1a122d0e_f283b8ca","line":45,"in_reply_to":"1a122d0e_521eecba","updated":"2016-04-17 16:11:50.000000000","message":"Actually, maybe we should just inspect the LANG var (and document this)?","commit_id":"d6ac3f13b9c9f9ea33fa6c048094f663f22e23f8"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e7a08ee9224f587fd58c96b790a74cfbd1cd06c0","unresolved":false,"context_lines":[{"line_number":42,"context_line":"# now the archive has everything in it, and is about 100MiB.  Strip it"},{"line_number":43,"context_line":"# to just en_US (basically, this is the locale we support if you ssh"},{"line_number":44,"context_line":"# in, other than POSIX)"},{"line_number":45,"context_line":"localedef --delete-from-archive \\"},{"line_number":46,"context_line":"          $(localedef --list-archive | grep -v \u0027^en_US\u0027 | xargs)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"# This removes the locales from the archive index but doesn\u0027t rebuild"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"1a122d0e_1540d9a4","line":45,"in_reply_to":"1a122d0e_521eecba","updated":"2016-04-18 05:06:08.000000000","message":"Generating en_US is the same as debootstrap and I think perfectly fine here as a common fallback for people logging in.  \n\nI\u0027d prefer not to complicate things with flags that will never be used; it\u0027s already too much of a hack \u0026 complex.\nOn \u003e F23, like debian, there will be packages to install extra languages.  So such a variable would be Centos specific in just a few months and, considering there\u0027s nobody asking for it, unused (the status quo is blowing away *all* locales and nobody cared until we got to some puppet jobs looking for the tools we removed, which led us down this path).","commit_id":"d6ac3f13b9c9f9ea33fa6c048094f663f22e23f8"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e7a08ee9224f587fd58c96b790a74cfbd1cd06c0","unresolved":false,"context_lines":[{"line_number":42,"context_line":"# now the archive has everything in it, and is about 100MiB.  Strip it"},{"line_number":43,"context_line":"# to just en_US (basically, this is the locale we support if you ssh"},{"line_number":44,"context_line":"# in, other than POSIX)"},{"line_number":45,"context_line":"localedef --delete-from-archive \\"},{"line_number":46,"context_line":"          $(localedef --list-archive | grep -v \u0027^en_US\u0027 | xargs)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"# This removes the locales from the archive index but doesn\u0027t rebuild"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"1a122d0e_35c23534","line":45,"in_reply_to":"1a122d0e_f283b8ca","updated":"2016-04-18 05:06:08.000000000","message":"definitely should not depend on the build env LANG.  this is passed via ssh, so someone in another language logging in and building manually could get a weird result.","commit_id":"d6ac3f13b9c9f9ea33fa6c048094f663f22e23f8"}]}