555684 - 'cairo-cocoa' (= MOZ_WIDGET_COCOA) build (which use Carbon!) doesn't support GetWDInfo(): investigate whether a rewrite is needed

Status: RESOLVED WONTFIX

(MailNews Core :: Build Config, defect)

Description

[Nomis101]

Bug 514787 removed TARGET_CARBON and TARGET_API_MAC_CARBON from m-c. I'ts also unused in c-c. But we have a problem there if we remove the ifdef from nsMsgAppleEncode.cpp
https://bug514787.bugzilla.mozilla.org/attachment.cgi?id=426716

It than doesn't build anymore because of 
/mailnews/compose/src/nsMsgAppleEncode.cpp:182: error: ‘GetWDInfo’ was not declared in this scope
This is, because GetWDInfo is not supported in Mac OS X and Carbon, see http://developer.apple.com/legacy/mac/library/documentation/Carbon/Reference/Carbon_Spec_Porting/File_Manager.html

The removal of the AC_DEFINEs in configure.in is unproblematic, I do this for a few month now without any problems. I suppose nsMsgAppleEncode.cpp must be rewriten also for Bug 537496, because its not 64bit compatible.

Comment 1

[Serge Gautherie (:sgautherie)]

(In reply to comment #0)
> http://developer.apple.com/legacy/mac/library/documentation/Carbon/Reference/Carbon_Spec_Porting/File_Manager.html

{
File Manager functions that support MFS (the Macintosh file system) are not supported in Carbon. These include
      * functions, such as OpenWD and GetWDInfo, that manipulate working directories
}

Currently, this code is #ifdef'ed out.