apitrace: new package

PATCH Superseded Jérôme Pouiller 4 participants 5 messages Jan 2, 2014 – Jan 10, 2014

lore

Share & tools

lore

Jérôme Pouiller

author

2014-01-02 21:27 UTC #0 lore raw

PATCH v2 1/1 apitrace: new package

Trace and replay OpenGL and OpenGL ES APIs calls to/from a file.

http://apitrace.github.io/

Signed-off-by J?r?me Pouiller <[email protected]>

---
v2:
Fix dependencies problems (identified by Samuel)

 package/Config.in            |  1 +
 package/apitrace/Config.in   | 14 ++++++++++++++
 package/apitrace/apitrace.mk | 17 +++++++++++++++++
 3 files changed, 32 insertions(+)
 create mode 100644 package/apitrace/Config.in
 create mode 100644 package/apitrace/apitrace.mk

package/Config.in +1 -0

index 1ee60c1..097b487 100644
@@ -216,6 +216,7 @@ source "package/weston/Config.in"
 
 comment "X applications"
 source "package/alsamixergui/Config.in"
+source "package/apitrace/Config.in"
 source "package/docker/Config.in"
 source "package/feh/Config.in"
 source "package/freerdp/Config.in"

package/apitrace/Config.in new file +14 -0

new file mode 100644
index 0000000..388b3e9
@@ -0,0 +1,14 @@
+comment "apitrace needs a toolchain w/ C++"
+	depends on !BR2_INSTALL_LIBSTDCPP
+
+config BR2_PACKAGE_APITRACE
+	bool "apitrace"
+	depends on BR2_PACKAGE_XORG7
+	depends on BR2_INSTALL_LIBSTDCPP
+	select BR2_PACKAGE_XLIB_LIBX11
+	help
+	  Trace and replay OpenGL and OpenGL ES APIs calls to/from a file. You 
+	  may install apitrace GUI from your distribution to inspect generated 
+	  traces.
+	  
+	  http://apitrace.github.io/

package/apitrace/apitrace.mk new file +17 -0

new file mode 100644
index 0000000..c31a5b3
@@ -0,0 +1,17 @@
+################################################################################
+#
+# apitrace
+#
+################################################################################
+
+APITRACE_VERSION = c181b7fbf4b1c3912424761fec8ac7124640543a
+APITRACE_SITE = http://github.com/apitrace/apitrace/tarball/$(APITRACE_VERSION)
+APITRACE_LICENSE = MIT
+APITRACE_LICENSE_FILES = LICENSE
+
+APITRACE_DEPENDENCIES = xlib_libX11 host-python 
+
+# Gui was never tested, so we prefer to explicitly disable it
+APITRACE_CONF_OPT += -DENABLE_GUI=false
+
+$(eval $(cmake-package))

Signature · 1 line

1.8.3.2

Jerzy Grzegorek

2014-01-02 22:20 UTC #1 lore raw

PATCH v2 1/1 apitrace: new package

Hi J?r?me,

Quoted 54 lines

Trace and replay OpenGL and OpenGL ES APIs calls to/from a file.

http://apitrace.github.io/

Signed-off-by J?r?me Pouiller <[email protected]>

---
v2:
Fix dependencies problems (identified by Samuel)

  package/Config.in            |  1 +
  package/apitrace/Config.in   | 14 ++++++++++++++
  package/apitrace/apitrace.mk | 17 +++++++++++++++++
  3 files changed, 32 insertions(+)

create mode 100644 package/apitrace/Config.in
create mode 100644 package/apitrace/apitrace.mk

package/Config.in +1 -0

index 1ee60c1..097b487 100644
@@ -216,6 +216,7 @@ source "package/weston/Config.in"
 
  comment "X applications"
  source "package/alsamixergui/Config.in"
+source "package/apitrace/Config.in"
  source "package/docker/Config.in"
  source "package/feh/Config.in"
  source "package/freerdp/Config.in"

package/apitrace/Config.in new file +14 -0

new file mode 100644
index 0000000..388b3e9
@@ -0,0 +1,14 @@
+comment "apitrace needs a toolchain w/ C++"
+	depends on !BR2_INSTALL_LIBSTDCPP
+
+config BR2_PACKAGE_APITRACE
+	bool "apitrace"
+	depends on BR2_PACKAGE_XORG7
+	depends on BR2_INSTALL_LIBSTDCPP
+	select BR2_PACKAGE_XLIB_LIBX11
+	help
+	  Trace and replay OpenGL and OpenGL ES APIs calls to/from a file. You
+	  may install apitrace GUI from your distribution to inspect generated
+	  traces.
+	
+	  http://apitrace.github.io/

package/apitrace/apitrace.mk new file +0 -0

new file mode 100644
index 0000000..c31a5b3

package/apitrace/apitrace.mk line 8

Quoted 8 lines

package/apitrace/apitrace.mk new file +8 -0

@@ -0,0 +1,17 @@
+################################################################################
+#
+# apitrace
+#
+################################################################################
+
+APITRACE_VERSION = c181b7fbf4b1c3912424761fec8ac7124640543a
+APITRACE_SITE = http://github.com/apitrace/apitrace/tarball/$(APITRACE_VERSION)

I think, we should use github download helper APITRACE_SITE = $(call github,apitrace,apitrace,$(APITRACE_VERSION))

Regards,
Jerzy

Quoted 9 lines

package/apitrace/apitrace.mk new file +9 -0

+APITRACE_LICENSE = MIT
+APITRACE_LICENSE_FILES = LICENSE
+
+APITRACE_DEPENDENCIES = xlib_libX11 host-python
+
+# Gui was never tested, so we prefer to explicitly disable it
+APITRACE_CONF_OPT += -DENABLE_GUI=false
+
+$(eval $(cmake-package))

Jérôme Pouiller

author

2014-01-03 09:01 UTC #2 lore raw

PATCH v2 1/1 apitrace: new package

Hello Jerzy,

Quoted 1 line · On Thu 02 Jan 2014 23:20:22 Jerzy Grzegorek wrote:

Hi J?r?me,

[...]

Quoted 5 lines

Quoted 3 lines
(quoted diff) +2 -0
+APITRACE_VERSION = c181b7fbf4b1c3912424761fec8ac7124640543a
+APITRACE_SITE =
http://github.com/apitrace/apitrace/tarball/$(APITRACE_VERSION)
I think, we should use github download helper APITRACE_SITE = $(call github,apitrace,apitrace,$(APITRACE_VERSION))

hmm... This function is not yet included in master branch, is it? Do you think I should still base my patch on?

Signature · 3 lines

J?r?me Pouiller, Sysmic
Embedded Linux specialist
http://www.sysmic.fr

Baruch Siach

2014-01-03 09:05 UTC #3 lore raw

PATCH v2 1/1 apitrace: new package

Hi J?r?me,

Quoted 11 lines · J?r?me Pouiller

Quoted 1 line · On Thu 02 Jan 2014 23:20:22 Jerzy Grzegorek wrote:
Hi J?r?me,
[...]
Quoted 5 lines
Quoted 3 lines
(quoted diff) +2 -0
+APITRACE_VERSION = c181b7fbf4b1c3912424761fec8ac7124640543a
+APITRACE_SITE =
http://github.com/apitrace/apitrace/tarball/$(APITRACE_VERSION)
I think, we should use github download helper APITRACE_SITE = $(call github,apitrace,apitrace,$(APITRACE_VERSION))
hmm... This function is not yet included in master branch, is it? Do you think I should still base my patch on?

It's in master branch since commit bb083e95cfae9 (infra: introduce github helper function).

baruch

Signature · 3 lines

     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -

Peter Korsgaard

2014-01-10 22:55 UTC #4 lore raw

PATCH v2 1/1 apitrace: new package

In reply to Jérôme Pouiller

Quoted 35 lines

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
"J?r?me" == J?r?me Pouiller <[email protected]> writes:
Trace and replay OpenGL and OpenGL ES APIs calls to/from a file.
http://apitrace.github.io/
Signed-off-by J?r?me Pouiller <[email protected]>
---
v2:
Fix dependencies problems (identified by Samuel)
 package/Config.in            |  1 +
 package/apitrace/Config.in   | 14 ++++++++++++++
 package/apitrace/apitrace.mk | 17 +++++++++++++++++
 3 files changed, 32 insertions(+)
 create mode 100644 package/apitrace/Config.in
 create mode 100644 package/apitrace/apitrace.mk
package/Config.in +1 -0
index 1ee60c1..097b487 100644
@@ -216,6 +216,7 @@ source "package/weston/Config.in"
 
 comment "X applications"
 source "package/alsamixergui/Config.in"
+source "package/apitrace/Config.in"
 source "package/docker/Config.in"
 source "package/feh/Config.in"
 source "package/freerdp/Config.in"
package/apitrace/Config.in new file +0 -0
new file mode 100644
index 0000000..388b3e9

package/apitrace/Config.in line 2

Quoted 2 lines

package/apitrace/Config.in new file +2 -0

@@ -0,0 +1,14 @@
+comment "apitrace needs a toolchain w/ C++"
+	depends on !BR2_INSTALL_LIBSTDCPP

This comment should also depend on _XORG7 so it only shows up if X11 support is enabled.

package/apitrace/Config.in line 14

Quoted 12 lines

package/apitrace/Config.in new file +12 -0

+
+config BR2_PACKAGE_APITRACE
+	bool "apitrace"
+	depends on BR2_PACKAGE_XORG7
+	depends on BR2_INSTALL_LIBSTDCPP
+	select BR2_PACKAGE_XLIB_LIBX11
+	help
+	  Trace and replay OpenGL and OpenGL ES APIs calls to/from a file. You 
+	  may install apitrace GUI from your distribution to inspect generated 
+	  traces.
+	  
+	  http://apitrace.github.io/

Some trailing spaces and too long help text lines.

I did a test build on uClibc which failed because it tries to use the glibc specific __libc_dlsym symbol, so I made the package depend on BR2_TOOLCHAIN_USES_GLIBC.

Quoted 6 lines

package/apitrace/apitrace.mk new file +0 -0
new file mode 100644
index 0000000..c31a5b3

package/apitrace/apitrace.mk line 8

Quoted 8 lines

package/apitrace/apitrace.mk new file +8 -0

@@ -0,0 +1,17 @@
+################################################################################
+#
+# apitrace
+#
+################################################################################
+
+APITRACE_VERSION = c181b7fbf4b1c3912424761fec8ac7124640543a
+APITRACE_SITE = http://github.com/apitrace/apitrace/tarball/$(APITRACE_VERSION)

I've changed it to use the github helper.

Committed with these changes, thanks.

Signature · 1 line

Bye, Peter Korsgaard