165632 - Get typeaheadfind building on mac

Status: VERIFIED FIXED

(SeaMonkey :: Find In Page, defect, P1)

Description

[Aaron Leventhal]

I have all the MozillaBuildList.pl changes, I think.
I still need to generate the .xml build files.

Unfortunately, I'm having a lot of bad luck getting Mozilla to build on my Mac
at all. Tried pulling a fresh tree, getting CodeWarrior 7.2, but still no luck.
So I can't move forward on this at the moment.

Comment 1

[Aaron Leventhal]

Hmmm, I'm closer but I keep getting

# Error: Can't create a Finder alias at
(:mozilla:dist:viewer_debug:Components:typeaheadfindDebug.shlb")
for :mozilla:extensions:typeaheadfind:macbuild:typeaheadfindDebug.shlb because
:mozilla:extensions:typeaheadfind:macbuild:typeaheadfindDebug.shlb does not exist.

Comment 2

[Aaron Leventhal]

Attachment: typeaheadfind/macbuild/typeaheadfind.xml

Comment 3

[Aaron Leventhal]

Attachment: typeaheadfind/macbuild/typeaheadfindIDL.xml

Comment 4

[Aaron Leventhal]

Attachment: Diff for the rest of the changes, when combined with xml attachments this should be everything

Comment 5

[hacker formerly known as [e-mail]]

Comment on attachment 98681 [details] [diff] [review]
Diff for the rest of the changes, when combined with xml attachments this should be everything

MozillaBuildFlags.txt contains a change to the 'pull' variable.  We probably
don't want that.

When you combine separate patches, please make sure that they start from the
same directory.  Otherwise, just attach them as separate files.

Comment 6

[hacker formerly known as [e-mail]]

I applied the patches, dropped in the xml files and everything seems to have
built properly.  And typeaheadfind works in my cfm build.

Comment 7

[hacker formerly known as [e-mail]]

Comment on attachment 98679 [details]
typeaheadfind/macbuild/typeaheadfind.xml

r=cls

Comment 8

[hacker formerly known as [e-mail]]

Comment on attachment 98680 [details]
typeaheadfind/macbuild/typeaheadfindIDL.xml

r=cls

Comment 9

[Aaron Leventhal]

Attachment: Whoops - good catch! Removes the change the "pull" line

Seeking r=cls

Comment 10

[hacker formerly known as [e-mail]]

Comment on attachment 98855 [details] [diff] [review]
Whoops - good catch! Removes the change the "pull" line

r=cls

Comment 11

[Simon Fraser [no longer active]]

Comment on attachment 98855 [details] [diff] [review]
Whoops - good catch! Removes the change the "pull" line

Just a minor spacing issue here:

+    {
+      
MakeAlias(":mozilla:extensions:typeaheadfind:resources:locale:en-US:typeaheadfi
nd.properties", "$resource_dir");
+	}

sr=sfraser

Comment 12

[Aaron Leventhal]

Should work in tomrrow's Mozilla trunk builds, if we are blessed by the mac deities

Comment 13

[sairuh (rarely reading bugmail)]

yep, it's there --vrfy'd fixed with 2002.09.20.08 mozilla trunk bits in mac os 10.2.