[Buildroot] [PATCH] package/x11r7/xserver_xorg-server: Add patch to fix v1.19.1 with SIGIO

[Mylene Josserand]

Hello,

On 15/02/2017 11:06, Gary Bisson wrote:
> Hi Mylène, All,
>
> On Wed, Feb 15, 2017 at 10:16 AM, Mylène Josserand
> <mylene.josserand at free-electrons.com> wrote:
>> The commit 6a5a4e60373c from xserver v1.19.1 removes SIGIO support for
>> input [1].
>>
>> The current patch coming from Open Embedded [2] removes a check of
>> 'useSIGIO'. This check was producing a segfault error while using
>> X11 and Vivante driver [3]. Thanks to Gary Bisson to point out this
>> patch!
>>
>> [1] https://cgit.freedesktop.org/xorg/xserver/commit/?id=6a5a4e60373c1386b311b2a8bb666c32d68a9d99
>> [2] https://patchwork.openembedded.org/patch/136696/
>> [3] http://lists.busybox.net/pipermail/buildroot/2017-February/183994.html
>>
>> Signed-off-by: Mylène Josserand <mylene.josserand at free-electrons.com>
>> ---
>>  .../0003-Remove-check-for-useSIGIO-option.patch    | 47 ++++++++++++++++++++++
>>  1 file changed, 47 insertions(+)
>>  create mode 100644 package/x11r7/xserver_xorg-server/1.19.1/0003-Remove-check-for-useSIGIO-option.patch
>>
>> diff --git a/package/x11r7/xserver_xorg-server/1.19.1/0003-Remove-check-for-useSIGIO-option.patch b/package/x11r7/xserver_xorg-server/1.19.1/0003-Remove-check-for-useSIGIO-option.patch
>> new file mode 100644
>> index 000000000..beed6cb4a
>> --- /dev/null
>> +++ b/package/x11r7/xserver_xorg-server/1.19.1/0003-Remove-check-for-useSIGIO-option.patch
>> @@ -0,0 +1,47 @@
>> +From cf407b16cd65ad6e26a9c8e5984e163409a5c0f7 Mon Sep 17 00:00:00 2001
>> +From: Prabhu Sundararaj <prabhu.sundararaj at nxp.com>
>> +Date: Mon, 30 Jan 2017 16:32:06 -0600
>> +Subject: [PATCH] Remove check for useSIGIO option
>> +
>> +Commit 6a5a4e60373c1386b311b2a8bb666c32d68a9d99 removes the configure of useSIGIO
>> +option.
>> +
>> +As the xfree86 SIGIO support is reworked to use internal versions of OsBlockSIGIO
>> +and OsReleaseSIGIO.
>> +
>> +No longer the check for useSIGIO is needed
>> +
>> +Upstream-Status: Pending
>> +
>> +Signed-off-by: Prabhu Sundararaj <prabhu.sundararaj at nxp.com>
>
> I think you need to add your SOB to the patch itself too:
> https://buildroot.org/downloads/manual/manual.html#_format_and_licensing_of_the_package_patches

Ah yes, thank you for the review and the link. V2 is coming!

Best regards,

-- 
Mylène Josserand, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com