Bug 52197 - [Patch] portability issue in configure.ac
Summary: [Patch] portability issue in configure.ac
Status: RESOLVED FIXED
Alias: None
Product: poppler
Classification: Unclassified
Component: general (show other bugs)
Version: unspecified
Hardware: Other Solaris
: medium normal
Assignee: poppler-bugs
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-17 12:53 UTC by Torsten Kasch
Modified: 2012-07-18 22:07 UTC (History)
0 users

See Also:
i915 platform:
i915 features:

Attachments
portability patch for configure.ac (288 bytes, application/octet-stream)
2012-07-17 12:53 UTC, Torsten Kasch 		Details
View All

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.