[pkgsrc/trunk]: pkgsrc/print/hpijs Drop ONLY_FOR_COMPILER. Fix a bunch of war...

[joerg <joerg%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/6a091fbcd804
branches:  trunk
changeset: 650776:6a091fbcd804
user:      joerg <joerg%pkgsrc.org@localhost>
date:      Sun Apr 26 13:28:23 2015 +0000

description:
Drop ONLY_FOR_COMPILER. Fix a bunch of warnings from clang, missing
includes warranting a revision bump.

diffstat:

 print/hpijs/Makefile                       |   5 ++---
 print/hpijs/distinfo                       |   6 +++++-
 print/hpijs/patches/patch-djgenericvip.cpp |  22 ++++++++++++++++++++++
 print/hpijs/patches/patch-hpijs.cpp        |  24 ++++++++++++++++++++++++
 print/hpijs/patches/patch-ldlencap.h       |  13 +++++++++++++
 print/hpijs/patches/patch-ljfastraster.cpp |  20 ++++++++++++++++++++
 6 files changed, 86 insertions(+), 4 deletions(-)

diffs (133 lines):

diff -r 48f19c720d05 -r 6a091fbcd804 print/hpijs/Makefile
--- a/print/hpijs/Makefile      Sun Apr 26 13:26:57 2015 +0000
+++ b/print/hpijs/Makefile      Sun Apr 26 13:28:23 2015 +0000
@@ -1,7 +1,7 @@
-# $NetBSD: Makefile,v 1.30 2012/10/08 13:45:39 asau Exp $
+# $NetBSD: Makefile,v 1.31 2015/04/26 13:28:23 joerg Exp $
 
 DISTNAME=      hpijs-2.1.4
-PKGREVISION=   2
+PKGREVISION=   3
 CATEGORIES=    print
 MASTER_SITES=  ${MASTER_SITE_SOURCEFORGE:=hpinkjet/}
 
@@ -16,7 +16,6 @@
 CPPFLAGS+=             -DAPDK_BUFFER_SEND
 
 USE_TOOLS+=            gzip
-ONLY_FOR_COMPILER=     gcc
 INSTALLATION_DIRS=     bin share/ppd share/doc/html/hpijs
 
 do-install:
diff -r 48f19c720d05 -r 6a091fbcd804 print/hpijs/distinfo
--- a/print/hpijs/distinfo      Sun Apr 26 13:26:57 2015 +0000
+++ b/print/hpijs/distinfo      Sun Apr 26 13:28:23 2015 +0000
@@ -1,4 +1,4 @@
-$NetBSD: distinfo,v 1.9 2006/01/29 01:51:23 gavan Exp $
+$NetBSD: distinfo,v 1.10 2015/04/26 13:28:23 joerg Exp $
 
 SHA1 (hpijs-2.1.4.tar.gz) = 8b8baa2d410f8918ae2b8381811983c923b29703
 RMD160 (hpijs-2.1.4.tar.gz) = b498a578d89bdd2dcc12d2df846f9950df67db8d
@@ -6,3 +6,7 @@
 SHA1 (patch-aa) = 1d89574d8f54cec6493304a663c0f546bd8df111
 SHA1 (patch-ab) = cf2224faf7bb81b6d656a3fe19c668413a54707c
 SHA1 (patch-ac) = b6bd6df7a30152df93d579e12d2b465f15c130c8
+SHA1 (patch-djgenericvip.cpp) = 1ad602d1086c7972e240646e5eaed199b60c63a5
+SHA1 (patch-hpijs.cpp) = 0be9dce9ba7470ff5ddf1d74a02e058aaea6605e
+SHA1 (patch-ldlencap.h) = e768093c0208f850ca0500259f8539dcf028be91
+SHA1 (patch-ljfastraster.cpp) = 34860e7daafcd47ae5a8506b477e320ee3f7eb2e
diff -r 48f19c720d05 -r 6a091fbcd804 print/hpijs/patches/patch-djgenericvip.cpp
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/print/hpijs/patches/patch-djgenericvip.cpp        Sun Apr 26 13:28:23 2015 +0000
@@ -0,0 +1,22 @@
+$NetBSD: patch-djgenericvip.cpp,v 1.1 2015/04/26 13:28:23 joerg Exp $
+
+--- djgenericvip.cpp.orig      2015-04-26 12:32:41.000000000 +0000
++++ djgenericvip.cpp
+@@ -371,7 +371,7 @@ BOOL DJGenericVIP::HagakiFeedPresent(BOO
+     // skip over ";S:<version=2bytes><topcover><inklid><duplexer>"
+     pStr += 8;
+     BYTE b = *pStr;
+-    return (b & 4 == 4);
++    return (b & 4) == 4;
+ }
+ 
+ #ifdef APDK_AUTODUPLEX
+@@ -402,7 +402,7 @@ BOOL DJGenericVIP::HagakiFeedDuplexerPre
+     // skip over ";S:<version=2bytes><topcover><inklid>"
+     pStr += 6;
+     BYTE b = *pStr;
+-    return (b & 4 == 4);
++    return (b & 4) == 4;
+ }
+ #endif
+ 
diff -r 48f19c720d05 -r 6a091fbcd804 print/hpijs/patches/patch-hpijs.cpp
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/print/hpijs/patches/patch-hpijs.cpp       Sun Apr 26 13:28:23 2015 +0000
@@ -0,0 +1,24 @@
+$NetBSD: patch-hpijs.cpp,v 1.1 2015/04/26 13:28:23 joerg Exp $
+
+--- hpijs.cpp.orig     2015-04-26 12:31:06.000000000 +0000
++++ hpijs.cpp
+@@ -56,8 +56,8 @@ int bug(const char *fmt, ...)
+    if ((n = vsnprintf(buf, 256, fmt, args)) == -1)
+       buf[255] = 0;     /* output was truncated */
+ 
+-   fprintf(stderr, buf);
+-   syslog(LOG_WARNING, buf);
++   fprintf(stderr, "%s", buf);
++   syslog(LOG_WARNING, "%s", buf);
+ 
+    fflush(stderr);
+    va_end(args);
+@@ -357,7 +357,7 @@ int hpijs_get_cb(void *get_cb_data, IjsS
+    }
+    else if (!strcmp (key, "ColorSpace"))
+    {
+-      return snprintf(value_buf, value_size, pSS->ph.cs);
++      return snprintf(value_buf, value_size, "%s", pSS->ph.cs);
+    }
+    else if (!strcmp (key, "PageImageFormat"))
+    {
diff -r 48f19c720d05 -r 6a091fbcd804 print/hpijs/patches/patch-ldlencap.h
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/print/hpijs/patches/patch-ldlencap.h      Sun Apr 26 13:28:23 2015 +0000
@@ -0,0 +1,13 @@
+$NetBSD: patch-ldlencap.h,v 1.1 2015/04/26 13:28:23 joerg Exp $
+
+--- ldlencap.h.orig    2015-04-26 12:32:01.000000000 +0000
++++ ldlencap.h
+@@ -28,7 +28,7 @@
+   THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+   \***************************************************************************/
+ #ifndef _LDLENCAP
+-#define _LDLECNAP
++#define _LDLENCAP
+ 
+ // Wait times
+ #define PACKET_WAIT                 5
diff -r 48f19c720d05 -r 6a091fbcd804 print/hpijs/patches/patch-ljfastraster.cpp
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/print/hpijs/patches/patch-ljfastraster.cpp        Sun Apr 26 13:28:23 2015 +0000
@@ -0,0 +1,20 @@
+$NetBSD: patch-ljfastraster.cpp,v 1.1 2015/04/26 13:28:23 joerg Exp $
+
+--- ljfastraster.cpp.orig      2015-04-26 12:33:48.000000000 +0000
++++ ljfastraster.cpp
+@@ -412,13 +412,13 @@ DRIVER_ERROR HeaderLJFastRaster::SendCAP
+ //** this enumeration is for specifying the current color plane that is being
+ //** sent via fast raster path.  Toucan ordering is YMCK
+ 
+-typedef enum TonerColor
++typedef enum
+ {
+       eYellow = 0,
+       eMagenta,
+       eCyan,
+       eK
+-};
++} TonerColor;
+ 
+ typedef enum
+ {