Bug 48020

Summary: Fix for src/makealias on Solaris 10
Product: fontconfig Reporter: mark
Component: confAssignee: Akira TAGOH <akira>
Status: RESOLVED FIXED QA Contact: Behdad Esfahbod <freedesktop>
Severity: blocker    
Priority: medium CC: akira, fontconfig-bugs
Version: 2.8   
Hardware: SPARC   
OS: Solaris   
Whiteboard:
i915 platform: i915 features:

Description mark 2012-03-28 22:26:11 UTC
SunOS host 5.10 Generic_142901-13 i86pc i386 i86pc
SunOS host2 5.10 Generic_142909-17 sun4v sparc SUNW,T5240

Version:
fontconfig 2.9.0 (this version is not an option in the bug submitter)

Overview:
The shell code syntax used in src/makealias bombs on Solaris 10. (Works on Solaris 11 however). The $(....) syntax can be replaced by `....` and things are happy.

Error:
Making all in src
gmake[2]: Entering directory `/var/tmp/fontconfig-2.9.0/src'
sh ../src/makealias "../src" fcalias.h fcaliastail.h ../fontconfig/fontconfig.h ../src/fcdeprecate.h ../fontconfig/fcprivate.h
../src/makealias: syntax error at line 20: `ifdef=$' unexpected

Bad line:
ifdef=$(grep -l '^'$name'[ (]' "$SRCDIR"/*.c | sed -n 1p | sed -e 's/^.*\/\([^.]*\)\.c/__\1__/')

Fix:
    /bin/perl -pe 's/\$\(/\`/ && s/\)$/`/' -i src/makealias

Good line:
ifdef=`grep -l '^'$name'[ (]' "$SRCDIR"/*.c | sed -n 1p | sed -e 's/^.*\/\([^.]*\)\.c/__\1__/'`
Comment 1 Akira TAGOH 2012-03-28 23:46:01 UTC
(In reply to comment #0)
> fontconfig 2.9.0 (this version is not an option in the bug submitter)

Yes, I'm requesting to add new one to bugzilla now.

> Overview:
> The shell code syntax used in src/makealias bombs on Solaris 10. (Works on
> Solaris 11 however). The $(....) syntax can be replaced by `....` and things
> are happy.

Thanks for reporting. fixed in 9fe7c986.

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.