pls comment on weston non-PAM patch (link inside)

pls comment on weston non-PAM patch (link inside)

[Alexander Kanavin]

[-- Attachment #1: Type: text/plain, Size: 217 bytes --]

Hello folks,

I sent the weston non-PAM patch upstream and they have raised questions
about its validity, can you please comment on it?
https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/725

Thanks,
Alex

[-- Attachment #2: Type: text/html, Size: 448 bytes --]

Re: pls comment on weston non-PAM patch (link inside)

[Alexander Kanavin]

[-- Attachment #1: Type: text/plain, Size: 517 bytes --]

Hello guys,

reminding you to speak up in the patch submission; if you don't, upstream
will close it without merging, and I will start working towards removing
the patch from Yocto as well.

Alex

On Fri, 12 Nov 2021 at 10:48, Alexander Kanavin <alex.kanavin@gmail.com>
wrote:

> Hello folks,
>
> I sent the weston non-PAM patch upstream and they have raised questions
> about its validity, can you please comment on it?
> https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/725
>
> Thanks,
> Alex
>
>
>
>

[-- Attachment #2: Type: text/html, Size: 1091 bytes --]

RE: pls comment on weston non-PAM patch (link inside)

[Tom Hochstein]

[-- Attachment #1: Type: text/plain, Size: 1669 bytes --]

Thanks Alex. @Otavio<mailto:otavio.salvador@ossystems.com.br> pushed the hardest for the use case and I don't really have anything to add. Given that Pekka has suggested that there is a better approach without weston-launch, it seems that removing the patch is fine. In the worst case we can backport the patch to meta-freescale until we have a new solution.

Tom

From: Alexander Kanavin <alex.kanavin@gmail.com>
Sent: Friday, November 19, 2021 11:06 AM
To: OE-core <openembedded-core@lists.openembedded.org>; Tom Hochstein <tom.hochstein@nxp.com>; Denys Dmytriyenko <denis@denix.org>
Cc: ppaalanen@gmail.com
Subject: Re: pls comment on weston non-PAM patch (link inside)

Hello guys,

reminding you to speak up in the patch submission; if you don't, upstream will close it without merging, and I will start working towards removing the patch from Yocto as well.

Alex

On Fri, 12 Nov 2021 at 10:48, Alexander Kanavin <alex.kanavin@gmail.com<mailto:alex.kanavin@gmail.com>> wrote:
Hello folks,

I sent the weston non-PAM patch upstream and they have raised questions about its validity, can you please comment on it?
https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/725<https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.freedesktop.org%2Fwayland%2Fweston%2F-%2Fmerge_requests%2F725&data=04%7C01%7Ctom.hochstein%40nxp.com%7C6a003f79d8a44b3beaf008d9ab7ee8ad%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637729383818649026%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=mOGax0pBnQjJBMrDsfv6bv%2FgZ96u%2Fd5UcP3wZDph36Q%3D&reserved=0>

Thanks,
Alex




[-- Attachment #2: Type: text/html, Size: 5068 bytes --]

Re: pls comment on weston non-PAM patch (link inside)

[Denys Dmytriyenko]

On Fri, Nov 19, 2021 at 05:58:42PM +0000, Tom Hochstein wrote:
> Thanks Alex. @Otavio<mailto:otavio.salvador@ossystems.com.br> pushed the 
> hardest for the use case and I don't really have anything to add. Given that 
> Pekka has suggested that there is a better approach without weston-launch, 
> it seems that removing the patch is fine. In the worst case we can backport 
> the patch to meta-freescale until we have a new solution.

I would agree - if upstream supports our use case differently, we should 
probably switch to using it and drop the patch. That would be less burden 
updating it between the versions...

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964