)]}'
{"lib/facter/os_package_type.rb":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"1ae73c65b7b1517c1e4a64511dec0924a3f7163f","unresolved":false,"context_lines":[{"line_number":27,"context_line":"        os_package_type \u003d \u0027ubuntu\u0027"},{"line_number":28,"context_line":"      end"},{"line_number":29,"context_line":"    else"},{"line_number":30,"context_line":"      os_package_type \u003d \u0027rpm\u0027"},{"line_number":31,"context_line":"    end"},{"line_number":32,"context_line":"    os_package_type"},{"line_number":33,"context_line":"  end"}],"source_content_type":"text/x-ruby","patch_set":6,"id":"fa80f949_e84bdaae","line":30,"updated":"2015-11-04 15:37:38.000000000","message":"what about other distros?\nIf you look at our params.pp, we fail on other distros.","commit_id":"94721ec7562c5df0a0f2c235f82b7d83891c4f81"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"3938102ec994bcc03020bb3a366b2155250c4048","unresolved":false,"context_lines":[{"line_number":27,"context_line":"        os_package_type \u003d \u0027ubuntu\u0027"},{"line_number":28,"context_line":"      end"},{"line_number":29,"context_line":"    else"},{"line_number":30,"context_line":"      os_package_type \u003d \u0027rpm\u0027"},{"line_number":31,"context_line":"    end"},{"line_number":32,"context_line":"    os_package_type"},{"line_number":33,"context_line":"  end"}],"source_content_type":"text/x-ruby","patch_set":6,"id":"fa80f949_b0d9b885","line":30,"in_reply_to":"fa80f949_e84bdaae","updated":"2015-11-04 16:28:33.000000000","message":"This was address in a later patch by switching to an \u0027unknown\u0027 for osfamily not in [\u0027Debian\u0027,\u0027 RedHat\u0027,\u0027Solaris\u0027] so we could use that to fail on.","commit_id":"94721ec7562c5df0a0f2c235f82b7d83891c4f81"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"9af531d5fbd2bc7575ce0ea4d86600f44eaa2c73","unresolved":false,"context_lines":[{"line_number":15,"context_line":"# value of $::operatingsystem."},{"line_number":16,"context_line":"#"},{"line_number":17,"context_line":"# Having the below snipets helps simplifying checks within individual project"},{"line_number":18,"context_line":"# manfiests, so that we can just reuse $::os_package_type directly without"},{"line_number":19,"context_line":"# having to also check if it contains a value, then check for the content of"},{"line_number":20,"context_line":"# $::operatingsystem (ie: what\u0027s below factors the check once and for all)."},{"line_number":21,"context_line":"Facter.add(\u0027os_package_type\u0027) do"}],"source_content_type":"text/x-ruby","patch_set":7,"id":"fa80f949_143f51d7","line":18,"updated":"2015-11-04 19:38:14.000000000","message":"typo: manifests","commit_id":"69b886f9445d961cb0a9fdfcd05d0af689de2c67"}]}