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

Re: What's cooking in git.git (May 2016, #09; Tue, 31) t1308 broken

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 2, 2016, 21:31 UTC
Message-ID
<xmqqwpm7s2ex.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<xmqq1t4ftheu.fsf@gitster.mtv.corp.google.com>
Junio C Hamano <gitster@pobox.com> writes:
Show 19 quoted lines
> Torsten Bögershausen <tboegi@web.de> writes:
>
>> It seams as  ./t1308-config-set.sh is broken,
>> when the the directory is a soft link:
>>
>> -name=/home/tb/NoBackup/projects/git/git.pu/t/trash
>> directory.t1308-config-set/.gitconfig
>> +name=/home/tb/projects/git/git.pu/t/trash directory.t1308-config-set/.gitconfig
>>  scope=global
>
> It does seem that way.  Somebody is affected by $PWD when we should
> be consistently using the physical / real path.
>
>>
>>  key=foo.bar
>> not ok 28 - iteration shows correct origins
>>
>> I havent't digged further, too many conflicts in the config code, may be
>> somebody knows it directly ?

Perhaps like this, taking hint from the log message of 6eafa6d0 (submodules: don't stumble over symbolic links when cloning recursively, 2012-07-12)?

 t/t1308-config-set.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/t1308-config-set.sh b/t/t1308-config-set.sh
index 065d5eb..1ba9ecb 100755
--- a/t/t1308-config-set.sh
+++ b/t/t1308-config-set.sh
@@ -237,7 +237,7 @@ test_expect_success 'iteration shows correct origins' '
 	key=foo.bar
 	value=from-home
 	origin=file
-	name=$(pwd)/.gitconfig
+	name=$TRASH_DIRECTORY/.gitconfig
 	scope=global
 
 	key=foo.bar
Previous: Junio C HamanoNext: Jeff King
Message 4 of 14 in “What's cooking in git.git (May 2016, #09; Tue, 31)”
  1. Junio C HamanoMay 31, 2016
  2. Torsten BögershausenJun 2, 2016
  3. Junio C HamanoJun 2, 2016
  4. Junio C HamanoJun 2, 2016
  5. Jeff KingJun 2, 2016
  6. t1308: do not get fooled by symbolic links to the source treeJunio C Hamano, Jun 2, 2016
  7. Jeff KingJun 2, 2016
  8. Stefan BellerJun 2, 2016
  9. Jeff KingJun 3, 2016
  10. Johannes SixtJun 3, 2016
  11. Jeff KingJun 3, 2016
  12. Johannes SixtJun 3, 2016
  13. Torsten BögershausenJun 3, 2016
  14. Johannes SchindelinJul 14, 2016

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.