{"action":"created","issue":{"url":"https://api.github.com/repos/sbabic/swupdate/issues/12","labels_url":"https://api.github.com/repos/sbabic/swupdate/issues/12/labels{/name}","comments_url":"https://api.github.com/repos/sbabic/swupdate/issues/12/comments","events_url":"https://api.github.com/repos/sbabic/swupdate/issues/12/events","html_url":"https://github.com/sbabic/swupdate/pull/12","id":122270088,"number":12,"title":"Changes to support extracting file permissions, ownership, time, etc","user":{"login":"BrendanSimon","id":472245,"avatar_url":"https://avatars.githubusercontent.com/u/472245?v=3","gravatar_id":"","url":"https://api.github.com/users/BrendanSimon","html_url":"https://github.com/BrendanSimon","followers_url":"https://api.github.com/users/BrendanSimon/followers","following_url":"https://api.github.com/users/BrendanSimon/following{/other_user}","gists_url":"https://api.github.com/users/BrendanSimon/gists{/gist_id}","starred_url":"https://api.github.com/users/BrendanSimon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BrendanSimon/subscriptions","organizations_url":"https://api.github.com/users/BrendanSimon/orgs","repos_url":"https://api.github.com/users/BrendanSimon/repos","events_url":"https://api.github.com/users/BrendanSimon/events{/privacy}","received_events_url":"https://api.github.com/users/BrendanSimon/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-15T13:17:40Z","updated_at":"2015-12-15T13:31:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/sbabic/swupdate/pulls/12","html_url":"https://github.com/sbabic/swupdate/pull/12","diff_url":"https://github.com/sbabic/swupdate/pull/12.diff","patch_url":"https://github.com/sbabic/swupdate/pull/12.patch"},"body":"Hi Stefano,\r\n\r\nI've made changes to support extracting file permissions, ownership, time, etc.\r\nThe code looks ok to me, however the extracted files do not have the expected attributes.  i.e. everything is still owned by root, times are not preserved, etc.\r\n\r\nI was referring to the `A Complete Extractor` section of this [page](https://github.com/libarchive/libarchive/wiki/Examples).\r\n\r\nHopefully you can see if there is something I may have missed.\r\n\r\nThanks,\r\nBrendan."},"comment":{"url":"https://api.github.com/repos/sbabic/swupdate/issues/comments/164765439","html_url":"https://github.com/sbabic/swupdate/pull/12#issuecomment-164765439","issue_url":"https://api.github.com/repos/sbabic/swupdate/issues/12","id":164765439,"user":{"login":"sbabic","id":1411982,"avatar_url":"https://avatars.githubusercontent.com/u/1411982?v=3","gravatar_id":"","url":"https://api.github.com/users/sbabic","html_url":"https://github.com/sbabic","followers_url":"https://api.github.com/users/sbabic/followers","following_url":"https://api.github.com/users/sbabic/following{/other_user}","gists_url":"https://api.github.com/users/sbabic/gists{/gist_id}","starred_url":"https://api.github.com/users/sbabic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sbabic/subscriptions","organizations_url":"https://api.github.com/users/sbabic/orgs","repos_url":"https://api.github.com/users/sbabic/repos","events_url":"https://api.github.com/users/sbabic/events{/privacy}","received_events_url":"https://api.github.com/users/sbabic/received_events","type":"User","site_admin":false},"created_at":"2015-12-15T13:31:37Z","updated_at":"2015-12-15T13:31:37Z","body":"Hi Brendan,\n\nOn 15/12/2015 14:17, Brendan Simon wrote:\n> Hi Stefano,\n> \n> I've made changes to support extracting file permissions, ownership,\n> time, etc.\n> The code looks ok to me, however the extracted files do not have the\n> expected attributes. i.e. everything is still owned by root, times are\n> not preserved, etc.\n> \n> I was referring to the |A Complete Extractor| section of this page\n> <https://github.com/libarchive/libarchive/wiki/Examples>.\n> \n> Hopefully you can see if there is something I may have missed.\n> \n\nPlease send your patch and/or questions to the ML list, not via github.\nSee file README how to submit patches, thanks !\n\nBest regards,\nStefano\n\n> Thanks,\n> Brendan.\n> \n> ------------------------------------------------------------------------\n> \n> \n>         You can view, comment on, or merge this pull request online at:\n> \n>   https://github.com/sbabic/swupdate/pull/12\n> \n> \n>         Commit Summary\n> \n>   * Add sw-description archive extract options.\n>   * Fix indentation (space => tab) for previous commit.\n> \n> \n>         File Changes\n> \n>   * *M* handlers/archive_handler.c\n>     <https://github.com/sbabic/swupdate/pull/12/files#diff-0> (17)\n>   * *M* include/swupdate.h\n>     <https://github.com/sbabic/swupdate/pull/12/files#diff-1> (4)\n>   * *M* parser/parser.c\n>     <https://github.com/sbabic/swupdate/pull/12/files#diff-2> (5)\n> \n> \n>         Patch Links:\n> \n>   * https://github.com/sbabic/swupdate/pull/12.patch\n>   * https://github.com/sbabic/swupdate/pull/12.diff\n> \n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/sbabic/swupdate/pull/12>.\n> \n\n\n-- \n=====================================================================\nDENX Software Engineering GmbH,      Managing Director: Wolfgang Denk\nHRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany\nPhone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de\n=====================================================================\n"}}