Last Comment Bug 652364 - Building with osxspell is broken since Bug 629734
: Building with osxspell is broken since Bug 629734
Status: RESOLVED FIXED
: regression
Product: Core
Classification: Components
Component: Spelling checker (show other bugs)
: unspecified
: x86 Mac OS X
: -- normal (vote)
: mozilla6
Assigned To: Kyle Huey [:khuey] (Exited; not receiving bugmail, email if necessary)
:
:
Mentors:
Depends on: 629734
Blocks:
  Show dependency treegraph
 
Reported: 2011-04-23 14:31 PDT by Nomis101
Modified: 2011-05-14 16:44 PDT (History)
2 users (show)
khuey: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Fix build error with osxspell. (830 bytes, patch)
2011-04-23 16:28 PDT, Kyle Huey [:khuey] (Exited; not receiving bugmail, email if necessary)
ehsan: review+
Details | Diff | Splinter Review

Description Nomis101 2011-04-23 14:31:58 PDT
I normally build my mozilla applications with osxspell instead of hunspell (by reverting Bug 422760). Since I've recently switched from mozilla-2.0 to mozilla-aurora, this is broken. At the end it everytime shows me:

ld: warning: ignoring file /Developer/SDKs/MacOSX10.6.sdk/System/ 
Library/Frameworks//QuickTime.framework/QuickTime, missing required 
architecture x86_64 in file 
ld: warning: could not create compact unwind for _ffi_call_unix64: 
does not use RBP or RSP based frame 
Undefined symbols for architecture x86_64: 
  "mozOSXSpell::LoadDictionariesFromDir(nsIFile*)", referenced from: 
      vtable for mozOSXSpellin mozOSXSpell.o 
ld: symbol(s) not found for architecture x86_64 
collect2: ld returned 1 exit status 
make[5]: *** [XUL] Error 1 
make[4]: *** [libs_tier_platform] Error 2 
make[3]: *** [tier_platform] Error 2 
make[2]: *** [default] Error 2 
make[1]: *** [default] Error 2 
make: *** [build] Error 2 



Its broken in mozilla-aurora and mozilla-central. I now spend a few days to find the culprit. It is the change from Bug 629734 in mozISpellCheckingEngine.idl. If I back this change out, than it builds fine again. 

The patch I used to back it out and workaround the error is:

diff --git a/extensions/spellcheck/idl/mozISpellCheckingEngine.idl b/extensions/spellcheck/idl/mozISpellCheckingEngine.idl
--- a/extensions/spellcheck/idl/mozISpellCheckingEngine.idl
+++ b/extensions/spellcheck/idl/mozISpellCheckingEngine.idl
@@ -32,20 +32,19 @@
  * and other provisions required by the GPL or the LGPL. If you do not delete
  * the provisions above, a recipient may use your version of this file under
  * the terms of any one of the MPL, the GPL or the LGPL.
  *
  * ***** END LICENSE BLOCK ***** */
 
 #include "nsISupports.idl"
 
-interface nsIFile;
 interface mozIPersonalDictionary;
 
-[scriptable, uuid(6eb307d6-3567-481a-971a-feb666b8ae72)]
+[scriptable, uuid(43987F7B-0FAA-4019-811E-42BECAC73FC5)]
 
 /**
  * This interface represents a SpellChecker.
  */
 
 interface mozISpellCheckingEngine : nsISupports {
   /**
    * The name of the current dictionary
@@ -91,19 +90,14 @@ interface mozISpellCheckingEngine : nsIS
    * check a word
    */
   boolean check(in wstring word);
 
   /**
    * get a list of suggestions for a misspelled word
    */
   void suggest(in wstring word,[array, size_is(count)] out wstring suggestions, out PRUint32 count);
-
-  /**
-   * Load dictionaries from the specified dir
-   */
-  void loadDictionariesFromDir(in nsIFile dir);
 };
 
 %{C++
 #define DICTIONARY_SEARCH_DIRECTORY "DictD"
 #define DICTIONARY_SEARCH_DIRECTORY_LIST "DictDL"
 %}
Comment 1 Kyle Huey [:khuey] (Exited; not receiving bugmail, email if necessary) 2011-04-23 16:28:59 PDT
Created attachment 527970 [details] [diff] [review]
Fix build error with osxspell.
Comment 2 :Ehsan Akhgari 2011-04-25 12:55:40 PDT
Comment on attachment 527970 [details] [diff] [review]
Fix build error with osxspell.

Review of attachment 527970 [details] [diff] [review]:

r=me.
Comment 3 Kyle Huey [:khuey] (Exited; not receiving bugmail, email if necessary) 2011-04-28 02:21:25 PDT
I'll land this myself.  Was going to do it last night but the tree was broken :-(
Comment 4 Nomis101 2011-05-14 14:42:22 PDT
(In reply to comment #3)
> I'll land this myself.
Ping
Comment 5 Kyle Huey [:khuey] (Exited; not receiving bugmail, email if necessary) 2011-05-14 16:44:01 PDT
(In reply to comment #4)
> (In reply to comment #3)
> > I'll land this myself.
> Ping

Sorry, I totally lost track of this.

http://hg.mozilla.org/mozilla-central/rev/f0f14683ce7b

Note You need to log in before you can comment on or make changes to this bug.