[oe] [PATCH] pm-qa-utils: Add power management test utils

[Maupin, Chase]

>-----Original Message-----
>From: openembedded-devel-bounces at lists.openembedded.org
>[mailto:openembedded-devel-bounces at lists.openembedded.org] On
>Behalf Of Koen Kooi
>Sent: Friday, March 21, 2014 7:11 AM
>To: openembedded-devel at lists.openembedded.org
>Subject: Re: [oe] [PATCH] pm-qa-utils: Add power management test
>utils
>
>-----BEGIN PGP SIGNED MESSAGE-----
>Hash: SHA1
>
>Maupin, Chase schreef op 21-03-14 13:07:
>
>>> -----Original Message----- From:
>>> openembedded-devel-bounces at lists.openembedded.org
>>> [mailto:openembedded-devel-bounces at lists.openembedded.org] On
>Behalf Of
>>> Koen Kooi Sent: Friday, March 21, 2014 4:10 AM To:
>>> openembedded-devel at lists.openembedded.org Subject: Re: [oe]
>[PATCH]
>>> pm-qa-utils: Add power management test utils
>>>
>> Chase Maupin schreef op 20-03-14 15:47:
>>>>> * The power management test utilities from Linaro can be used
>to
>> test
>>>>> multiple PM features such as cpufreq, cpuidle, thermal
>control
>> and
>>>>> suspend operations. * These utilities are not platform
>specific
>> and can
>>>>> be used by a broader base of users.
>>>>>
>>>>> Signed-off-by: Chase Maupin <Chase.Maupin at ti.com>
>>
>> Everything refers to this as 'pm-qa', not 'pm-qa-utils'
>>
>>> I assume you are saying you would rather see this recipe as pm-
>qa
>>> instead of pm-qa-utils?
>
>Exactly!
>
>>
>>
>>>>> --- .../pm-qa-utils/pm-qa-utils_git.bb                 |   63
>>>>> ++++++++++++++++++++ 1 file changed, 63 insertions(+) create
>> mode 100644
>>>>> meta-oe/recipes-devtools/pm-qa-utils/pm-qa-utils_git.bb
>>>>>
>>>>> diff --git a/meta-oe/recipes-devtools/pm-qa-utils/pm-qa-
>> utils_git.bb
>>>>> b/meta-oe/recipes-devtools/pm-qa-utils/pm-qa-utils_git.bb new
>> file mode
>>>>> 100644 index 0000000..610f3a7 --- /dev/null +++
>>>>> b/meta-oe/recipes-devtools/pm-qa-utils/pm-qa-utils_git.bb @@
>-
>> 0,0 +1,63
>>>>> @@ +DESCRIPTION = "Utilities from Linaro for testing Power
>> Management"
>>>>> +HOMEPAGE =
>>>>>
>>
>"https://wiki.linaro.org/WorkingGroups/PowerManagement/Resources/T
>> estSuite/PmQa"
>>>>>
>>>>>
>> +
>>>>> +LICENSE = "GPLv2" +LIC_FILES_CHKSUM =
>>>>> "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263" + +PV =
>> "0.4.4" +
>>>>> +BRANCH ?= "master" +SRCREV =
>> "fbc2762359b863dfbf4fd0bab1e8abd2a6125ed4"
>>>>> + +SRC_URI = "git://git.linaro.org/tools/pm-
>> qa.git;protocol=git;branch=${BRANCH}" + +S
>>>>> = "${WORKDIR}/git" + +CFLAGS += "-pthread" + +do_compile () {
>+
>> # Find
>>>>> all the .c files in this project and build them. +    for x
>in
>> `find .
>>>>> -name "*.c"` +    do +        util=`echo ${x} | sed s/.c$//`
>+
>>>>> oe_runmake ${util} +    done +} + +do_install () { +
>install
>> -d
>>>>> ${D}${bindir} +    install -d ${D}${bindir}/linaro-pm-qa-
>include
>>
>> Please don't use subdirs in $bindir, that's what $libdir and
>$libexecdir
>> are for.
>>
>>> Good point.  I did this because the script going into that
>directory
>>> was named "functions.sh" in the original sources.  I thought
>this was a
>>> bit generic and could possibly conflict.
>>
>>> What are your thoughts on a subdirectory under $libdir or
>$libexecdir?
>>> Any preference?
>
>Right it will amount to the basically the same due to the
>misguided
>libexecdir change oe-core did last year. If it's only used
>'internally'
>$libdir/PN would be OK, otherwise use libexecdir (which currently
>is
>libdir/PN, but might change again)

One last dense question.  What do you mean by 'internally'?

Internally to just this package as in you don't expect any sharing between this and another package?

>-----BEGIN PGP SIGNATURE-----
>Version: GnuPG v1.4.5 (Darwin)
>Comment: GPGTools - http://gpgtools.org
>
>iD8DBQFTLCxQMkyGM64RGpERAi53AJ9bjuFeJfliwQV9hzHnqd+28zkH0ACeLS9r
>VXYU8eXR2dQcBS74EnzVBd4=
>=DHAP
>-----END PGP SIGNATURE-----
>
>--
>_______________________________________________
>Openembedded-devel mailing list
>Openembedded-devel at lists.openembedded.org
>http://lists.openembedded.org/mailman/listinfo/openembedded-devel