[Dune] [#958] do not write random files to the user's home directory

Dune flyspray at dune-project.org
Thu Oct 6 12:13:00 CEST 2011


THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

The following task has a new comment added:

FS#958 - do not write random files to the user's home directory
User who did this - Jö Fahlke (joe)

----------
Oh, I did like your old patch, I just didn't like some of the claims made in
the reasoning (and those claims weren't even necessary to make the point).

There are two reasons why I wont apply the patch (or any patch to the resume
feature):

 1. I don't use the resume feature.  I don't feel qualified to make the
    decision whether to change the behavior on behalf of those who do use the
    feature.  This reason of course goes away when at least one other resume
    user besides you speaks in favour (or it turns out that you are the only
    resume user) and noone speaks against.

 2. The second reason is a very selfish one.  Very often such patches turn out
    to cause problems in some corner cases, and then it usually falls to the
    committer to debug this.  Since I don't see the big benefit, besides
    aesthetic benefits, I'm not prepared to make that commitment in this
    case.  I do have a thesis to finish.

These are just reasons why *I* won't apply the patch, I won't object if
someone else applies it.

Regarding your new patch: I like it even better.  It's beautiful concept.  It
makes explicitly specifying a resume file completely superfluous.  (But I
haven't looked at the actual code to see whether there are any bugs, nor have
I actually tested it.)
----------

More information can be found at the following URL:
http://www.dune-project.org/flyspray/index.php?do=details&task_id=958#comment2768

You are receiving this message because you have requested it from the Flyspray bugtracking system.  If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.




More information about the Dune mailing list