[GTK] Use ${PYTHON_EXECUTABLE} to run generate-gtkdoc
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 21 Mar 2020 15:22:05 +0000 (15:22 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 21 Mar 2020 15:22:05 +0000 (15:22 +0000)
https://bugs.webkit.org/show_bug.cgi?id=208970

Patch by Michael Catanzaro <mcatanzaro@gnome.org> on 2020-03-21
Reviewed by Konstantin Tokarev.

.:

There's no need to rely on shebangs when executing python scripts from CMake. This should
ensure the script always works even if the shebang does not.

* Source/cmake/GtkDoc.cmake:

Tools:

We are allowed to use /usr/bin/env in this script because it's not installed (so not
packaged). Therefore we can make the shebang compatible with FreeBSD in this particularl
case.

* gtkdoc/generate-gtkdoc:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@258808 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
Source/cmake/GtkDoc.cmake
Tools/ChangeLog
Tools/gtkdoc/generate-gtkdoc

index 736a06a..3c53ba9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2020-03-21  Michael Catanzaro  <mcatanzaro@gnome.org>
+
+        [GTK] Use ${PYTHON_EXECUTABLE} to run generate-gtkdoc
+        https://bugs.webkit.org/show_bug.cgi?id=208970
+
+        Reviewed by Konstantin Tokarev.
+
+        There's no need to rely on shebangs when executing python scripts from CMake. This should
+        ensure the script always works even if the shebang does not.
+
+        * Source/cmake/GtkDoc.cmake:
+
 2020-03-20  Justin Michaud  <justin@justinmichaud.com>
 
         Fix JSCOnly build without unified sources
index b0fb31c..18e8644 100644 (file)
@@ -4,7 +4,7 @@ macro(ADD_GTKDOC_GENERATOR _stamp_name _extra_args)
     add_custom_command(
         OUTPUT "${CMAKE_BINARY_DIR}/${_stamp_name}"
         DEPENDS ${DocumentationDependencies}
-        COMMAND ${CMAKE_COMMAND} -E env "CC=${CMAKE_C_COMPILER}" "CFLAGS=${CMAKE_C_FLAGS} -Wno-unused-parameter" "LDFLAGS=${CMAKE_EXE_LINKER_FLAGS}" ${CMAKE_SOURCE_DIR}/Tools/gtkdoc/generate-gtkdoc ${_extra_args}
+        COMMAND ${CMAKE_COMMAND} -E env "CC=${CMAKE_C_COMPILER}" "CFLAGS=${CMAKE_C_FLAGS} -Wno-unused-parameter" "LDFLAGS=${CMAKE_EXE_LINKER_FLAGS}" ${PYTHON_EXECUTABLE} ${CMAKE_SOURCE_DIR}/Tools/gtkdoc/generate-gtkdoc ${_extra_args}
         COMMAND touch ${_stamp_name}
         WORKING_DIRECTORY "${CMAKE_BINARY_DIR}"
         VERBATIM
index 1259f74..d39ed4f 100644 (file)
@@ -1,3 +1,16 @@
+2020-03-21  Michael Catanzaro  <mcatanzaro@gnome.org>
+
+        [GTK] Use ${PYTHON_EXECUTABLE} to run generate-gtkdoc
+        https://bugs.webkit.org/show_bug.cgi?id=208970
+
+        Reviewed by Konstantin Tokarev.
+
+        We are allowed to use /usr/bin/env in this script because it's not installed (so not
+        packaged). Therefore we can make the shebang compatible with FreeBSD in this particularl
+        case.
+
+        * gtkdoc/generate-gtkdoc:
+
 2020-03-20  Jonathan Bedard  <jbedard@apple.com>
 
         run-webkit-tests: Report MacCatalyst as a unique platform
index a38a02f..efc74d2 100755 (executable)
@@ -1,4 +1,4 @@
-#!/usr/bin/python3
+#!/usr/bin/env python3
 # Copyright (C) 2011 Igalia S.L.
 #
 # This library is free software; you can redistribute it and/or