On reflection you're right. My patch would strip a digit off the inode.
A better solution is to use awk, which now I check the git is exactly
what has been submitted.
Thanks.
On 6/10/2010 8:07 PM, Werner LEMBERG wrote:
Configure isnt working correctly in freetype 2.4.3 on solaris. It
immediately errors out with a makefile error about target setup
unknown.
I tracked the problem down to a bug in the configure script
processing the output of ls which on solaris has a leading space the
script wasn't expecting.
The following patch resolves the issue and should be safe on other
platforms as well.
Thanks, however, your patch is incorrect; it removes the last digit of
the inode number too. Maybe a transmission error?
Note that this your problem has been already reported, and fixed
(hopefully) in git; please test.
Werner
======================================================================
diff --git a/configure b/configure
index 06fdc8e..b397556 100755
--- a/configure
+++ b/configure
@@ -80,8 +80,8 @@ fi
# build a dummy Makefile if we are not building in the source tree;
# we use inodes to avoid issues with symbolic links
-inode_src=`ls -id $abs_ft2_dir | sed 's/ .*//'`
-inode_dst=`ls -id $abs_curr_dir | sed 's/ .*//'`
+inode_src=`ls -id $abs_ft2_dir | awk '{print $1}'`
+inode_dst=`ls -id $abs_curr_dir | awk '{print $1}'`
if test $inode_src -ne $inode_dst; then
if test ! -d reference; then
_______________________________________________
Freetype-devel mailing list
[email protected]
http://lists.nongnu.org/mailman/listinfo/freetype-devel
