Re: Looking for MacOS 10.5 testers

[Alexander Hansen]

On 6/16/12 10:07 PM, Ben Abbott wrote:
> On Jun 17, 2012, at 12:43 AM, Ben Abbott wrote:
> 
>> On Jun 16, 2012, at 11:04 PM, Alexander Hansen wrote:
>>
>>> On 6/16/12 9:06 AM, Ben Abbott wrote:
>>>
>>>> On Jun 16, 2012, at 11:27 AM, Ben Abbott wrote:
>>>>
>>>>> On Jun 15, 2012, at 11:20 PM, Alexander Hansen wrote:
>>>>>
>>>>>> On 6/14/12 9:15 PM, Alexander Hansen wrote:
>>>>>>
>>>>>>> On 6/14/12 9:08 PM, Alexander Hansen wrote:
>>>>>>>
>>>>>>>> On 6/14/12 3:04 PM, Ben Abbott wrote:
>>>>>>>>
>>>>>>>>> I've prepared at changeset which detects the presence of Carbon 
>>>>>>>>> Framework's CGDispalyBitsPerPixel, and replaces it with a new 
>>>>>>>>> function if absent.
>>>>>>>>>
>>>>>>>>> I don't have a 10.5 box to test this on. If anyone else has a 10.5 
>>>>>>>>> box which they use to build the default branch, I'd appreciate 
>>>>>>>>> testing this changeset.
>>>>>>>>>
>>>>>>>>> Ben
>>>>>>>>
>>>>>>>> I'm in the process of looking at this.  I typically don't install 
>>>>>>>> Octave
>>>>>>>> by hand, so it's taking a while :-) .  Plus, I'm also building an 
>>>>>>>> update
>>>>>>>> to gcc4.6 on the same single-processor G4 machine, so that's slowing
>>>>>>>> things down a bit.
>>>>>>>
>>>>>>> It didn't work here:
>>>>>>>
>>>>>>> libtool: compile:  g++ -DHAVE_CONFIG_H -I. -I.. -I../libgnu -I../libgnu
>>>>>>> -I../libcruft/misc -I../liboctave -I../liboctave -I. -I. -I/sw/include
>>>>>>> -I/sw/include -g -O2 -DHAVE_CONFIG_H -I/sw/include
>>>>>>> -I/sw/include/freetype2 -I/sw/include -I/sw/include -Wall -W -Wshadow
>>>>>>> -Wold-style-cast -Wformat -Wpointer-arith -Wwrite-strings -Wcast-align
>>>>>>> -Wcast-qual -g -O2 -D_THREAD_SAFE -pthread -g -O2 -MT
>>>>>>> liboctinterp_la-display.lo -MD -MP -MF .deps/liboctinterp_la-display.Tpo
>>>>>>> -c display.cc  -fno-common -DPIC -o .libs/liboctinterp_la-display.o
>>>>>>> display.cc: In function 'size_t DisplayBitsPerPixel(CGDirectDisplayID)':
>>>>>>> display.cc:49: error: 'CGDisplayModeRef' was not declared in this scope
>>>>>>> display.cc:49: error: expected `;' before 'mode'
>>>>>>> display.cc:50: error: 'mode' was not declared in this scope
>>>>>>> display.cc:50: error: 'CGDisplayModeCopyPixelEncoding' was not declared
>>>>>>> in this scope
>>>>>>> display.cc:52: warning: use of old-style cast
>>>>>>> display.cc:54: warning: use of old-style cast
>>>>>>> display.cc: At global scope:
>>>>>>> display.cc:47: warning: unused parameter 'display'
>>>>>>
>>>>>> With the most recent changeset I still get:
>>>>>>
>>>>>> libtool: compile:  g++-4.2 -DHAVE_CONFIG_H -I. -I.. -I../libgnu
>>>>>> -I../libgnu -I../libcruft/misc -I../liboctave -I../liboctave -I. -I.
>>>>>> -I/sw/include -I/sw/include -g -O2 -DHAVE_CONFIG_H -I/sw/include
>>>>>> -I/sw/include/freetype2 -I/sw/include -I/sw/include -Wall -W -Wshadow
>>>>>> -Wold-style-cast -Wformat -Wpointer-arith -Wwrite-strings -Wcast-align
>>>>>> -Wcast-qual -g -O2 -D_THREAD_SAFE -pthread -g -O2 -MT
>>>>>> liboctinterp_la-display.lo -MD -MP -MF .deps/liboctinterp_la-display.Tpo
>>>>>> -c display.cc  -fno-common -DPIC -o .libs/liboctinterp_la-display.o
>>>>>> display.cc: In function 'size_t DisplayBitsPerPixel(CGDirectDisplayID)':
>>>>>> display.cc:49: error: 'CGDisplayModeRef' was not declared in this scope
>>>>>> display.cc:49: error: expected `;' before 'mode'
>>>>>> display.cc:50: error: 'mode' was not declared in this scope
>>>>>> display.cc:50: error: 'CGDisplayModeCopyPixelEncoding' was not declared
>>>>>> in this scope
>>>>>> display.cc:52: warning: use of old-style cast
>>>>>> display.cc:54: warning: use of old-style cast
>>>>>> display.cc: At global scope:
>>>>>> display.cc:47: warning: unused parameter 'display'
>>>>>> make[3]: *** [liboctinterp_la-display.lo] Error 1
>>>>>
>>>>> Alex,
>>>>>
>>>>> I think I found another syntax problem with my patch to acinclude.m4. 
>>>>> I'll fix it and repost a patch later today.
>>>>>
>>>>> Ben
>>>>
>>>> My third attempt is attached.
>>>>
>>>> Ben
>>>
>>> I got the same error.  config.log attached.
>>
>> Thanks again.
>>
>> The log file indicates the test passed for you.
>>
>> configure:58930: checking whether CGDisplayBitsPerPixel is defined in the 
>> Carbon Framework
>> configure:58975: g++-4.2 -c -g -O2 -I/sw/include conftest.cpp >&5
>> configure:58982: $? = 0
>> configure:58997: result: yes
>>
>> Looking at the preprocessor logic, I see I have at least one typo. I'll 
>> build with a corrected changeset and followup on the list tomorrow.
>>
>> Ben
> 
> Sorry for the mistakes. This is my first time writing a configure test and my 
> first time using the preprocessor. I occurred to me that I could set the 
> HAVE_CARBON_CGDISPLAYBITSPERPIXEL variable manually, and then manually 
> compile display.o using the 10.5 SDK.
> 
> /opt/local/bin/g++-mp-4.5 -isysroot /Developer/SDKs/MacOSX10.5.sdk 
> -DHAVE_CONFIG_H -I. -I..  -I../libgnu -I../libgnu -I../libcruft/misc 
> -I../liboctave -I../liboctave -I. -I. -I/opt/local/include 
> -I/opt/local/include -pipe -O2 -g -m64 -ggdb3  -DHAVE_CONFIG_H 
> -I/opt/local/include -I/opt/local/include/freetype2 -I/opt/local/include 
> -I/opt/local/include   -Wall -W -Wshadow -Wold-style-cast -Wformat 
> -Wpointer-arith -Wwrite-strings -Wcast-align -Wcast-qual -pipe -O2 -g -m64 
> -ggdb3 -D_THREAD_SAFE -pthread -pipe -O2 -g -m64 -ggdb3 -MT display.o -MD -MP 
> -MF $depbase.Tpo -c -o display.o display.cc
> 
> I'm hopeful this fourth attempt will work.
> 
> Ben
> 
> 
> 
> 
> 

It looks like it worked for me.

-- 
Alexander Hansen, Ph.D.
Fink User Liaison
My package updates: http://finkakh.wordpress.com/