Backport #8593: MKMF have_framework check assumes the framework and header are identically named

• Status: Closed
• Assignee: nagachika (Tomoyuki Chikanaga)

[karljs (Karl Smeltzer)]

I am working on a Ruby extension in which the C code links against SDL2.0 on OSX. This is distributed as an OSX framework (SDL2.framework directory) but the primary header file is simply SDL.h and not SDL2.h

When I add have_framework('SDL2') to my extconf.rb file, it fails the check because it only tests for SDL2/SDL2.h

If I create a symlink named SDL2.h in the same directory, then the check passes and everything builds correctly.

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Feedback

=begin
How about passing the header name by an optional argument?

have_framework("SDL2", header: "SDL.h")

diff --git a/lib/mkmf.rb b/lib/mkmf.rb
index 7f472f4..464cbbf 100644
--- a/lib/mkmf.rb
+++ b/lib/mkmf.rb
@@ -1081,9 +1081,9 @@ SRC
# the +HAVE_FRAMEWORK_RUBY+ preprocessor macro would be passed to the
# compiler.
#

• def have_framework(fw, &b)

• def have_framework(fw, header: "#{fw}.h", &b)
  checking_for fw do

•  src = cpp_include("#{fw}/#{fw}.h") << "\n" "int main(void){return 0;}"
  

•  src = cpp_include("#{fw}/#{header}") << "\n" "int main(void){return 0;}"
   opt = " -framework #{fw}"
   if try_link(src, "-ObjC#{opt}", &b)
     $defs.push(format("-DHAVE_FRAMEWORK_%s", fw.tr_cpp))
  

=end

[nobu (Nobuyoshi Nakada)]

-framework option implies -l option, does the library name differ from its framework name?

[nobu (Nobuyoshi Nakada)]

• Status changed from Feedback to Closed
• % Done changed from 0 to 100

This issue was solved with changeset r41777.
Karl, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

mkmf.rb: header in framework

• lib/mkmf.rb (have_framework): allow header file to check.
  [ruby-core:55745] [Bug #8593]

[nagachika (Tomoyuki Chikanaga)]

• Tracker changed from Bug to Backport
• Project changed from Ruby to 15
• Status changed from Closed to Assigned
• Assignee set to nagachika (Tomoyuki Chikanaga)

[nagachika (Tomoyuki Chikanaga)]

• Status changed from Assigned to Closed

This issue was solved with changeset r42357.
Karl, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

merge revision(s) 41777,41779: [Backport #8593]

* lib/mkmf.rb (have_framework): allow header file to check.
  [ruby-core:55745] [Bug #8593]