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

slash in branch name

From
KKK <horizn@wp.pl>
Date
Jun 17, 2015, 19:16 UTC
Message-ID
<5581C77A.3040703@wp.pl>
Hi,

After upgrade GIT from 1.7.2.5-3.1 to 1.7.10.4-1+wheezy1 following error appear:

git push central versions/4.3.2 Counting objects: 45, done. Delta compression using up to 8 threads. Compressing objects: 100% (28/28), done. Writing objects: 100% (28/28), 13.01 KiB, done. Total 28 (delta 22), reused 0 (delta 0) remote: Update script started: Tue Jun 9 13:25:34 BST 2015 remote: Arguments: refs/heads/versions/4.3.2 2e5233728aecc6ac337f4d3a9f32281e7c786e27 ae25cc33f6cf745cfa1061cbdfaf445344a60d13 remote: Warning: using temporary hooks remote: error: invalid key: hooks.denypush.branch.versions/4.3.2 remote: error: invalid key: hooks.allowmerge.versions/4.3.2 remote: Packages changed by this update: remote: think_3 remote: hls_plugin_1 remote: hal_av_1 remote: rtsp_plugin_1 remote: Notifying the following package owners of this update:

hooks were downloaded from: git://git.et.redhat.com/ovirt-server.git

My colleague did some research about that and it seems that this commit has stopped update hook working:

commit b09c53a3e331211fc0154de8ebb271e48f8c7ee5
Author: Libor Pechacek <lpechacek@suse.cz>
Date:   Sun Jan 30 20:40:41 2011 +0100
     Sanity-check config variable names
     Sanity-check config variable names when adding and retrieving them. 
  As a side
     effect code duplication between git_config_set_multivar and 
get_value (in
     builtin/config.c) was removed and the common functionality was 
placed in
     git_config_parse_key.
     This breaks a test in t1300 which used invalid section-less keys in 
the tests
     for "git -c". However, allowing such names there was useless, since 
there was
     no way to set them via config file, and no part of git actually 
tried to use
     section-less keys. This patch updates the test to use more 
realistic examples
     as well as adding its own test.
     Signed-off-by: Libor Pechacek <lpechacek@suse.cz>
     Acked-by: Jeff King <peff@peff.net>
     Signed-off-by: Junio C Hamano <gitster@pobox.com>
Could you please advise how to fix/revert this?
brgds,
Next: Jeff King
Message 1 of 3 in “slash in branch name”
  1. KKJun 17, 2015
  2. Jeff KingJun 17, 2015
  3. KKJun 17, 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.