{"thread":{"id":"38133","subject":"[PATCH] git-svn: propset support v2","startedAt":"2014-12-06T22:29:18Z","lastAt":"2014-12-07T09:25:26Z","messageCount":2,"participants":["Alfred Perlstein","Eric Sunshine"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"253257","messageId":"20141206222918.GA91825@elvis.mu.org","threadId":"38133","inReplyTo":null,"subject":"[PATCH] git-svn: propset support v2","fromName":"Alfred Perlstein","fromEmail":"alfred@freebsd.org","sentAt":"2014-12-06T22:29:18Z","receivedAt":"2014-12-06T22:29:18Z","isPatch":true,"sender":{"key":"alfred@freebsd.org","avatar":"https://avatars.githubusercontent.com/u/629072?v=4"},"body":"I have incorporated Eric Wong's feedback into the git-svn propset support patch.\n\nIssues resolved:\n1) Test-case written.\n2) Remove dead code.\n3) Use shorter vars for formatting.\n4) Fix bool comparisons.\n5) Check for filesystem errors on write to .gitattribute file.\n6) Use command_output_pipe() instead of open my $fh, '-|', ...\n7) Refactor check for existing props.\n\nThere is a nit that I want to point out.  The code does not support adding props\nunless there are also content changes to the files as well.  You can see this in\nthe testcase.\n\nThat is still sufficient for many people's workflows (FreeBSD at\nleast).  So I am wondering if this is OK. \n\nI would gladly take any pointers to making it work with unchanged\nfiles either for a later diff or to wrap this up.\n\nThank you.\n\nAlfred Perlstein (1):\n  git-svn: Support for git-svn propset\n\n git-svn.perl               | 49 +++++++++++++++++++++++++++++++-\n perl/Git/SVN/Editor.pm     | 43 ++++++++++++++++++++++++++++\n t/t9148-git-svn-propset.sh | 71 ++++++++++++++++++++++++++++++++++++++++++++++\n 3 files changed, 162 insertions(+), 1 deletion(-)\n create mode 100755 t/t9148-git-svn-propset.sh\n\n-- \n2.1.2\n"},{"id":"253271","messageId":"CAPig+cQaP8KUFjFUqvxPZV1KiJePe_KisU4HSfZ1qVATOMriGw@mail.gmail.com","threadId":"38133","inReplyTo":"20141206222918.GA91825@elvis.mu.org","subject":"Re: [PATCH] git-svn: propset support v2","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2014-12-07T09:25:26Z","receivedAt":"2014-12-07T09:25:26Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Sat, Dec 6, 2014 at 5:29 PM, Alfred Perlstein <alfred@freebsd.org> wrote:\n> I have incorporated Eric Wong's feedback into the git-svn propset support patch.\n>\n> There is a nit that I want to point out.  The code does not support adding props\n> unless there are also content changes to the files as well.  You can see this in\n> the testcase.\n\nThis is an important nugget of information which would be worthwhile\nto mention in the commit message of the patch rather than here as mere\ncommentary.\n\n> That is still sufficient for many people's workflows (FreeBSD at\n> least).  So I am wondering if this is OK.\n>\n> I would gladly take any pointers to making it work with unchanged\n> files either for a later diff or to wrap this up.\n"}]}