mozregression should use a persist folder by default

RESOLVED WORKSFORME

Status

Testing
mozregression
RESOLVED WORKSFORME
3 years ago
3 years ago

People

(Reporter: parkouss, Unassigned)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox43 affected)

Details

(Reporter)

Description

3 years ago
mozregression should use a persist folder by default. This would be good for new users, as future bisections may reuse already downloaded builds without any special user interaction.

Comment 1

3 years ago
Will the location be user-definable?
(Reporter)

Comment 2

3 years ago
Sure. :) If you already use a config file and defined a destination in there, it will also be used (hm, maybe we will move the location of that file in the same time, this needs discussion).

This is just that the default will no more be "no persist", but a folder in the user directory configuration. But if specified, (command line or config) the user choice will be respected.

Comment 3

3 years ago
Seems pretty awesome to me. Indeed, new users should have this option enabled without interaction. This is, I think, how most people would want the tool to behave.

Why do you want to move the config file? In my opinion, it is fine to have it into user directory.
(Reporter)

Comment 4

3 years ago
(In reply to jonathan.pigree from comment #3)
> Seems pretty awesome to me. Indeed, new users should have this option
> enabled without interaction. This is, I think, how most people would want
> the tool to behave.
> 
> Why do you want to move the config file? In my opinion, it is fine to have
> it into user directory.

Well, it will stay in user directory. This is just that for now, the config is in:

~/.mozregression.cfg

I would prefer to have everything in one folder, so maybe something like:

~/.mozilla/mozregression/mozregression.cfg
~/.mozilla/mozregression/persist

Comment 5

3 years ago
Okay. I searched for a standard logic on where to put the config file in my tools (maven, gradle, git, npm, vim) and I can't. Some will do as you want, some just put the config file directly into the user directory.

But I agree with you, move the config file into the mozilla dedicated folder seems cleaner to me.
(Reporter)

Comment 6

3 years ago
With 0.42 release, we now warn the user to use a config file, and then we guide him to use a persist folder.

So I think this is good enough - please reopen the bug if I'm missing something!
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.