bugOpenEXR - Bugs: bug #13428, configure.ac doesnt quote tests...

 
 

bug #13428: configure.ac doesnt quote tests properly

Submitted by:  Mike Frysinger <vapier>
Submitted on:  Thu 16 Jun 2005 10:32:21 PM UTC  
 
Category: configureSeverity: 3 - Normal
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: Drew Hess <dhess>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Tue 06 Jun 2006 02:50:56 AM UTC, comment #1:

Fixed in CVS.

Drew Hess <dhess>In charge of this item.
Thu 16 Jun 2005 10:32:21 PM UTC, original submission:

the configure.ac has a bunch of checks such as:
if test -z $PKG_CONFIG_PATH; then

however, if the variable you check contains whitespace, this will cause an error:
$ export PKG_CONFIG_PATH="/some weird/path/"
$ test -z $PKG_CONFIG_PATH && echo empty
bash: test: /some: binary operator expected

a quick check shows these tests need more quoting:
if test -z $PKG_CONFIG_PATH; then
if test $have_fltk = "yes"; then
if test $have_cg = "yes"; then

Mike Frysinger <vapier>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

Follow 3 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Tue 06 Jun 2006 02:50:56 AM UTCdhessStatusNone=>Fixed
  Assigned toNone=>dhess
  Open/ClosedOpen=>Closed

Back to the top


Powered by Savane 3.1-cleanup1