{"thread":{"id":"39640","subject":"[PATCH] Fix power checking on OS X","startedAt":"2015-06-16T07:36:07Z","lastAt":"2015-06-16T17:57:16Z","messageCount":4,"participants":["Panagiotis Astithas","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"263916","messageId":"1434440167-1046-1-git-send-email-pastith@gmail.com","threadId":"39640","inReplyTo":null,"subject":"[PATCH] Fix power checking on OS X","fromName":"Panagiotis Astithas","fromEmail":"pastith@gmail.com","sentAt":"2015-06-16T07:36:07Z","receivedAt":"2015-06-16T07:36:07Z","isPatch":true,"sender":{"key":"pastith@gmail.com","avatar":"https://avatars.githubusercontent.com/u/49118?v=4"},"body":"The output of \"pmset -g batt\" changed at some point from\n\"Currently drawing from 'AC Power'\" to the slightly different\n\"Now drawing from 'AC Power'\". Starting the match from \"drawing\"\nmakes the check work in both old and new versions of OS X.\n\nSigned-off-by: Panagiotis Astithas \n---\n contrib/hooks/pre-auto-gc-battery | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/contrib/hooks/pre-auto-gc-battery b/contrib/hooks/pre-auto-gc-battery\nindex 9d0c2d1..6a2cdeb 100755\n--- a/contrib/hooks/pre-auto-gc-battery\n+++ b/contrib/hooks/pre-auto-gc-battery\n@@ -33,7 +33,7 @@ elif grep -q \"AC Power \\+: 1\" /proc/pmu/info 2>/dev/null\n then\n \texit 0\n elif test -x /usr/bin/pmset && /usr/bin/pmset -g batt |\n-\tgrep -q \"Currently drawing from 'AC Power'\"\n+\tgrep -q \"drawing from 'AC Power'\"\n then\n \texit 0\n fi\n-- \n2.4.1\n"},{"id":"263961","messageId":"xmqqr3pb5zl1.fsf@gitster.dls.corp.google.com","threadId":"39640","inReplyTo":"1434440167-1046-1-git-send-email-pastith@gmail.com","subject":"Re: [PATCH] Fix power checking on OS X","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2015-06-16T17:32:10Z","receivedAt":"2015-06-16T17:32:10Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Panagiotis Astithas writes:\n\n> The output of \"pmset -g batt\" changed at some point from\n> \"Currently drawing from 'AC Power'\" to the slightly different\n> \"Now drawing from 'AC Power'\". Starting the match from \"drawing\"\n> makes the check work in both old and new versions of OS X.\n>\n> Signed-off-by: Panagiotis Astithas \n> ---\n\nHmph, is there any difference between this one and the original one\nthat I already queued several days ago?\n\n> contrib/hooks/pre-auto-gc-battery | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/contrib/hooks/pre-auto-gc-battery b/contrib/hooks/pre-auto-gc-battery\n> index 9d0c2d1..6a2cdeb 100755\n> --- a/contrib/hooks/pre-auto-gc-battery\n> +++ b/contrib/hooks/pre-auto-gc-battery\n> @@ -33,7 +33,7 @@ elif grep -q \"AC Power \\+: 1\" /proc/pmu/info 2>/dev/null\n> then\n> \texit 0\n> elif test -x /usr/bin/pmset && /usr/bin/pmset -g batt |\n> -\tgrep -q \"Currently drawing from 'AC Power'\"\n> +\tgrep -q \"drawing from 'AC Power'\"\n> then\n> \texit 0\n> fi\n"},{"id":"263962","messageId":"CACvVhd6j_zWpvfXti1=U8iOaXE+oTUtmjrHu39VR12UNBxkY4A@mail.gmail.com","threadId":"39640","inReplyTo":"xmqqr3pb5zl1.fsf@gitster.dls.corp.google.com","subject":"Re: [PATCH] Fix power checking on OS X","fromName":"Panagiotis Astithas","fromEmail":"pastith@gmail.com","sentAt":"2015-06-16T17:35:31Z","receivedAt":"2015-06-16T17:35:31Z","isPatch":true,"sender":{"key":"pastith@gmail.com","avatar":"https://avatars.githubusercontent.com/u/49118?v=4"},"body":"On Tue, Jun 16, 2015 at 8:32 PM, Junio C Hamano wrote:\n> Panagiotis Astithas writes:\n>\n>> The output of \"pmset -g batt\" changed at some point from\n>> \"Currently drawing from 'AC Power'\" to the slightly different\n>> \"Now drawing from 'AC Power'\". Starting the match from \"drawing\"\n>> makes the check work in both old and new versions of OS X.\n>>\n>> Signed-off-by: Panagiotis Astithas \n>> ---\n>\n> Hmph, is there any difference between this one and the original one\n> that I already queued several days ago?\n\nNone, I was just following the process outlined in SubmittingPatches\nthat mentions a separate email after the discussion is over.\n"},{"id":"263963","messageId":"CAPc5daXuJWyi2Zc-Su9w1Rn38wjTkZNo4zML8m3Hq_EwoBjnfg@mail.gmail.com","threadId":"39640","inReplyTo":"CACvVhd6j_zWpvfXti1=U8iOaXE+oTUtmjrHu39VR12UNBxkY4A@mail.gmail.com","subject":"Re: [PATCH] Fix power checking on OS X","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2015-06-16T17:57:16Z","receivedAt":"2015-06-16T17:57:16Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ah, thanks. Not many people do that, so I was wondering what was going on ;-).\n\nOn Tue, Jun 16, 2015 at 10:35 AM, Panagiotis Astithas wrote:\n> On Tue, Jun 16, 2015 at 8:32 PM, Junio C Hamano wrote:\n>> Panagiotis Astithas writes:\n>>\n>>> The output of \"pmset -g batt\" changed at some point from\n>>> \"Currently drawing from 'AC Power'\" to the slightly different\n>>> \"Now drawing from 'AC Power'\". Starting the match from \"drawing\"\n>>> makes the check work in both old and new versions of OS X.\n>>>\n>>> Signed-off-by: Panagiotis Astithas \n>>> ---\n>>\n>> Hmph, is there any difference between this one and the original one\n>> that I already queued several days ago?\n>\n> None, I was just following the process outlined in SubmittingPatches\n> that mentions a separate email after the discussion is over.\n"}]}