[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