{"thread":{"id":"32057","subject":"overriding/removing inherited credential.helper","startedAt":"2012-11-10T15:12:50Z","lastAt":"2013-02-07T18:38:06Z","messageCount":4,"participants":["乙酸鋰","Ted Zlatanov","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"202755","messageId":"CAHtLG6QaHOOYgVFPyOWo44-jTX__cd0dCyu+vs+Uf4_U-HxySw@mail.gmail.com","threadId":"32057","inReplyTo":null,"subject":"overriding/removing inherited credential.helper","fromName":"乙酸鋰","fromEmail":"ch3cooli@gmail.com","sentAt":"2012-11-10T15:12:50Z","receivedAt":"2012-11-10T15:12:50Z","isPatch":false,"sender":{"key":"ch3cooli@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2804967?v=4"},"body":"In credential.c, line 67:\n\n if (!strcmp(key, \"helper\"))\n string_list_append(&c->helpers, value);\n\nIn global config, I add one credential helper.\nBut I do not want to use any credential helper in a specific repository.\nCurrently there is no way in local config to override and remove\ninherited credential helper.\nOf course, I do not want to change global config.\n\nCould you suggest a patch?\n"},{"id":"208920","messageId":"87pq0cchsz.fsf@lifelogs.com","threadId":"32057","inReplyTo":"CAHtLG6QaHOOYgVFPyOWo44-jTX__cd0dCyu+vs+Uf4_U-HxySw@mail.gmail.com","subject":"Re: overriding/removing inherited credential.helper, Do not add an empty value from config credential.helper","fromName":"Ted Zlatanov","fromEmail":"tzz@lifelogs.com","sentAt":"2013-02-07T14:36:12Z","receivedAt":"2013-02-07T14:36:12Z","isPatch":false,"sender":{"key":"tzz@lifelogs.com","avatar":"https://avatars.githubusercontent.com/u/67764?v=4"},"body":"On Sat, 10 Nov 2012 23:12:50 +0800 乙酸鋰 wrote: \n\n> In credential.c, line 67:\n> if (!strcmp(key, \"helper\"))\n>\t string_list_append(&c->helpers, value);\n\n> In global config, I add one credential helper.\n> But I do not want to use any credential helper in a specific repository.\n> Currently there is no way in local config to override and remove\n> inherited credential helper.\n> Of course, I do not want to change global config.\n\n> Could you suggest a patch?\n\nOn Sat, 10 Nov 2012 23:08:04 +0800 乙酸鋰 wrote: \n\n> Below is current git message when a local config credential.helper has\n> an empty value. Please skip an empty value.\n\n> $ git push --force origin master\n> git: 'credential-' is not a git command. See 'git --help'.\n\n> Did you mean this?\n>\t credential\n> Total 0 (delta 0), reused 0 (delta 0)\n> To https://user@github.com/user/myrepo.git\n> + d23aa6a...3405990 master -> master (forced update)\n\nI would like that too (needed it today). Maybe the empty string (as\nsuggested) or \"none\" could be acceptable.\n\n\"none\" seems friendlier and is very unlikely to collide with an actual\ncredential helper.\n\nTed\n"},{"id":"208927","messageId":"7vvca47zl3.fsf@alter.siamese.dyndns.org","threadId":"32057","inReplyTo":"87pq0cchsz.fsf@lifelogs.com","subject":"Re: overriding/removing inherited credential.helper, Do not add an empty value from config credential.helper","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-02-07T18:23:20Z","receivedAt":"2013-02-07T18:23:20Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ted Zlatanov writes:\n\n>> Below is current git message when a local config credential.helper has\n>> an empty value. Please skip an empty value.\n>\n>> $ git push --force origin master\n>> git: 'credential-' is not a git command. See 'git --help'.\n>> Did you mean this?\n>>\t credential\n\nWhy isn't \"do not add empty string, or any random string that ends\nup referring to a helper that you do not have\" a solution?\n\n>> Total 0 (delta 0), reused 0 (delta 0)\n>> To https://user@github.com/user/myrepo.git\n>> + d23aa6a...3405990 master -> master (forced update)\n>\n> I would like that too (needed it today). Maybe the empty string (as\n> suggested) or \"none\" could be acceptable.\n\nWhatever you do, I do not think introducing a per-variable hack\n\n\t[credential]\n \thelper = none ;# or \"helper = clear\"\n helper = mine ;# this is the only thing I use\n\nlike that is a sane way to go. \"Clear everything you saw so far\"\nwould be useful for variables other than \"credential.helper\";\nshouldn't it be done by adding a general syntax to the configuration\nfile format and teach the configuration parser to clear the\ncumulative definitions so far?\n"},{"id":"208930","messageId":"871ucsc6lt.fsf@lifelogs.com","threadId":"32057","inReplyTo":"7vvca47zl3.fsf@alter.siamese.dyndns.org","subject":"Re: overriding/removing inherited credential.helper, Do not add an empty value from config credential.helper","fromName":"Ted Zlatanov","fromEmail":"tzz@lifelogs.com","sentAt":"2013-02-07T18:38:06Z","receivedAt":"2013-02-07T18:38:06Z","isPatch":false,"sender":{"key":"tzz@lifelogs.com","avatar":"https://avatars.githubusercontent.com/u/67764?v=4"},"body":"On Thu, 07 Feb 2013 10:23:20 -0800 Junio C Hamano wrote: \n\nJCH> \"Clear everything you saw so far\" would be useful for variables\nJCH> other than \"credential.helper\"; shouldn't it be done by adding a\nJCH> general syntax to the configuration file format and teach the\nJCH> configuration parser to clear the cumulative definitions so far?\n\nThat's a question for the configuration parser, right? I don't have an\ninformed opinion on how it should be implemented, and am OK with any\nsyntax to clear a variable.\n\nTed\n"}]}