Building with osxspell is broken since Bug 629734

RESOLVED FIXED in mozilla6

Status

()

Core
Spelling checker
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Nomis101, Assigned: khuey)

Tracking

({regression})

unspecified
mozilla6
x86
Mac OS X
regression
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
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"
 %}
Created attachment 527970 [details] [diff] [review]
Fix build error with osxspell.
Attachment #527970 - Flags: review?(ehsan)
Comment on attachment 527970 [details] [diff] [review]
Fix build error with osxspell.

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

r=me.
Attachment #527970 - Flags: review?(ehsan) → review+
(Reporter)

Updated

6 years ago
Keywords: checkin-needed

Updated

6 years ago
Assignee: nobody → khuey
I'll land this myself.  Was going to do it last night but the tree was broken :-(
Keywords: checkin-needed
(Reporter)

Comment 4

6 years ago
(In reply to comment #3)
> I'll land this myself.
Ping
(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
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
You need to log in before you can comment on or make changes to this bug.