mbox series

[meta-security,kirkstone,0/5] sssd: fix python3 support

Message ID 20240626133913.790145-1-gael.portay@rtone.fr
Headers show
Series sssd: fix python3 support | expand

Message

Gaël PORTAY June 26, 2024, 1:39 p.m. UTC 
Dear Maintainers,

This patch series brings several fixes to enable the support for python3
to the recipe sssd.

I have tested the build on the branch kirkstone with following change to
enable python3:

	PACKAGECONFIG:append:pn-sssd = " python3"

The command below runs successfully since then, however, I have not
tested it on scarthgap (yet?).

	$ bitbake sssd

The patch series creates the new target package sssd-python; tell me if
is not misnamed (thinking about python-sssd?) or if the python script
and its modules should be part of the target package sssd instead.

Also, tell me if you want me to apply these changes on top of master.

Important: I am no expert in python packaging at all, so I have probably
made not-the-right choices in the first commit! Sorry...

Kind Regards,
Gaël PORTAY (5):
  sssd: fix missing python3.X-config script
  sssd: fix ac_cv_prog_HAVE_PYTHON3 value
  sssd: fix issue if build machine is Debian
  sssd: fix shipping python script and modules
  sssd: fix path to python3 interpreter

 recipes-security/sssd/sssd_2.5.2.bb | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

Comments

Gaël PORTAY July 1, 2024, 12:44 p.m. UTC | #1 
Hello,

On Wed Jun 26, 2024 at 3:39 PM CEST, =?UTF-8?q?Ga=C3=ABl=20PORTAY?= wrote:
> Dear Maintainers,
>
> This patch series brings several fixes to enable the support for python3
> to the recipe sssd.
>
> I have tested the build on the branch kirkstone with following change to
> enable python3:
>
> 	PACKAGECONFIG:append:pn-sssd = " python3"
>
> The command below runs successfully since then, however, I have not
> tested it on scarthgap (yet?).
>
> 	$ bitbake sssd
>
> The patch series creates the new target package sssd-python; tell me if
> is not misnamed (thinking about python-sssd?) or if the python script
> and its modules should be part of the target package sssd instead.
>
> Also, tell me if you want me to apply these changes on top of master.
>
> Important: I am no expert in python packaging at all, so I have probably
> made not-the-right choices in the first commit! Sorry...
>
> Kind Regards,
> Gaël PORTAY (5):
>   sssd: fix missing python3.X-config script
>   sssd: fix ac_cv_prog_HAVE_PYTHON3 value
>   sssd: fix issue if build machine is Debian

Armin, do you want me to send these three changes on top of master, even
if they have not been tested on that branch?

>   sssd: fix shipping python script and modules
>   sssd: fix path to python3 interpreter

(that change "cleanly" applies I guess).

>
>  recipes-security/sssd/sssd_2.5.2.bb | 14 ++++++++++++--
>  1 file changed, 12 insertions(+), 2 deletions(-)

Regards,
Gaël