git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2 1/4] git-p4: add optional type specifier to gitConfig reader

From
Luke Diamand <luke@diamand.org>
Date
Sep 3, 2015, 19:55 UTC
Message-ID
<55E8A5C2.9090500@diamand.org>
In-Reply-To
<1441298148-63885-2-git-send-email-larsxschneider@gmail.com>
On 03/09/15 17:35, larsxschneider@gmail.com wrote:
> From: Lars Schneider <larsxschneider@gmail.com>
>
I think this commit may need some explanation!
Show 37 quoted lines
> Signed-off-by: Lars Schneider <larsxschneider@gmail.com>
> ---
>   git-p4.py | 12 ++++++------
>   1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/git-p4.py b/git-p4.py
> index 073f87b..c139cab 100755
> --- a/git-p4.py
> +++ b/git-p4.py
> @@ -604,9 +604,12 @@ def gitBranchExists(branch):
>
>   _gitConfig = {}
>
> -def gitConfig(key):
> +def gitConfig(key, typeSpecifier=None):
>       if not _gitConfig.has_key(key):
> -        cmd = [ "git", "config", key ]
> +        cmd = [ "git", "config" ]
> +        if typeSpecifier:
> +            cmd += [ typeSpecifier ]
> +        cmd += [ key ]
>           s = read_pipe(cmd, ignore_error=True)
>           _gitConfig[key] = s.strip()
>       return _gitConfig[key]
> @@ -617,10 +620,7 @@ def gitConfigBool(key):
>          in the config."""
>
>       if not _gitConfig.has_key(key):
> -        cmd = [ "git", "config", "--bool", key ]
> -        s = read_pipe(cmd, ignore_error=True)
> -        v = s.strip()
> -        _gitConfig[key] = v == "true"
> +        _gitConfig[key] = gitConfig(key, '--bool') == "true"
>       return _gitConfig[key]
>
>   def gitConfigList(key):
>
Previous: larsxschneider@gmail.comNext: Lars Schneider
Message 3 of 17 in “git-p4: add support for large file systems”
  1. git-p4: add support for large file systemslarsxschneider@gmail.com, Sep 3, 2015
  2. 1/4 git-p4: add optional type specifier to gitConfig readerlarsxschneider@gmail.com, Sep 3, 2015
  3. Luke DiamandSep 3, 2015
  4. Lars SchneiderSep 3, 2015
  5. Junio C HamanoSep 3, 2015
  6. Lars SchneiderSep 3, 2015
  7. Junio C HamanoSep 3, 2015
  8. Lars SchneiderSep 4, 2015
  9. 2/4 git-p4: add gitConfigInt readerlarsxschneider@gmail.com, Sep 3, 2015
  10. Luke DiamandSep 3, 2015
  11. Lars SchneiderSep 3, 2015
  12. Luke DiamandSep 3, 2015
  13. 3/4 git-p4: return an empty list if a list config has no valueslarsxschneider@gmail.com, Sep 3, 2015
  14. 4/4 git-p4: add support for large file systemslarsxschneider@gmail.com, Sep 3, 2015
  15. Luke DiamandSep 3, 2015
  16. Lars SchneiderSep 3, 2015
  17. Jeff KingSep 4, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.