On 2015-12-17 12:53 PM, Simon Sapin wrote:
On 17/12/15 17:09, Josh Matthews wrote:
Right, I know there are expected conventions. My previous suggestion was
an attempt to provide a mechanism to allow testing persistence between
sessions without actually committing to any of the work involved in
creating a proper setup for persistent profiles. That feels like a
distraction to me, at the moment, and we could spend the time on more
interesting things.
Since rust-xdg already exists, I don’t think using it in code and
setting an environment variable in tests is harder than adding a
non-standard command-line option. This doesn’t have to be a distraction
if we don’t bikeshed it.
To me, the distraction comes from dealing with technical decisions about
testing + persistence, clearing a profile, dealing with stale persisted
data, upgrading old data formats, etc. Choosing a location can be easy,
but the effects of introducing are intricate. I propose bypassing this
by making non-persistent (ie. a temporary directory for the duration of
a single run) the default, and only introducing persistence via an
explicit developer choice to allow for local experiments.
_______________________________________________
dev-servo mailing list
dev-servo@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-servo
