[ITP] gtkperf 0.40

[ITP] gtkperf 0.40

[Michael DePaulo]

The benefit to Cygwin is that we will have a commonly used[1]
benchmark for testing Cygwin/X performance.

Thank you Jon Turney for helping me package this.

https://github.com/mikedep333/gtkperf-cygport
Currently at commit e707eaa

https://apps.fedoraproject.org/packages/gtkperf
https://packages.debian.org/jessie/gtkperf

[1] It is included in the Phoronix Test Suite:
http://openbenchmarking.org/test/pts/gtkperf
Note that "1.2.1" is their version number for their packaging and
benchmarking scripts, not a new upstream gtkperf version.

----
category: X11
requires: libatk1.0_0 libgdk_pixbuf2.0_0 libglib2.0_0 libgtk2.0_0
libintl8 libpango1.0_0 font-adobe-dpi75
sdesc: "GTK+ performance test"
ldesc: "GtkPerf is an application designed to test GTK+ performance. The point
is to create common testing platform to run predefined GTK+ widgets
and this way define the speed of device/platform."

Re: [ITP] gtkperf 0.40

[Yaakov Selkowitz]

On Fri, 2015-02-27 at 08:49 -0500, Michael DePaulo wrote:
> The benefit to Cygwin is that we will have a commonly used[1]
> benchmark for testing Cygwin/X performance.
> 
> Thank you Jon Turney for helping me package this.
> 
> https://github.com/mikedep333/gtkperf-cygport
> Currently at commit e707eaa

Why the NO_AUTOPOINT=1?

You should also change the following:

-SRC_URI=http://downloads.sourceforge.net/gtkperf/gtkperf_0.40.tar.gz
+HOMEPAGE="http://gtkperf.sourceforge.net/"
+SRC_URI="mirror://sourceforge/gtkperf/gtkperf_${VERSION}.tar.gz"

${VERSION} should be used in SRC_URI to prevent mistakes when (if?) this
package gets a new version upstream.

--
Yaakov

Re: [ITP] gtkperf 0.40

[Michael DePaulo]

On Fri, Feb 27, 2015 at 12:37 PM, Yaakov Selkowitz
<yselkowitz@cygwin.com> wrote:
> On Fri, 2015-02-27 at 08:49 -0500, Michael DePaulo wrote:
>> The benefit to Cygwin is that we will have a commonly used[1]
>> benchmark for testing Cygwin/X performance.
>>
>> Thank you Jon Turney for helping me package this.
>>

Now at revision c81e08d
https://github.com/mikedep333/gtkperf-cygport

>> https://github.com/mikedep333/gtkperf-cygport
>> Currently at commit e707eaa
>
> Why the NO_AUTOPOINT=1?

Otherwise, you run into this problem with the very old autotools:

make  all-recursive
make[1]: Entering directory '/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build'
Making all in intl
make[2]: Entering directory
'/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build/intl'
make[2]: Nothing to be done for 'all'.
make[2]: Leaving directory
'/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build/intl'
Making all in po
make[2]: Entering directory
'/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build/po'
make[2]: *** No rule to make target 'Makevars', needed by 'Makefile'.  Stop.
make[2]: Leaving directory
'/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build/po'
Makefile:363: recipe for target 'all-recursive' failed
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory '/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build'
Makefile:273: recipe for target 'all' failed
make: *** [all] Error 2
*** ERROR: make failed

I added a 3 line comment about this.

>
> You should also change the following:
>
> -SRC_URI=http://downloads.sourceforge.net/gtkperf/gtkperf_0.40.tar.gz
> +HOMEPAGE="http://gtkperf.sourceforge.net/"
> +SRC_URI="mirror://sourceforge/gtkperf/gtkperf_${VERSION}.tar.gz"
>
> ${VERSION} should be used in SRC_URI to prevent mistakes when (if?) this
> package gets a new version upstream.

Done.

-Mike

Re: [ITP] gtkperf 0.40

[Michael DePaulo]

On Fri, Feb 27, 2015 at 6:00 PM, Michael DePaulo <mikedep333@gmail.com> wrote:
> On Fri, Feb 27, 2015 at 12:37 PM, Yaakov Selkowitz
> <yselkowitz@cygwin.com> wrote:
>> On Fri, 2015-02-27 at 08:49 -0500, Michael DePaulo wrote:
>>> The benefit to Cygwin is that we will have a commonly used[1]
>>> benchmark for testing Cygwin/X performance.
>>>
>>> Thank you Jon Turney for helping me package this.
>>>
>
> Now at revision c81e08d
> https://github.com/mikedep333/gtkperf-cygport
>
>>> https://github.com/mikedep333/gtkperf-cygport
>>> Currently at commit e707eaa
>>
>> Why the NO_AUTOPOINT=1?
>
> Otherwise, you run into this problem with the very old autotools:
>
> make  all-recursive
> make[1]: Entering directory '/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build'
> Making all in intl
> make[2]: Entering directory
> '/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build/intl'
> make[2]: Nothing to be done for 'all'.
> make[2]: Leaving directory
> '/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build/intl'
> Making all in po
> make[2]: Entering directory
> '/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build/po'
> make[2]: *** No rule to make target 'Makevars', needed by 'Makefile'.  Stop.
> make[2]: Leaving directory
> '/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build/po'
> Makefile:363: recipe for target 'all-recursive' failed
> make[1]: *** [all-recursive] Error 1
> make[1]: Leaving directory '/usr/src/gtkperf-cygport/gtkperf-0.40-1.i686/build'
> Makefile:273: recipe for target 'all' failed
> make: *** [all] Error 2
> *** ERROR: make failed
>
> I added a 3 line comment about this.
>
>>
>> You should also change the following:
>>
>> -SRC_URI=http://downloads.sourceforge.net/gtkperf/gtkperf_0.40.tar.gz
>> +HOMEPAGE="http://gtkperf.sourceforge.net/"
>> +SRC_URI="mirror://sourceforge/gtkperf/gtkperf_${VERSION}.tar.gz"
>>
>> ${VERSION} should be used in SRC_URI to prevent mistakes when (if?) this
>> package gets a new version upstream.
>
> Done.
>
> -Mike

Questions/Comments/Complaints?

Re: [ITP] gtkperf 0.40

[Jon TURNEY]

On 05/03/2015 12:38, Michael DePaulo wrote:
> On Fri, Feb 27, 2015 at 6:00 PM, Michael DePaulo <mikedep333-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>> On Fri, Feb 27, 2015 at 12:37 PM, Yaakov Selkowitz
>> <yselkowitz-rDBXBDvO6BXQT0dZR+AlfA@public.gmane.org> wrote:
>>> On Fri, 2015-02-27 at 08:49 -0500, Michael DePaulo wrote:
>>>> The benefit to Cygwin is that we will have a commonly used[1]
>>>> benchmark for testing Cygwin/X performance.
>>>>
>>>> Thank you Jon Turney for helping me package this.
>>
>> Now at revision c81e08d
>> https://github.com/mikedep333/gtkperf-cygport
>>
>>>> https://github.com/mikedep333/gtkperf-cygport
>>>> Currently at commit e707eaa
>>>
>>> Why the NO_AUTOPOINT=1?
>>
>> Otherwise, you run into this problem with the very old autotools:
[...]
>> make[2]: *** No rule to make target 'Makevars', needed by 'Makefile'.  Stop.
[...]

Looks like this can also be fixed just by patching to add a copy of 
Makevars.tmpl as Makevars.  Yaakov probably has a better idea than me 
about which is the better way to fix this.

Either way, GTG from me.