)]}'
{
  "commit": "7710e7fc273dd9af6799ea565bbd21ce03745a44",
  "tree": "46bee50accf6f4c0ada1d89382ab92de5d0eebea",
  "parents": [
    "2cb3db338a17bd26fef1f56fc155be5d01a6ef7e"
  ],
  "author": {
    "name": "Ian Wienand",
    "email": "iwienand@redhat.com",
    "time": "Wed Aug 27 16:15:32 2014 +1000"
  },
  "committer": {
    "name": "Ian Wienand",
    "email": "iwienand@redhat.com",
    "time": "Tue Jan 12 09:23:35 2016 +1100"
  },
  "message": "Only use lsb_release for distro tags\n\nThe existing GetOSVersion has a lot of unused code which is wrong in\nseveral ways\n\n - the only path tested in upstream CI is with lsb_release, because\n   it\u0027s pre-installed on all nodes\n\n - the /etc/redhat-release checking probably still works, but is\n   unnecessary\n\n - If using lsb_release, os_UPDATE has never actually been set.\n\n - the /etc/SuSE-release branch checking is broken if the lsb package\n   is actually installed.  lsb checking does not set os_UPDATE but yet\n   the SuSE DISTRO setting relies on this to set a patch level (and so\n   does some of the rpm tags).  SuSE 11 is up to update 3, but the rpm\n   matching is stuck hard-coded to update 2.  I\u0027m guessing\n   installation is actually broken there.\n\n - the debian checking branch is broken.  The VERSION tags have been\n   removed and were not supposed to be relied on anyway (see notes in\n   [1])\n\nThis simplifies things:\n\n - remove OSX checking (moved here after discussions in\n   I31d0fdd30928ecc8d959a95838b1d3affd28ac6f)\n\n - only use the output of lsb_release.\n\n - A small best-effort check to pre-install lsb packages if not\n   detected (that avoids chicken-egg-problem of package-install\n   wrappers relying on os_* flags).\n\n - The unset os_UPDATE is removed.  It\u0027s only previous use was for\n   setting separate suse versions in the DISTRO element for matching\n   during package installs (since removed)\n\n - DISTRO setting is modified to use the parts of os_RELEASE it wants.\n   Per-above, this is the correct place to parse out specifics.\n\n - Call out the is_* functions, which are a better way to detect\n   platforms\n\n - Export the variables as read-only, since they shouldn\u0027t be reset\n\n[1] http://sources.debian.net/src/base-files/7.5/debian/changelog/\n\nChange-Id: I46a2c36d95327087085df07cb797eb91249a893c\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "fc6f80dad4f143f370e145eaa8eacf6024574c84",
      "old_mode": 33261,
      "old_path": "clean.sh",
      "new_id": "57e3b5edf806fa33072dbbc05d550bcfa8854b09",
      "new_mode": 33261,
      "new_path": "clean.sh"
    },
    {
      "type": "modify",
      "old_id": "dbf605cdd843d8393ec506962636054a629e97c3",
      "old_mode": 33188,
      "old_path": "functions-common",
      "new_id": "e6f081ce0a4b8173719151f852483af0473e8698",
      "new_mode": 33188,
      "new_path": "functions-common"
    },
    {
      "type": "modify",
      "old_id": "5e4a4f831007c121a609b8c1aabb0bd372ac7138",
      "old_mode": 33261,
      "old_path": "stack.sh",
      "new_id": "697dce156f7dd1e341a10aa3647126393f02aa4c",
      "new_mode": 33261,
      "new_path": "stack.sh"
    },
    {
      "type": "modify",
      "old_id": "b49164b22ab600384bb03d1bb241519278a1c492",
      "old_mode": 33261,
      "old_path": "tools/create-stack-user.sh",
      "new_id": "c0b7ac70aa22dab3af2d78e88e577f3fd1a68cc2",
      "new_mode": 33261,
      "new_path": "tools/create-stack-user.sh"
    },
    {
      "type": "modify",
      "old_id": "433206e8ae17d5f140eb1f647c4f6096772edf44",
      "old_mode": 33261,
      "old_path": "tools/info.sh",
      "new_id": "c056fa73f898162347ce8dee5df5051a939ee41e",
      "new_mode": 33261,
      "new_path": "tools/info.sh"
    },
    {
      "type": "modify",
      "old_id": "031f8a8ecac570a1145e47735b9e6cac2e18a340",
      "old_mode": 33261,
      "old_path": "tools/install_prereqs.sh",
      "new_id": "8895e1e77ce0c285a810ada02c345fe5ec951af5",
      "new_mode": 33261,
      "new_path": "tools/install_prereqs.sh"
    },
    {
      "type": "modify",
      "old_id": "8eded837fd6588956bef52b0fa60f5fc57775421",
      "old_mode": 33261,
      "old_path": "unstack.sh",
      "new_id": "c4dfdc8d9e33bda1476cde2c3604008f23623f6e",
      "new_mode": 33261,
      "new_path": "unstack.sh"
    }
  ]
}
