aboutsummaryrefslogtreecommitdiffstats
path: root/Library
diff options
context:
space:
mode:
authorDominyk Tiller2014-06-27 21:04:06 +0100
committerMike McQuaid2014-06-30 08:00:03 +0100
commit49ad54b94f4d36cdd5827e77086802b5418632f4 (patch)
tree0c959af04b3d868d1871573da24225afba033fc2 /Library
parent4695f6dc6024ef2ab2af0eb64866099e315de417 (diff)
downloadhomebrew-49ad54b94f4d36cdd5827e77086802b5418632f4.tar.bz2
subversion : fix --perl bindings.
Patch suggested by @pwagland that should finally fix the --perl binding mv issue that causes the entire formula to error out if --perl build option is invoked. The issue & consequent patch will not affect anyone who doesn't build subversion with perl. I've included a # explanation as to why the change has been made, as hopefully this patch can be replaced either by a subversion fix or a better Ruby-invoked fix in time. Although this could work as a permanent fix, the hope is that it's a temporary workaround until something cleaner is worked out. See the following issues for more detail: [1](https://github.com/Homebrew/homebrew/issues/30370) [2](https://github.com/Homebrew/homebrew/issues/30224) Closes #30500. Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>
Diffstat (limited to 'Library')
-rw-r--r--Library/Formula/subversion.rb6
1 files changed, 5 insertions, 1 deletions
diff --git a/Library/Formula/subversion.rb b/Library/Formula/subversion.rb
index 4ed5483fd..d917297af 100644
--- a/Library/Formula/subversion.rb
+++ b/Library/Formula/subversion.rb
@@ -171,9 +171,13 @@ class Subversion < Formula
end
system "make swig-pl"
system "make", "install-swig-pl", "DESTDIR=#{prefix}"
+
# Some of the libraries get installed into the wrong place, they end up having the
# prefix in the directory name twice.
- mv Dir["#{prefix}/#{lib}/*"], "#{lib}"
+ # There's an ongoing issue with Ruby's mv placing files in the wrong directory and erroring out.
+ # The below addition should resolve the issue for now. See https://github.com/Homebrew/homebrew/issues/30370
+
+ lib.install Dir["#{prefix}/#{lib}/*"]
end
if build.include? 'java'