Merge pull request #2165 from take-cheeze/header_searcher

Implement header searcher to use readline in mirb if possible.

mrbgems/mruby-bin-mirb/mrbgem.rake

@@ -3,7 +3,12 @@ MRuby::Gem::Specification.new('mruby-bin-mirb') do |spec|
   spec.author  = 'mruby developers'
   spec.summary = 'mirb command'
 
-  spec.linker.libraries << 'readline' if spec.cc.defines.include? "ENABLE_READLINE"
+  if spec.build.cc.search_header_path 'readline/readline.h'
+    spec.cc.defines << "ENABLE_READLINE"
+    spec.linker.libraries << 'readline'
+  elsif spec.build.cc.search_header_path 'linenoise.h'
+    spec.cc.defines << "ENABLE_LINENOISE"
+  end
 
   spec.bins = %w(mirb)
 end

mrbgems/mruby-bin-mirb/tools/mirb/mirb.c

@@ -22,7 +22,7 @@
 #define MIRB_WRITE_HISTORY(path) write_history(path)
 #define MIRB_READ_HISTORY(path) read_history(path)
 #define MIRB_USING_HISTORY() using_history()
-#elif ENABLE_LINENOISE
+#elif defined(ENABLE_LINENOISE)
 #define ENABLE_READLINE
 #include <linenoise.h>
 #define MIRB_ADD_HISTORY(line) linenoiseHistoryAdd(line)

tasks/mruby_build_commands.rake

@@ -54,6 +54,18 @@ module MRuby
       @compile_options = '%{flags} -o %{outfile} -c %{infile}'
     end
 
+    alias header_search_paths include_paths
+    def search_header_path(name)
+      header_search_paths.find do |v|
+        File.exist? build.filename("#{v}/#{name}").sub(/^"(.*)"$/, '\1')
+      end
+    end
+
+    def search_header(name)
+      path = search_header_path name
+      path && build.filename("#{path}/#{name}").sub(/^"(.*)"$/, '\1')
+    end
+
     def all_flags(_defineds=[], _include_paths=[], _flags=[])
       define_flags = [defines, _defineds].flatten.map{ |d| option_define % d }
       include_path_flags = [include_paths, _include_paths].flatten.map do |f|

tasks/toolchains/gcc.rake

@@ -28,4 +28,28 @@ MRuby::Toolchain.new(:gcc) do |conf|
     linker.option_library_path = '-L%s'
     linker.link_options = '%{flags} -o %{outfile} %{objs} %{flags_before_libraries} %{libs} %{flags_after_libraries}'
   end
+
+  [[conf.cc, 'c'], [conf.cxx, 'c++']].each do |cc, lang|
+    cc.define_singleton_method(:header_search_paths) do
+      if @header_search_command != command
+        result = `echo | #{build.filename command} -x#{lang} -Wp,-v - -fsyntax-only 2>&1`
+        result = `echo | #{command} -x#{lang} -Wp,-v - -fsyntax-only 2>&1` if $?.exitstatus != 0
+        return include_paths if  $?.exitstatus != 0
+
+        @frameworks = []
+        @header_search_paths = result.lines.map { |v|
+          framework = v.match(/^ (.*)(?: \(framework directory\))$/)
+          if framework
+            @frameworks << framework[1]
+            next nil
+          end
+
+          v.match(/^ (.*)$/)
+        }.compact.map { |v| v[1] }.select { |v| File.directory? v }
+        @header_search_paths += include_paths
+        @header_search_command = command
+      end
+      @header_search_paths
+    end
+  end
 end