All users were logged out of Bugzilla on October 13th, 2018
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.
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?
Created attachment 219493 [details] [diff] [review] pass arguments to the binary if URL-ish parameter isn't last
Attachment #219493 - Flags: superreview?(neil) → superreview+
Status: NEW → RESOLVED
Last Resolved: 13 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.