Re: Looking for MacOS 10.5 testers

[Ben Abbott]

On Jun 17, 2012, at 4:00 PM, Alexander Hansen wrote:

> 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.

Thanks again!

I've pushed the changeset.

        http://hg.savannah.gnu.org/hgweb/octave/rev/0eb1b1eb2c76

Ben