the -createprofile can write out the path it created profile at

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: ray, Assigned: ray)

Tracking

Trunk
x86
macOS
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

This will enable some automation. There are reasons to want to create a new profile auotmatically. As it is, one can run the app with the '-createprofile' option and one gets a new profile and then the info about what was created is gone. One could guess the directory based on the platform, or find all the profiles and see what is new in the list, or one could call the directory service in another script, or ....

Or the '-createprofile' option could just write out the path. That might be easiest.

By the way, it is a couple of lines worth of change.
Attachment #260792 - Flags: review?(benjamin)
Comment on attachment 260792 [details] [diff] [review]
patch for changing the output from -createprofile

This is fine. I would recommend, however, that for most tests you use an explicit -profile <path> argument, instead of -P <name>. That avoids polluting the "global" profile list on the machine.
Attachment #260792 - Flags: review?(benjamin) → review+
I am figuring that I would have to create the profile, get the location, and then move the profile myself. This would work.

It sounded as though you were suggesting I could give the '-createprofile' a path so that it would not create the profile in the usual places. I have found that one cannot use '-createprofile' in this way.

(In reply to comment #2)
> I am figuring that I would have to create the profile, get the location, and
> then move the profile myself. This would work.

I think Benjamin meant "mkdir ~/tmp-whatever && firefox -profile ~/tmp-whatever".
Whiteboard: [needs checkin]
Whiteboard: [needs checkin] → [checkin needed]
Comment on attachment 260792 [details] [diff] [review]
patch for changing the output from -createprofile

Is a super-review needed?
Attachment #260792 - Flags: superreview?(gavin.sharp)
Comment on attachment 260792 [details] [diff] [review]
patch for changing the output from -createprofile

No, this is ready to be landed.
Attachment #260792 - Flags: superreview?(gavin.sharp)
Assignee: nobody → ray
Flags: in-testsuite?
Checking in nsAppRunner.cpp;
/cvsroot/mozilla/toolkit/xre/nsAppRunner.cpp,v  <--  nsAppRunner.cpp
new revision: 1.161; previous revision: 1.160
done
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite? → in-testsuite-
Resolution: --- → FIXED
Whiteboard: [checkin needed]
You need to log in before you can comment on or make changes to this bug.