Merge ab/5111269 into master

am: 816a8bb0

Change-Id: I2afd0756285435d56a592b122432959b683c566a

Android.bp

+//
+// Copyright (C) 2017 The Android Open Source Project
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//      http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+subdirs = [
+    "host",
+]

host/Android.bp

+//
+// Copyright (C) 2017 The Android Open Source Project
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//      http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+subdirs = [
+    "commands",
+    "libs",
+]
+
 cc_library_headers {
   name: "virtio_gpu_uapi_headers",
   host_supported: true,

host/commands/Android.bp

@@ -16,17 +16,3 @@
 subdirs = [
     "emugen",
 ]
-
-python_binary_host {
-  name: "gen_entries_cuttlefish",
-  srcs: ["gen_entries.py"],
-  main: "gen_entries.py",
-  version: {
-    py2: {
-      enabled: true,
-    },
-    py3: {
-      enabled: false,
-    },
-  },
-}

host/libs/virglrenderer/Android.bp

@@ -89,68 +89,66 @@ genrule {
   ],
 }
 
-// out/host/linux-x86/bin/emugen_cuttlefish -i device/generic/opengl-transport/host/libs/virglrenderer/renderControl_dec -D /tmp/foo renderControl
-
 genrule {
   name: "gles1_core_functions_hdr",
-  tools: ["gen_entries_cuttlefish"],
-  cmd: "$(location gen_entries_cuttlefish) --mode=funcargs $(in) --output $(out)",
+  tool_files: ["gen_entries.py"],
+  cmd: "python device/generic/opengl-transport/host/libs/virglrenderer/gen_entries.py --mode=funcargs $(in) --output $(out)",
   srcs: ["OpenGLESDispatch/gles1_core.entries"],
   out: ["gles1_core_functions.h"],
 }
 
 genrule {
   name: "gles1_extensions_functions_hdr",
-  tools: ["gen_entries_cuttlefish"],
-  cmd: "$(location gen_entries_cuttlefish) --mode=funcargs $(in) --output $(out)",
+  tool_files: ["gen_entries.py"],
+  cmd: "python device/generic/opengl-transport/host/libs/virglrenderer/gen_entries.py --mode=funcargs $(in) --output $(out)",
   srcs: ["OpenGLESDispatch/gles1_extensions.entries"],
   out: ["gles1_extensions_functions.h"],
 }
 
 genrule {
   name: "egl_functions_hdr",
-  tools: ["gen_entries_cuttlefish"],
-  cmd: "$(location gen_entries_cuttlefish) --mode=funcargs $(in) --output $(out)",
+  tool_files: ["gen_entries.py"],
+  cmd: "python device/generic/opengl-transport/host/libs/virglrenderer/gen_entries.py --mode=funcargs $(in) --output $(out)",
   srcs: ["OpenGLESDispatch/egl.entries"],
   out: ["egl_functions.h"],
 }
 
 genrule {
   name: "gles3_only_functions_hdr",
-  tools: ["gen_entries_cuttlefish"],
-  cmd: "$(location gen_entries_cuttlefish) --mode=funcargs $(in) --output $(out)",
+  tool_files: ["gen_entries.py"],
+  cmd: "python device/generic/opengl-transport/host/libs/virglrenderer/gen_entries.py --mode=funcargs $(in) --output $(out)",
   srcs: ["OpenGLESDispatch/gles3_only.entries"],
   out: ["gles3_only_functions.h"],
 }
 
 genrule {
   name: "gles31_only_functions_hdr",
-  tools: ["gen_entries_cuttlefish"],
-  cmd: "$(location gen_entries_cuttlefish) --mode=funcargs $(in) --output $(out)",
+  tool_files: ["gen_entries.py"],
+  cmd: "python device/generic/opengl-transport/host/libs/virglrenderer/gen_entries.py --mode=funcargs $(in) --output $(out)",
   srcs: ["OpenGLESDispatch/gles31_only.entries"],
   out: ["gles31_only_functions.h"],
 }
 
 genrule {
   name: "gles2_extensions_functions_hdr",
-  tools: ["gen_entries_cuttlefish"],
-  cmd: "$(location gen_entries_cuttlefish) --mode=funcargs $(in) --output $(out)",
+  tool_files: ["gen_entries.py"],
+  cmd: "python device/generic/opengl-transport/host/libs/virglrenderer/gen_entries.py --mode=funcargs $(in) --output $(out)",
   srcs: ["OpenGLESDispatch/gles2_extensions.entries"],
   out: ["gles2_extensions_functions.h"],
 }
 
 genrule {
   name: "egl_extensions_functions_hdr",
-  tools: ["gen_entries_cuttlefish"],
-  cmd: "$(location gen_entries_cuttlefish) --mode=funcargs $(in) --output $(out)",
+  tool_files: ["gen_entries.py"],
+  cmd: "python device/generic/opengl-transport/host/libs/virglrenderer/gen_entries.py --mode=funcargs $(in) --output $(out)",
   srcs: ["OpenGLESDispatch/egl_extensions.entries"],
   out: ["egl_extensions_functions.h"],
 }
 
 genrule {
   name: "gles2_core_functions_hdr",
-  tools: ["gen_entries_cuttlefish"],
-  cmd: "$(location gen_entries_cuttlefish) --mode=funcargs $(in) --output $(out)",
+  tool_files: ["gen_entries.py"],
+  cmd: "python device/generic/opengl-transport/host/libs/virglrenderer/gen_entries.py --mode=funcargs $(in) --output $(out)",
   srcs: ["OpenGLESDispatch/gles2_core.entries"],
   out: ["gles2_core_functions.h"],
 }
@@ -181,8 +179,6 @@ cc_library_host_shared {
     ],
     cflags: ["-Wno-unused-parameter", "-DOPENGL_DEBUG_PRINTOUT"],
     host_ldlibs: [ "-ldl" ],
-    version_script : "libvirglrenderer.lds",
-    defaults: [ "cuttlefish_host_only" ],
     generated_sources: [
       "glesv1_dec_cuttlefish_gensrc",
       "glesv3_dec_cuttlefish_gensrc",
@@ -205,4 +201,8 @@ cc_library_host_shared {
       "virtio_gpu_uapi_headers",
       "virgl_headers"
     ],
+
+    // TODO(b/118466250): Make this work on the Mac
+    version_script : "libvirglrenderer.lds",
+    defaults: [ "cuttlefish_host_only" ],
 }