Closed Bug 334267 Opened 18 years ago Closed 18 years ago

handle -P profile in startup script remote sniffing

Categories

(SeaMonkey :: UI Design, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ajschult784, Assigned: ajschult784)

Details

Attachments

(1 file, 1 obsolete file)

XRemote has the ability to take a -p profile argument, but the startup script doesn't try to do remote sniffing if -P is specified.  I got the other stuff working because handling -P basically means fully parsing the arguments within the script, so it's a bit heavy.  But jag seemed to like the idea.  I'll attach a patch.
Attached patch patch (obsolete) — Splinter Review
Attachment #218615 - Flags: review?(jag)
Comment on attachment 218615 [details] [diff] [review]
patch

> MOZ_CLIENT_PROGRAM="$dist_bin/mozilla-xremote-client"
> 
> if [ "$OSTYPE" = "beos" ]; then
>   mimeset -F "$MOZ_CLIENT_PROGRAM"
> fi

please assume this is inside the big |if|, between the chunk that parses things (while loop) and the chunk that actually does stuff.
Comment on attachment 218615 [details] [diff] [review]
patch

Please fix the comments around the URL/file: fix-up (move the first bit up to where you grab the (last? assert this?) argument and stuff it in URL, and do s/Last argument/URL/ or something), and as you pointed out yourself, add $profile to the |exec ... openURL|.
Attachment #218615 - Flags: review?(jag) → review-
what do you mean by "assert"?  print to the console?  not use a remote instance?  bail completely?  comment?
Attachment #218615 - Attachment is obsolete: true
Attachment #219493 - Flags: review?(jag)
Attachment #219493 - Flags: review?(jag) → review+
Attachment #219493 - Flags: superreview?(neil)
Attachment #219493 - Flags: superreview?(neil) → superreview+
fixed-on-trunk
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Attachment #219493 - Flags: approval-branch-1.8.1?(neil)
Comment on attachment 219493 [details] [diff] [review]
pass arguments to the binary if URL-ish parameter isn't last

Assuming you're going to let this bake on the trunk first ;-)
Attachment #219493 - Flags: approval-branch-1.8.1?(neil) → approval-branch-1.8.1+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: