From e5b6f79b221ecf539ff9ade285667ca729e6c7da Mon Sep 17 00:00:00 2001 From: Xu Cheng Date: Wed, 10 Feb 2016 19:29:40 +0800 Subject: bottle: do not follow symlink when setting mtime Also set atime to mtime, which seems to be a more common practice. Fixes Homebrew/homebrew#49007 Closes Homebrew/homebrew#49027. Signed-off-by: Xu Cheng --- Library/Homebrew/test/test_integration_cmds.rb | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'Library/Homebrew/test') diff --git a/Library/Homebrew/test/test_integration_cmds.rb b/Library/Homebrew/test/test_integration_cmds.rb index bca031f5f..ffdcaa8b5 100644 --- a/Library/Homebrew/test/test_integration_cmds.rb +++ b/Library/Homebrew/test/test_integration_cmds.rb @@ -152,6 +152,11 @@ class IntegrationCommandTests < Homebrew::TestCase url "https://example.com/testball-0.1.tar.gz" end EOS + # `brew bottle` should not fail with dead symlink + # https://github.com/Homebrew/homebrew/issues/49007 + (HOMEBREW_CELLAR/"testball/0.1").cd do + FileUtils.ln_s "not-exist", "symlink" + end assert_match(/testball-0\.1.*\.bottle\.tar\.gz/, cmd_output("bottle", "--no-revision", "testball")) ensure -- cgit v1.2.3