Skip to content
This repository

Prevent failure when "prefix" is empty string. #71

Open
wants to merge 1 commit into from

1 participant

Vít Ondruch
Vít Ondruch
voxik commented March 11, 2013

In case that "prefix" is empty string, the config_string returns false. Default to empty string for that case.

Vít Ondruch Prevent failure when "prefix" is empty string.
In case that "prefix" is empty string, the config_string returns false. Default to empty string for that case.
f20b4ed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Mar 11, 2013
Vít Ondruch Prevent failure when "prefix" is empty string.
In case that "prefix" is empty string, the config_string returns false. Default to empty string for that case.
f20b4ed
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 1 addition and 1 deletion. Show diff stats Hide diff stats

  1. 2  ext/RMagick/extconf.rb
2  ext/RMagick/extconf.rb
@@ -78,7 +78,7 @@ def check_multiple_imagemagick_versions()
78 78
 # binaries). This causes problems when /usr/lib is in the path (e.g., using
79 79
 # the default Ruby installation).
80 80
 def check_partial_imagemagick_versions()
81  
-   prefix = config_string("prefix")
  81
+   prefix = config_string("prefix") || ""
82 82
    matches = [
83 83
      prefix+"/lib/lib?agick*",
84 84
      prefix+"/include/ImageMagick",
Commit_comment_tip

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.