Re: [PATCH 1/4] drm: qxl: Drop misleading comment

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 31 Jan 2017, Gerd Hoffmann <kraxel@xxxxxxxxxx> wrote:
>   Hi,
>
>> declare -n introduced in bash 4.3 is needed for alias
>> support. Workaround at [1].
>> 
>> [1] https://lists.freedesktop.org/archives/intel-gfx/2017-January/118474.html
>
> Works for me.  Thanks.

Thanks, pushed.

>> N.b. dim currently doesn't do all its magic, such as adding Link: tag,
>> if you apply more than one patch at a time. If you apply a multi-patch
>> mbox, only the last one will get the Link: tag.
>
> Hmm, that is bad.

Agreed.

> One of the things the patches tool actually is very good at is applying
> a patch series.  Grab the patches from the list, put them into a mbox
> file, in the correct order (which is a PITA to do manually with a MUA,
> especially in case there are replies with reviewed-by, ...).

Oh, it's a Notmuch based tool? Cool. I should add that to the Notmuch
wiki.

The notmuch-emacs interface lets you pipe all the open messages in a
thread using "c-u |". So you can expand/collapse messages as you like,
and pipe them all as mbox. But that doesn't work either because dim
can't properly handle it. I have a vested interest to fix this, but I
haven't gotten around to it...

> Does dim detect that a Link: tag is already present and doesn't add
> another one then?

Nope. Patches welcome. ;)

BR,
Jani.


-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux