wbrawner/homebrew-core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fontforge: remove redundant post-install

Browse source 
The install-name error was rectified by the latest release and the
whole post-install block can be killed. Yay!

Note to self: When merged close
https://github.com/fontforge/fontforge/issues/2022.

Closes Homebrew/homebrew#38410.

Signed-off-by: Tim D. Smith <git@tim-smith.us>
This commit is contained in:
Dominyk Tiller 2015-04-06 17:08:28 +01:00 committed by Tim D. Smith
parent d3e6cb16e1
commit 3859b3c69e
1 changed files with 2 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
Formula/fontforge.rb
View file

@ -85,21 +85,8 @@ class Fontforge < Formula
system "./configure", *args
system "make"
system "make", "install"
end
def post_install
# Link this to enable symlinking into /Applications with brew linkapps.
# The name is case-sensitive. It breaks without both F's capitalised.
# If you build with x11 now, it automatically creates an dynamic link from bin/fontforge
# to @executable_path/../Frameworks/Breakpad.framework/Versions/A/Breakpad which
# obviously doesn't exist given fontforge and FontForge.app are in different places.
# If this isn't fixed within a couple releases, consider dumping everything in libexec.
# https://github.com/fontforge/fontforge/issues/2022
if build.with? "x11"
ln_s "#{share}/fontforge/osx/FontForge.app", prefix
system "install_name_tool", "-change", "@executable_path/../Frameworks/Breakpad.framework/Versions/A/Breakpad",
"#{bin}/fontforge", "#{share}/fontforge/osx/FontForge.app/Contents/Frameworks/Breakpad.framework/Versions/A/Breakpad"
end
# The name is case-sensitive. Don't downcase it when linking.
ln_s "#{share}/fontforge/osx/FontForge.app", prefix if build.with? "x11"
end
test do