Closed Bug 202340 Opened 22 years ago Closed 22 years ago

Need AppleScript dictionary files in Mozilla

Categories

(SeaMonkey :: Build Config, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: sfraser_bugs, Assigned: sfraser_bugs)

References

Details

Attachments

(2 files, 1 obsolete file)

Mozilla needs .scriptSuite and .scriptTerminology files for its AppleScript dictionary.
Blocks: 202072
Actually, .scriptSuite and .scriptTerminology resources are for Cocoa apps only. Mozilla still needs a .aete, but there's a cool new way to make one from an XML file: see the last paragraph on <http://developer.apple.com/techpubs/macosx/ReleaseNotes/AppleScript.html>
Attachment #120872 - Flags: superreview?(bryner)
Attachment #120872 - Flags: review?(ccarlen)
Wow, what a great way to make an aete. Cool. Looks like the patch has the World Wide Web suite and the URL Suite - would it be possible to get the required, core, and Mozilla suites too? Another bug? I had some scripts I used to use before Mozilla was stable to do state persistence, and with a few nightlies I've longed to dust them off. :)
Attachment #120872 - Flags: superreview?(bryner) → superreview+
This change cleans up dist/, and makes it easier to pull out files for the commercial build. Visualize ns/xpfe/bootstrap/Makefile.in changes very similar to those in mozilla/xpfe/bootstrap/Makefile.in :)
Attachment #120993 - Flags: superreview?(seawood)
Attachment #120993 - Flags: review?(ccarlen)
Comment on attachment 120872 [details] [diff] [review] Patch adding mozilla.sdef, with makefile changes r=ccarlen
Attachment #120872 - Flags: review?(ccarlen) → review+
Comment on attachment 120993 [details] [diff] [review] Build changes to put PrintPDE.plugin and mozillaSuite.rsrc in dist/package >diff -u -4 -r1.235 Makefile.in >--- xpfe/bootstrap/Makefile.in 17 Apr 2003 19:19:40 -0000 1.235 >+++ xpfe/bootstrap/Makefile.in 18 Apr 2003 19:34:06 -0000 >@@ -427,11 +427,13 @@ > rsync -a $(DIST)/bin/ $(DIST)/$(APP_NAME).app/Contents/MacOS > rm -f $(DIST)/$(APP_NAME).app/Contents/MacOS/$(PROGRAM) > rsync -aL $(PROGRAM) $(DIST)/$(APP_NAME).app/Contents/MacOS > mkdir -p $(DIST)/$(APP_NAME).app/Contents/Plug-Ins >- cp -R $(DIST)/PrintPDE.plugin $(DIST)/$(APP_NAME).app/Contents/Plug-Ins >+ cp -R $(DIST)/package/PrintPDE.plugin $(DIST)/$(APP_NAME).app/Contents/Plug-Ins/ > cp -RL $(srcdir)/macbuild/mach.icns $(DIST)/$(APP_NAME).app/Contents/Resources/mach.icns >+ cp -RL $(DIST)/package/mozillaSuite.rsrc $(DIST)/$(APP_NAME).app/Contents/Resources/$(PROGRAM).rsrc This will need change if we ever want more stuff in $(PROGRAM).rsrc, like libwidget.rsrc, but OK for now.
Attachment #120993 - Flags: review?(ccarlen) → review+
Comment on attachment 120993 [details] [diff] [review] Build changes to put PrintPDE.plugin and mozillaSuite.rsrc in dist/package RES_DEST & RSRC_DEST won't be removed on a distclean. Those files should be added to GARBAGE instead of creating a custom rule.
Attachment #120993 - Flags: superreview?(seawood) → superreview-
Attachment #120993 - Attachment is obsolete: true
Attachment #121001 - Flags: superreview?(seawood)
Attachment #121001 - Flags: superreview?(seawood) → superreview+
Fixed, along with fixes in the commercial tree.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Will this make AppleScript 1.9 a system requirement? The URL in comment 1 seems to indicate this XML description format is new for that version.
No. The .sdef gets compiled into an old-style aete resource.
Now, Mozilla Firefox does not have this dictionary file. Doesn't Mozilla Firefox support AppleScript?
It probably does, but needs to build loving to add the correct files, as is done here: http://lxr.mozilla.org/seamonkey/source/xpfe/bootstrap/appleevents/Makefile.in#87
Product: Browser → Seamonkey
Just checked the latest Deer Park nightly - we have available currently: OpenURL: Load a URL register URL echo: Register a URL echo handler. The handler will get called whenever a new URL is loaded unregister URL echo: Cancels URL echo Get URL: Load a URL but we don't have Required Suite or Standard Suite items.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: