)]}'
{"packstack/puppet/templates/mongodb.pp":[{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"d90cd1583d071c8b350b00ed5dabe1c0fb8a82a3","unresolved":false,"context_lines":[{"line_number":3,"context_line":""},{"line_number":4,"context_line":"# The MongoDB config files differ between versions"},{"line_number":5,"context_line":"if ($::operatingsystem \u003d\u003d \u0027fedora\u0027 and versioncmp($::operatingsystemrelease, \u002722\u0027) \u003e\u003d 0 or"},{"line_number":6,"context_line":"  $::operatingsystem !\u003d \u0027fedora\u0027 and versioncmp($::operatingsystemrelease, \u00277.0\u0027) \u003e\u003d 0) {"},{"line_number":7,"context_line":"  if $use_epel \u003d\u003d \u0027y\u0027 {"},{"line_number":8,"context_line":"     $config_file \u003d \u0027/etc/mongod.conf\u0027"},{"line_number":9,"context_line":"  } else  {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"5aa745d1_87948e37","line":6,"updated":"2015-04-16 14:20:41.000000000","message":"I think the conditional should contain more brackets:\n\n  if (($::operatingsystem \u003d\u003d \u0027fedora\u0027 and versioncmp($::operatingsystemrelease, \u002722\u0027) \u003e\u003d 0) or\t($::operatingsystem !\u003d \u0027fedora\u0027 and versioncmp($::operatingsystemrelease, \u00277.0\u0027) \u003e\u003d 0))\n\nand also you can incorporate $use_epel to one conditional, eg.:\n  if (\n    ($::operatingsystem \u003d\u003d \u0027fedora\u0027 and versioncmp($::operatingsystemrelease, \u002722\u0027) \u003e\u003d 0) \n    or\t\n    ($::operatingsystem !\u003d \u0027fedora\u0027 and versioncmp($::operatingsystemrelease, \u00277.0\u0027) \u003e\u003d 0 and $use_epel \u003d\u003d \u0027y\u0027)\n  ) {\n    $config_file \u003d \u0027/etc/mongod.conf\u0027\n  } else {\n    $config_file \u003d \u0027/etc/mongodb.conf\u0027\n  }","commit_id":"53eef3ced6960cf331f39e8f2d7b735d10b8e834"}]}