Bug 513350 – app-benchmarks/voglperf - Benchmarking tool for Linux OpenGL games

• Status: UNCONFIRMED
• Product: Gentoo Linux

[Description C. Wijtmans 2014-06-15 15:04:50 UTC]

Package source located at:

https://github.com/ValveSoftware/voglperf

i will give an attempt at an ebuild it should be easy right...

[Comment 1 C. Wijtmans 2014-06-15 15:55:00 UTC]

Created attachment 378950 [details]
voglperf-9999.ebuild

i have no clue how to work the multilib installs but this is what the website says:

"to build amd64 and i386 packages:
make

To build just i386:
make voglperf32

To build just amd64:
make voglperf64"

[Comment 2 C. Wijtmans 2014-06-15 16:37:02 UTC]

Created attachment 378952 [details]
voglperf-9999.ebuild

[Comment 3 C. Wijtmans 2014-06-15 16:37:24 UTC]

Created attachment 378954 [details]
voglperf-9999.ebuild

[Comment 4 C. Wijtmans 2014-06-16 18:57:50 UTC]

Created attachment 379056 [details]
voglperf-9999.ebuild

I think i am almost done, what do you think?
Might need to add some dependencies though.

[Comment 5 C. Wijtmans 2014-06-16 18:59:26 UTC]

How do i rename the libs to remove the 32 or 64? Is there a functionality for that or should i use the mv command?

[Comment 6 C. Wijtmans 2014-06-16 19:00:48 UTC]

(In reply to C.J. Wijtmans from comment #5)
> How do i rename the libs to remove the 32 or 64? Is there a functionality
> for that or should i use the mv command?

scratch that. i dont think that should be done for various reasons.

[Comment 7 C. Wijtmans 2014-06-16 20:17:45 UTC]

Created attachment 379086 [details]
voglperf-9999.ebuild

with dependencies. i hope this is proper.

[Comment 8 C. Wijtmans 2014-06-16 20:27:26 UTC]

Created attachment 379092 [details]
voglperf-9999.ebuild

[Comment 9 Jeroen Roovers (RETIRED) 2014-06-16 21:00:19 UTC]

Comment on attachment 379092 [details]
voglperf-9999.ebuild

>HOMEPAGE="github.com/ValveSoftware/voglperf"

That's not a URL. It needs an https:// (in this case).

>SRC_URI=""

You don't need it. Just leave SRC_URI out.

>DEPEND="libedit ? ( dev-libs/libedit[abi_x86_32?,abi_x86_64?] )
>    dev-util/cmake

inherit cmake-utils

>    sys-devel/gcc[cxx]

Implicit, don't set it in DEPEND.

>    sys-utils/glibc"

Likewise.

>KEYWORDS="~amd64 ~x86"
>IUSE="+abi_x86_32 +abi_x86_64 tinfo libedit"

Why do you need USE=tinfo? 

>src_compile() {
>    use abi_x86_32 && make voglperf32
>    use abi_x86_64 && make voglperf64

inherit cmake-utils multilib should take care of both, but this probably needs some special make targets as you have them already.
Also, use emake not make

>src_install() {
>    if use abi_x86_32; then
>        dobin bin/voglperfrun32
>        dobin bin/libvoglperf32.so

dolib maybe? Why would you put that library in /usr/bin?

>    fi
>    if use abi_x86_64; then
>        dobin bin/voglperfrun64
>        dobin bin/libvoglperf64.so

Likewise.

[Comment 10 C. Wijtmans 2014-06-16 21:16:29 UTC]

(In reply to Jeroen Roovers from comment #9)
> Comment on attachment 379092 [details]

Thank you. about the dobin isntead of dolib. I already reported the bug upstream.I thought a few things would be implicit but i wasnt sure.

[Comment 11 C. Wijtmans 2014-06-16 21:23:40 UTC]

the listed tinfo as a dependency but i i dont see it required on my end.
Still missing gl(mesa?) and libx11 libcb deps

[Comment 12 C. Wijtmans 2014-06-16 22:57:33 UTC]

i tried CMAKE_USE_DIR="src" and BUILD_DIR="./" but i just cant get it to work.