Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] foomatic-filters_4.0.16.bb: update recipe to work on Fedora 17
@ 2012-07-31 20:42 Matthew McClintock
  2012-08-02 14:30 ` Richard Purdie
  0 siblings, 1 reply; 4+ messages in thread
From: Matthew McClintock @ 2012-07-31 20:42 UTC (permalink / raw)
  To: openembedded-core

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
-- 
1.7.9.7





^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] foomatic-filters_4.0.16.bb: update recipe to work on Fedora 17
  2012-07-31 20:42 [PATCH] foomatic-filters_4.0.16.bb: update recipe to work on Fedora 17 Matthew McClintock
@ 2012-08-02 14:30 ` Richard Purdie
  2012-08-02 15:54   ` McClintock Matthew-B29882
  0 siblings, 1 reply; 4+ messages in thread
From: Richard Purdie @ 2012-08-02 14:30 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

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




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] foomatic-filters_4.0.16.bb: update recipe to work on Fedora 17
  2012-08-02 14:30 ` Richard Purdie
@ 2012-08-02 15:54   ` McClintock Matthew-B29882
  2012-08-02 15:56     ` McClintock Matthew-B29882
  0 siblings, 1 reply; 4+ messages in thread
From: McClintock Matthew-B29882 @ 2012-08-02 15:54 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

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



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] foomatic-filters_4.0.16.bb: update recipe to work on Fedora 17
  2012-08-02 15:54   ` McClintock Matthew-B29882
@ 2012-08-02 15:56     ` McClintock Matthew-B29882
  0 siblings, 0 replies; 4+ messages in thread
From: McClintock Matthew-B29882 @ 2012-08-02 15:56 UTC (permalink / raw)
  To: McClintock Matthew-B29882,
	Patches and discussions about the oe-core layer

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



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-08-02 16:07 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-31 20:42 [PATCH] foomatic-filters_4.0.16.bb: update recipe to work on Fedora 17 Matthew McClintock
2012-08-02 14:30 ` Richard Purdie
2012-08-02 15:54   ` McClintock Matthew-B29882
2012-08-02 15:56     ` McClintock Matthew-B29882

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox