{"thread":{"id":"11538","subject":"[STG PATCH] change usage string of refresh to not refer to removed options","startedAt":"2008-01-08T20:43:53Z","lastAt":"2008-01-09T07:20:36Z","messageCount":2,"participants":["Peter Oberndorfer","Karl Hasselström"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"64776","messageId":"200801082143.53125.kumbayo84@arcor.de","threadId":"11538","inReplyTo":null,"subject":"[STG PATCH] change usage string of refresh to not refer to removed options","fromName":"Peter Oberndorfer","fromEmail":"kumbayo84@arcor.de","sentAt":"2008-01-08T20:43:53Z","receivedAt":"2008-01-08T20:43:53Z","isPatch":true,"sender":{"key":"kumbayo84@arcor.de","avatar":"https://avatars.githubusercontent.com/u/1041267?v=4"},"body":"\nfor changing the patch author, commiter and description\n\nSigned-off-by: Peter Oberndorfer <kumbayo84@arcor.de>\n---\n\nI saw this while doing the refresh --index patch.\nThere was a discussion about bringing back refresh -e\nIf that also brings back this options, please do not apply the patch:-)\n\n stgit/commands/refresh.py |    8 +++-----\n 1 files changed, 3 insertions(+), 5 deletions(-)\n\ndiff --git a/stgit/commands/refresh.py b/stgit/commands/refresh.py\nindex 952b1b6..4695c62 100644\n--- a/stgit/commands/refresh.py\n+++ b/stgit/commands/refresh.py\n@@ -31,11 +31,9 @@ usage = \"\"\"%prog [options] [<files or dirs>]\n \n Include the latest tree changes in the current patch. This command\n generates a new GIT commit object with the patch details, the previous\n-one no longer being visible. The patch attributes like author,\n-committer and description can be changed with the command line\n-options. The '--force' option is useful when a commit object was\n-created with a different tool but the changes need to be included in\n-the current patch.\"\"\"\n+one no longer being visible. The '--force' option is useful\n+when a commit object was created with a different tool\n+but the changes need to be included in the current patch.\"\"\"\n \n directory = DirectoryHasRepository()\n options = [make_option('-f', '--force',\n-- \n1.5.4.rc2\n"},{"id":"64827","messageId":"20080109072036.GA28839@diana.vm.bytemark.co.uk","threadId":"11538","inReplyTo":"200801082143.53125.kumbayo84@arcor.de","subject":"Re: [STG PATCH] change usage string of refresh to not refer to removed options","fromName":"Karl Hasselström","fromEmail":"kha@treskal.com","sentAt":"2008-01-09T07:20:36Z","receivedAt":"2008-01-09T07:20:36Z","isPatch":true,"sender":{"key":"kha@treskal.com","avatar":"https://gravatar.com/avatar/f0120c734b5279b345075a28521e1ac66acb20c9913ffe9bf6ae97e53f7f3f13?d=mp&s=160"},"body":"Thanks! But ...\n\nOn 2008-01-08 21:43:53 +0100, Peter Oberndorfer wrote:\n\n> for changing the patch author, commiter and description\n\nthe first line in the commit message (which ends up being the Subject:\nline in the mail) should be a short and self-contained summary. No\nneed to resend just because of that, though -- I can fix it up.\n\n> I saw this while doing the refresh --index patch. There was a\n> discussion about bringing back refresh -e If that also brings back\n> this options, please do not apply the patch:-)\n\nResurrecting \"refresh -e\" is on my todo list. But until then, this is\na good fix.\n\n-- \nKarl Hasselström, kha@treskal.com\n      www.treskal.com/kalle\n"}]}