PR#49774: Ticket 48204 - remove python2 from scripts - 389-ds-base

389-ds-base

#49774 Ticket 48204 - remove python2 from scripts

Closed 3 years ago by spichugi. Opened 5 years ago by mreynolds.

mreynolds/389-ds-base ticket48204 into master

Ticket 48204 - remove python2 from scripts

Mark Reynolds • 5 years ago

0d7259f

configure.ac

file modified

+3 -3

		`@@ -545,12 +545,12 @@`
		`if test "$with_pythonexec" = yes ; then`
		`AC_MSG_ERROR([You must specify --with-pythonexec=/full/path/to/python])`
		`elif test "$with_pythonexec" = no ; then`
		`- with_pythonexec=/usr/bin/python2`
		`+ with_pythonexec=/usr/bin/python3`
		`else`
		`AC_MSG_RESULT([$with_pythonexec])`
		`fi`
		`else`
		`- with_pythonexec=/usr/bin/python2`
		`+ with_pythonexec=/usr/bin/python3`
		`fi`

		`AC_SUBST(prefixdir)`
		`@@ -630,7 +630,7 @@`
		`if test -n "$with_pythonexec"; then`
		`pythonexec="$with_pythonexec"`
		`else`
		`- pythonexec='/usr/bin/env python2'`
		`+ pythonexec='/usr/bin/python3'`
		`fi`

		`# Default to no atomic queue operations.`

mreynolds commented 5 years ago

Description: Change the @pythonexec@ macro to use python3

https://pagure.io/389-ds-base/issue/48204

Reviewed by: ?

vashirov commented on line 21 of configure.ac 5 years ago

We should not use /usr/bin/env as an interpreter, because RPM does dependency additions based on shebang line. In this case we would depend on env instead of python3. Instead we should use /usr/bin/python3 explicitly.