Harry,

According to your suggestion, I've reworked the patches which remove the 
modification of configure instead of adding LDFLAGS.

Thanks for your comments

Jim

> Jim,
>
>       You are patching configure. But we should patch configure.in or
> configure.ac instead. The configure is a generated file and easily
> changed to make your patch fail.
>
> Thanks,
> Harry
>
>
> On Fri, 2009-01-09 at 17:05 +0800, Jim Li wrote:
>    
>> diff -uNr libvisual-0.4.0.orig/configure libvisual-0.4.0/configure
>> --- libvisual-0.4.0.orig/configure      2006-03-21 02:42:25.000000000
>> +0800
>> +++ libvisual-0.4.0/configure   2009-01-07 11:31:12.020282000 +0800
>>      

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: bugfix6788530-1
URL: 
<http://mail.opensolaris.org/pipermail/jds-review/attachments/20090120/a7bcee90/attachment.ksh>

Reply via email to