[oe] [meta-oe][PATCH] hplip: Fix build with clang

[Martin Jansa]

On Tue, Dec 20, 2016 at 01:34:46PM +0100, Martin Jansa wrote:
> On Fri, Dec 16, 2016 at 02:10:33PM -0800, Khem Raj wrote:
> > Signed-off-by: Khem Raj <raj.khem at gmail.com>
> > ---
> >  ...de-cups-ppd.h-for-missing-ppd-definitions.patch | 42 ++++++++++++++++++++++
> >  meta-oe/recipes-extended/hplip/hplip_3.12.6.bb     |  1 +
> >  2 files changed, 43 insertions(+)
> >  create mode 100644 meta-oe/recipes-extended/hplip/hplip-3.12.6/0001-include-cups-ppd.h-for-missing-ppd-definitions.patch
> > 
> > diff --git a/meta-oe/recipes-extended/hplip/hplip-3.12.6/0001-include-cups-ppd.h-for-missing-ppd-definitions.patch b/meta-oe/recipes-extended/hplip/hplip-3.12.6/0001-include-cups-ppd.h-for-missing-ppd-definitions.patch
> > new file mode 100644
> > index 0000000..f259b5f
> > --- /dev/null
> > +++ b/meta-oe/recipes-extended/hplip/hplip-3.12.6/0001-include-cups-ppd.h-for-missing-ppd-definitions.patch
> > @@ -0,0 +1,42 @@
> > +From 7ef9c040dd959105c16d4dc67e14f3bbea25e77f Mon Sep 17 00:00:00 2001
> > +From: Khem Raj <raj.khem at gmail.com>
> > +Date: Fri, 16 Dec 2016 19:51:19 +0000
> > +Subject: [PATCH] include cups/ppd.h for missing ppd definitions
> > +
> > +Fixes errors spotted by clang
> > +
> > +| prnt/hpcups/HPCupsFilter.cpp:365:18: error: use of undeclared identifier 'ppdFindAttr'
> > +|     if (((attr = ppdFindAttr(m_ppd, "hpPrinterLanguage", NULL)) == NULL) ||
> > +|                  ^
> > +| prnt/hpcups/HPCupsFilter.cpp:368:13: error: use of undeclared identifier 'ppdClose'
> > +|             ppdClose(m_ppd);
> > +|             ^
> > +| prnt/hpcups/HPCupsFilter.cpp:444:9: error: use of undeclared identifier 'ppdClose'
> > +|         ppdClose(m_ppd);
> > +|         ^
> > +| prnt/hpcups/HPCupsFilter.cpp:497:13: error: use of undeclared identifier 'ppdOpenFile'
> > +|     m_ppd = ppdOpenFile(getenv("PPD"));
> > +|             ^
> > +
> > +Signed-off-by: Khem Raj <raj.khem at gmail.com>
> > +---
> > +Upstream-Status: Pending
> > +
> > + prnt/hpcups/HPCupsFilter.h | 1 +
> > + 1 file changed, 1 insertion(+)
> > +
> > +diff --git a/prnt/hpcups/HPCupsFilter.h b/prnt/hpcups/HPCupsFilter.h
> > +index 0431a7a..eb0cad6 100644
> > +--- a/prnt/hpcups/HPCupsFilter.h
> > ++++ b/prnt/hpcups/HPCupsFilter.h
> > +@@ -37,6 +37,7 @@
> > + #include "Job.h"
> > + 
> > + #include "dbuscomm.h"
> > ++#include <cups/ppd.h>
> > + 
> > + #define		DBITMAPFILEHEADER		14
> > + #define		DBITMAPINFOHEADER		40
> > +-- 
> > +1.9.1
> > +
> > diff --git a/meta-oe/recipes-extended/hplip/hplip_3.12.6.bb b/meta-oe/recipes-extended/hplip/hplip_3.12.6.bb
> > index 9848e5a..1f1e80c 100644
> > --- a/meta-oe/recipes-extended/hplip/hplip_3.12.6.bb
> > +++ b/meta-oe/recipes-extended/hplip/hplip_3.12.6.bb
> > @@ -9,6 +9,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BP}.tar.gz \
> >             file://fix-libusb-paths.patch \
> >             file://cups-1.6.patch \
> >             file://configure.patch \
> > +           file://0001-include-cups-ppd.h-for-missing-ppd-definitions.patch \
> >  "
> 
> It failed to build even without clang (when using default toolchain),
> but this patch fails to apply:
> 
> http://logs.nslu2-linux.org/buildlogs/oe/world/pyro/log.dependencies.qemuarm.20161218_140914.log//2_max/failed/hplip.log

I've cherry-picked this directly from you branch and it fixed the line
endings in the patch and might fix to application of the .patch, lets
see if it fixes also the build with default toolchain.

> 
> >  
> >  DEPENDS += "cups python libusb"
> > -- 
> > 2.10.2
> > 
> > -- 
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel at lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> 
> -- 
> Martin 'JaMa' Jansa     jabber: Martin.Jansa at gmail.com



-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20161221/15d60bf6/attachment-0002.sig>