[prev in list] [next in list] [prev in thread] [next in thread]
List: freetype-devel
Subject: [ft-devel] Re: configure bug in freetype 2.4,3 on solaris
From: Robert Cohen <robert.cohen () anu ! edu ! au>
Date: 2010-10-06 9:24:45
Message-ID: 4CAC405D.8040309 () anu ! edu ! au
[Download RAW message or body]
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
Freetype-devel@nongnu.org
http://lists.nongnu.org/mailman/listinfo/freetype-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic