[m-rev.] for review: a step towards making deep profiling work on Windows

Julien Fischer juliensf at csse.unimelb.edu.au
Mon Nov 7 01:52:05 AEDT 2011


On Sat, 5 Nov 2011, Paul Bone wrote:

> On Sat, Nov 05, 2011 at 02:00:17PM +1100, Julien Fischer wrote:
>>
>> On Sat, 5 Nov 2011, Paul Bone wrote:
>>
>>> On Fri, Nov 04, 2011 at 04:00:38PM +1100, Julien Fischer wrote:
>>>>
>>>> For review by Zoltan or Paul.
>>>>
>>>
>>> This looks fine to me.
>>>
>>>> Branches: main, 11.07
>>>
>>> Why put it on the release branch?
>>
>> Ignore that, force of habit when I was writing the log message.
>>
>>> More generally, I don't understand why we put non-bugfix changes on release
>>> branches.
>>
>> What non-bugfix changes have gone on to the 11.07 branch?
>>
>
> I'd have to trawl the mailing lists, but I've seen yourself and peter wang
> often include 11.07 in the branches string, maybe in most cases it's a force of
> habbit.

Alternatively, in most they cases they may actually be (and indeed, have
been) bug fixes.

(WRT this particular change, there's not a lot of point putting in on
the release branch.  The deep profiling tool isn't going to work on
Windows any time soon and I don't think the 11.07 version of the auto
parallisation stuff is all that usable -- certainly all your recent
changes have *not* been committed on the 11.07 branch.)

Julien.
--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------



More information about the reviews mailing list