Patchwork foomatic-filters_4.0.16.bb: update recipe to work on Fedora 17

login
register
mail settings
Submitter Matthew McClintock
Date July 31, 2012, 8:42 p.m.
Message ID <1343767324-11395-1-git-send-email-msm@freescale.com>
Download mbox | patch
Permalink /patch/33467/
State New
Headers show

Comments

Matthew McClintock - July 31, 2012, 8:42 p.m.
This fixes isses on distro that have perl in /bin/perl instead
of /usr/bin/perl such as Fedora 17

Signed-off-by: Matthew McClintock <msm@freescale.com>
---
 .../foomatic/foomatic-filters_4.0.16.bb            |    9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)
Richard Purdie - Aug. 2, 2012, 2:30 p.m.
On Tue, 2012-07-31 at 15:42 -0500, Matthew McClintock wrote:
> This fixes isses on distro that have perl in /bin/perl instead
> of /usr/bin/perl such as Fedora 17
> 
> Signed-off-by: Matthew McClintock <msm@freescale.com>
> ---
>  .../foomatic/foomatic-filters_4.0.16.bb            |    9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb b/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
> index 5f145b0..d5ae4c7 100644
> --- a/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
> +++ b/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
> @@ -10,7 +10,7 @@ format using a printer-specific, but spooler-independent PPD file. \
>  "
>  
>  DEPENDS += "cups perl libxml2"
> -PR = "r0"
> +PR = "r1"
>  
>  LICENSE = "GPLv2+"
>  LIC_FILES_CHKSUM = "file://${WORKDIR}/foomatic-filters-${PV}/COPYING;md5=393a5ca445f6965873eca0259a17f833"
> @@ -31,6 +31,13 @@ do_configure_prepend() {
>      export CUPS_PPDS=${datadir}/cups/model
>  }
>  
> +PERLPATH = "${bindir}/env perl"
> +PERLPATH_virtclass-nativesdk = "/usr/bin/env perl"
> +
> +do_install_append() {
> +    sed -i -e '1s,#!.*perl,#! ${PERLPATH},' ${D}${libdir}/cups/backend/beh
> +}
> +
>  do_install_append_linuxstdbase() {
>      install -d ${D}${exec_prefix}/lib/cups/filter
>      ln -sf ${bindir}/foomatic-rip ${D}${exec_prefix}/lib/cups/filter

Could you adjust this to account for:

http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=ac7a0311825e20c544d17bfd8be63546ad36e665

please?

Thanks,

Richard
McClintock Matthew-B29882 - Aug. 2, 2012, 3:54 p.m.
On Thu, Aug 2, 2012 at 9:30 AM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Tue, 2012-07-31 at 15:42 -0500, Matthew McClintock wrote:
>> This fixes isses on distro that have perl in /bin/perl instead
>> of /usr/bin/perl such as Fedora 17
>>
>> Signed-off-by: Matthew McClintock <msm@freescale.com>
>> ---
>>  .../foomatic/foomatic-filters_4.0.16.bb            |    9 ++++++++-
>>  1 file changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb b/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
>> index 5f145b0..d5ae4c7 100644
>> --- a/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
>> +++ b/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
>> @@ -10,7 +10,7 @@ format using a printer-specific, but spooler-independent PPD file. \
>>  "
>>
>>  DEPENDS += "cups perl libxml2"
>> -PR = "r0"
>> +PR = "r1"
>>
>>  LICENSE = "GPLv2+"
>>  LIC_FILES_CHKSUM = "file://${WORKDIR}/foomatic-filters-${PV}/COPYING;md5=393a5ca445f6965873eca0259a17f833"
>> @@ -31,6 +31,13 @@ do_configure_prepend() {
>>      export CUPS_PPDS=${datadir}/cups/model
>>  }
>>
>> +PERLPATH = "${bindir}/env perl"
>> +PERLPATH_virtclass-nativesdk = "/usr/bin/env perl"
>> +
>> +do_install_append() {
>> +    sed -i -e '1s,#!.*perl,#! ${PERLPATH},' ${D}${libdir}/cups/backend/beh
>> +}
>> +
>>  do_install_append_linuxstdbase() {
>>      install -d ${D}${exec_prefix}/lib/cups/filter
>>      ln -sf ${bindir}/foomatic-rip ${D}${exec_prefix}/lib/cups/filter
>
> Could you adjust this to account for:
>
> http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=ac7a0311825e20c544d17bfd8be63546ad36e665
>
> please?

Sure - did not see the later change on this...

Updated and pushed here:
http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=mattsm/master&id=973d055e33d5512fc4f2285c26b976ef8c21dc58

-M

>
> Thanks,
>
> Richard
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
McClintock Matthew-B29882 - Aug. 2, 2012, 3:56 p.m.
On Thu, Aug 2, 2012 at 10:54 AM, McClintock Matthew-B29882
<B29882@freescale.com> wrote:
> On Thu, Aug 2, 2012 at 9:30 AM, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
>> On Tue, 2012-07-31 at 15:42 -0500, Matthew McClintock wrote:
>>> This fixes isses on distro that have perl in /bin/perl instead
>>> of /usr/bin/perl such as Fedora 17
>>>
>>> Signed-off-by: Matthew McClintock <msm@freescale.com>
>>> ---
>>>  .../foomatic/foomatic-filters_4.0.16.bb            |    9 ++++++++-
>>>  1 file changed, 8 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb b/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
>>> index 5f145b0..d5ae4c7 100644
>>> --- a/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
>>> +++ b/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
>>> @@ -10,7 +10,7 @@ format using a printer-specific, but spooler-independent PPD file. \
>>>  "
>>>
>>>  DEPENDS += "cups perl libxml2"
>>> -PR = "r0"
>>> +PR = "r1"
>>>
>>>  LICENSE = "GPLv2+"
>>>  LIC_FILES_CHKSUM = "file://${WORKDIR}/foomatic-filters-${PV}/COPYING;md5=393a5ca445f6965873eca0259a17f833"
>>> @@ -31,6 +31,13 @@ do_configure_prepend() {
>>>      export CUPS_PPDS=${datadir}/cups/model
>>>  }
>>>
>>> +PERLPATH = "${bindir}/env perl"
>>> +PERLPATH_virtclass-nativesdk = "/usr/bin/env perl"
>>> +
>>> +do_install_append() {
>>> +    sed -i -e '1s,#!.*perl,#! ${PERLPATH},' ${D}${libdir}/cups/backend/beh
>>> +}
>>> +
>>>  do_install_append_linuxstdbase() {
>>>      install -d ${D}${exec_prefix}/lib/cups/filter
>>>      ln -sf ${bindir}/foomatic-rip ${D}${exec_prefix}/lib/cups/filter
>>
>> Could you adjust this to account for:
>>
>> http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=ac7a0311825e20c544d17bfd8be63546ad36e665
>>
>> please?
>
> Sure - did not see the later change on this...
>
> Updated and pushed here:
> http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=mattsm/master&id=973d055e33d5512fc4f2285c26b976ef8c21dc58

Err, use this one:

http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=mattsm/master&id=9e0c80ad919f63f524a51581873feda61a956b0e

-M

> -M
>
>>
>> Thanks,
>>
>> Richard
>>
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Patch

diff --git a/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb b/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
index 5f145b0..d5ae4c7 100644
--- a/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
+++ b/meta/recipes-extended/foomatic/foomatic-filters_4.0.16.bb
@@ -10,7 +10,7 @@  format using a printer-specific, but spooler-independent PPD file. \
 "
 
 DEPENDS += "cups perl libxml2"
-PR = "r0"
+PR = "r1"
 
 LICENSE = "GPLv2+"
 LIC_FILES_CHKSUM = "file://${WORKDIR}/foomatic-filters-${PV}/COPYING;md5=393a5ca445f6965873eca0259a17f833"
@@ -31,6 +31,13 @@  do_configure_prepend() {
     export CUPS_PPDS=${datadir}/cups/model
 }
 
+PERLPATH = "${bindir}/env perl"
+PERLPATH_virtclass-nativesdk = "/usr/bin/env perl"
+
+do_install_append() {
+    sed -i -e '1s,#!.*perl,#! ${PERLPATH},' ${D}${libdir}/cups/backend/beh
+}
+
 do_install_append_linuxstdbase() {
     install -d ${D}${exec_prefix}/lib/cups/filter
     ln -sf ${bindir}/foomatic-rip ${D}${exec_prefix}/lib/cups/filter