Bug 52197

Summary: [Patch] portability issue in configure.ac
Product: poppler Reporter: Torsten Kasch <tk>
Component: generalAssignee: poppler-bugs <poppler-bugs>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: Other   
OS: Solaris   
Whiteboard:
i915 platform: i915 features:
Attachments: portability patch for configure.ac

Description Torsten Kasch 2012-07-17 12:53:31 UTC
Created attachment 64313 [details]
portability patch for configure.ac

Configuring poppler-0.20.2 in a separate object dir failed for me on Solaris with the following error message:

/vol/src/X11/poppler/poppler-0.20.2/configure: syntax error at line 23951: `(' unexpected

This is due to the (wrong) assumption, that /bin/sh is always a bash. On Solaris (at least), /bin/sh is a real Bourne Shell and therefore does not know about $(...) command substitutions.

The attached patch resolves this issue by replacing the $(...) command substitution with the more portable backtick evaluation.
Comment 1 Albert Astals Cid 2012-07-18 22:07:14 UTC
Commited, thanks

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.