Build fails on progressui_osx.mm (gcc 4.0.0)

VERIFIED FIXED

Status

()

VERIFIED FIXED
13 years ago
13 years ago

People

(Reporter: krmathis, Assigned: jaas)

Tracking

Trunk
PowerPC
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

13 years ago
User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8b2) Gecko/20050617 Firefox/1.0+ (PowerBook)
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8b2) Gecko/20050617 Firefox/1.0+ (PowerBook)

Todays Trunk build fails on progressui_osx.mm
Building on Mac OS 10.4.1, Xcode 2.1, GCC 4.0.0 (build 5026), -O3, with the
following .mozconfig:
http://homepage.mac.com/krmathis/Trunk/mozconfig.txt

Yesterdays build completed successfully, like the previous 10 days, but todays
build error out with this message:
_______________________________________
c++ -o progressui_osx.o -c  -DOSTYPE=\"Darwin8.1.0\" -DOSARCH=\"Darwin\"
-DBUILD_ID=0000000000  -I../../../../../dist/include/libmar
-I../../../../../dist/include/libbz2 -I../../../../../dist/include
-I../../../../../dist/include -I../../../../../dist/include/nspr   
-I../../../../../dist/sdk/include    -mdynamic-no-pic   -fno-rtti
-fno-exceptions -Wall -Wconversion -Wpointer-arith -Wcast-align
-Woverloaded-virtual -Wsynth -Wno-ctor-dtor-privacy -Wno-non-virtual-dtor
-Wno-long-long -nostdinc -nostdinc++
-I/Developer/SDKs/MacOSX10.3.9.sdk/usr/include/gcc/darwin/4.0/c++
-I/Developer/SDKs/MacOSX10.3.9.sdk/usr/include/gcc/darwin/4.0/c++/powerpc-apple-darwin8
-I/Developer/SDKs/MacOSX10.3.9.sdk/usr/include/gcc/darwin/4.0/c++/backward
-isystem
/Developer/SDKs/MacOSX10.3.9.sdk/usr/lib/gcc/powerpc-apple-darwin8/4.0.0/include
-isystem /Developer/SDKs/MacOSX10.3.9.sdk/usr/include
-F/Developer/SDKs/MacOSX10.3.9.sdk/System/Library/Frameworks
-F/Developer/SDKs/MacOSX10.3.9.sdk/Library/Frameworks -fpascal-strings
-no-cpp-precomp -fno-common -fshort-wchar
-I/Developer/SDKs/MacOSX10.3.9.sdk/Developer/Headers/FlatCarbon -pipe  -DNDEBUG
-DTRIMMED -O3 -mcpu=7450 -faltivec   -DMOZILLA_CLIENT -include
../../../../../mozilla-config.h -Wp,-MD,.deps/progressui_osx.pp
/Users/krmathis/Firefox/mozilla/toolkit/mozapps/update/src/updater/progressui_osx.mm
/Users/krmathis/Firefox/mozilla/toolkit/mozapps/update/src/updater/progressui_osx.mm:
In function 'int ShowProgressUI()':
/Users/krmathis/Firefox/mozilla/toolkit/mozapps/update/src/updater/progressui_osx.mm:103:
error: invalid conversion from 'const char' to 'const char**'
/Users/krmathis/Firefox/mozilla/toolkit/mozapps/update/src/updater/progressui_osx.mm:103:
error:   initializing argument 2 of 'int NSApplicationMain(int, const char**)'
make[7]: *** [progressui_osx.o] Error 1
make[6]: *** [libs] Error 2
make[5]: *** [libs] Error 2
make[4]: *** [libs] Error 2
make[3]: *** [libs] Error 2
make[2]: *** [tier_50] Error 2
make[1]: *** [default] Error 2
make: *** [build] Error 2 
_______________________________________

Maybe this is a regression to yesterdays progressui_osx.mm checkin?



Reproducible: Always

Steps to Reproduce:
1.Pull latest trunk source from cvs
2.Compile using the above linked .mozconfig 
Actual Results:  
Build errors out

Expected Results:  
Build completes successfully
(Reporter)

Updated

13 years ago
Version: unspecified → Trunk

Updated

13 years ago
Assignee: nobody → joshmoz
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 1

13 years ago
Created attachment 186684 [details] [diff] [review]
fix

Should use address-of, not dereference.  It's upsetting that gcc 3.3 would even
accept this.

-  NSApplicationMain(1, *sProgramPath);
+  NSApplicationMain(1, &sProgramPath);

Josh, is there a reason not to just use *pargc and *pargv?
Attachment #186684 - Flags: review?(joshmoz)
(Assignee)

Comment 2

13 years ago
fixed
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
(Reporter)

Updated

13 years ago
Status: RESOLVED → VERIFIED
(Reporter)

Comment 3

13 years ago
Thanks guys!
Build completed successfully now.

Updated

13 years ago
Attachment #186684 - Flags: review?(joshmoz)
You need to log in before you can comment on or make changes to this bug.