hi,
Alan wrote (01 May 2013 14:04:56 GMT) :
>> Or perhaps save the paperwork time to just finish it in time for the
>> freeze if you prefer :)
>>
> Not sure I'll find time to do so, but perhaps there'll be a good
> surprise. There's still the part "if we implement this, the profiling
> script should probably be adapted to wait for another process." to do
> and I didn't look at this yet.
It would be great to have this feature in 0.18, but IMHO it's no big
deal if we have not. With my RM hat on, I suggest we focus on
feature/remember_installed_packages instead.
>> So, I had a quick look.
>>
>> May I suggest rebasing this feature branch on devel, rather than on
>> a feature branch that was not merged yet, and whose history has been
>> rewritten a few times already?
>>
> I can't do that easily, because I need d07288f "Abstract user
> notification in a dedicated command" from that branch. I'll however
> rebase on the latest version of the branch before asking for merge.
Makes sense :)
>> > + /usr/bin/iceweasel "$@"
>>
>> I think this does not do what you think it does.
>> It probably won't work when run with more than one argument,
>> e.g. with `-new-tab URL'.
>>
> Are you sure? Man sh reads:
You are right, and I was wrong. Thanks for teaching me this!
Cheers,
--
intrigeri
| GnuPG key @ https://gaffer.ptitcanardnoir.org/intrigeri/intrigeri.asc
| OTR fingerprint @ https://gaffer.ptitcanardnoir.org/intrigeri/otr.asc
