[mythtv-users] osx-packager-qtsdk.pl (was Re: [mythtv-commits] packaging/master commit: 1af486e9a by Jean-Yves Avenard (jyavenard))

[Andrew Leech]

On 24/05/12 12:02 PM, Jean-Yves Avenard wrote:
> On 24 May 2012 11:44, Andrew Leech<coronasensei at gmail.com>  wrote:
>> I did have to change it slightly to allow compilation with the older qt
>> 4.7.4:
>
>> --- a/OSX/build/osx-packager-qtsdk.pl
>> +++ b/OSX/build/osx-packager-qtsdk.pl
>> @@ -447,7 +447,7 @@ else
>>      if ( $hostos =~ m/^1[12]\./&&  $QTVERSION !~ m/^4\.[89]\./ )
>>      {
>> &Complain("You must use Qt 4.8.0 or above with your OS");
>> -        exit;
>> +        # exit;
>>      }
>>      $QTBIN="$PREFIX/bin";
>>      $QTPLUGINS="$PREFIX/plugins";
> which is a rubbish change.
> This is only required if you are building Qt from source
> 4.7.4 will not compile with lion. period
> not in 32 bits mode, not in 64 bits.. it just won't compile and isn't
> compatible.
>
> You can however, use the binary distribution 4.7.x and 4.6.x
>
> During compilation of myth, you will get thousands of warning that
> this OS isn't supported, but it will still work.
>
> I don't put special test in a code unless they are absolutely required..
>
> Don't expect any supports if you modify this build script.
>
Ah I understand, thanks for the clarification.
I had left that in from when I was trying to compile Qt from source, as 
I already had that downloaded and wanted to avoid the binary download 
(I've got rubbish internet). I have since switched to binary anyway and 
didn't realise that check didn't apply to binary. Sorry for wasting your 
time.

I massively appreciate your work on this new packager and in replying to 
my posts. I thought I may have been finding issues in the packager I 
should alert you to, I see now the issues were all of my own generating. 
I did think I had ruled out my own error before posting, but clearly hadn't.

Best Regards,
Andrew