diff options
| author | Dominyk Tiller | 2014-06-27 21:04:06 +0100 |
|---|---|---|
| committer | Mike McQuaid | 2014-06-30 08:00:03 +0100 |
| commit | 49ad54b94f4d36cdd5827e77086802b5418632f4 (patch) | |
| tree | 0c959af04b3d868d1871573da24225afba033fc2 /Library/Formula | |
| parent | 4695f6dc6024ef2ab2af0eb64866099e315de417 (diff) | |
| download | homebrew-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/Formula')
| -rw-r--r-- | Library/Formula/subversion.rb | 6 |
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' |
