Stuart Henderson <[email protected]> wrote: > The easy/safe way is to bump libjpeg's shared library major version. > > -- > Sent from a phone, apologies for poor formatting.
The major version? Going from 69.0 to 69.1 seemed to work. Do I need to go to 70.0? Haven't worked with shared libs in ports before. > > On 3 November 2018 14:47:00 trondd <[email protected]> wrote: > > > I noticed that at some point www/links+ wasn't able to display jpegs. Turns > > out not to be able to detect jpeglib.h on the system using configure. > > > > The root cause is a bug introduced in the upgrade of graphics/jpeg which > > incorrectly set #defines in a header file, causing compiler warnings and > > resulting in configure failing to correctly include jpeglib.h > > > > They've fixed it [0] and are targeting to reease it with 2.0.1 but I don't > > know > > when that'll happen. I'm backporting the fix to 2.0.0 so I can have a > > functional links+. > > > > I am not sure if I have to do something to force links+ to update since it's > > version didn't change. Is there a global way to do that or does each > > impacted > > downstream program need to be REVISON bumped? > > > > Tim. > > > > [0] > > https://github.com/libjpeg-turbo/libjpeg-turbo/commit/2260b66e1671038a362dca4a204c28d72b8f7e4e > > > > Index: Makefile =================================================================== RCS file: /cvs/ports/graphics/jpeg/Makefile,v retrieving revision 1.60 diff -u -p -r1.60 Makefile --- Makefile 1 Oct 2018 14:16:55 -0000 1.60 +++ Makefile 3 Nov 2018 15:54:51 -0000 @@ -5,9 +5,10 @@ COMMENT= SIMD-accelerated JPEG codec rep V= 2.0.0 DISTNAME= jpeg-turbo-${V} PKGNAME= jpeg-${V} +REVISION= 0 EPOCH= 0 -SHARED_LIBS+= jpeg 69.0 # 64.0 +SHARED_LIBS+= jpeg 69.1 # 64.0 CATEGORIES= graphics DPB_PROPERTIES= parallel Index: patches/patch-jconfig_h_in =================================================================== RCS file: patches/patch-jconfig_h_in diff -N patches/patch-jconfig_h_in --- /dev/null 1 Jan 1970 00:00:00 -0000 +++ patches/patch-jconfig_h_in 3 Nov 2018 15:54:51 -0000 @@ -0,0 +1,74 @@ +$OpenBSD$ + +Index: jconfig.h.in +--- jconfig.h.in.orig ++++ jconfig.h.in +@@ -10,16 +10,16 @@ + #define LIBJPEG_TURBO_VERSION_NUMBER @LIBJPEG_TURBO_VERSION_NUMBER@ + + /* Support arithmetic encoding */ +-#cmakedefine C_ARITH_CODING_SUPPORTED ++#cmakedefine C_ARITH_CODING_SUPPORTED 1 + + /* Support arithmetic decoding */ +-#cmakedefine D_ARITH_CODING_SUPPORTED ++#cmakedefine D_ARITH_CODING_SUPPORTED 1 + + /* Support in-memory source/destination managers */ +-#cmakedefine MEM_SRCDST_SUPPORTED ++#cmakedefine MEM_SRCDST_SUPPORTED 1 + + /* Use accelerated SIMD routines. */ +-#cmakedefine WITH_SIMD ++#cmakedefine WITH_SIMD 1 + + /* + * Define BITS_IN_JSAMPLE as either +@@ -33,37 +33,37 @@ + #define BITS_IN_JSAMPLE @BITS_IN_JSAMPLE@ /* use 8 or 12 */ + + /* Define to 1 if you have the <locale.h> header file. */ +-#cmakedefine HAVE_LOCALE_H ++#cmakedefine HAVE_LOCALE_H 1 + + /* Define to 1 if you have the <stddef.h> header file. */ +-#cmakedefine HAVE_STDDEF_H ++#cmakedefine HAVE_STDDEF_H 1 + + /* Define to 1 if you have the <stdlib.h> header file. */ +-#cmakedefine HAVE_STDLIB_H ++#cmakedefine HAVE_STDLIB_H 1 + + /* Define if you need to include <sys/types.h> to get size_t. */ +-#cmakedefine NEED_SYS_TYPES_H ++#cmakedefine NEED_SYS_TYPES_H 1 + + /* Define if you have BSD-like bzero and bcopy in <strings.h> rather than + memset/memcpy in <string.h>. */ +-#cmakedefine NEED_BSD_STRINGS ++#cmakedefine NEED_BSD_STRINGS 1 + + /* Define to 1 if the system has the type `unsigned char'. */ +-#cmakedefine HAVE_UNSIGNED_CHAR ++#cmakedefine HAVE_UNSIGNED_CHAR 1 + + /* Define to 1 if the system has the type `unsigned short'. */ +-#cmakedefine HAVE_UNSIGNED_SHORT ++#cmakedefine HAVE_UNSIGNED_SHORT 1 + + /* Compiler does not support pointers to undefined structures. */ +-#cmakedefine INCOMPLETE_TYPES_BROKEN ++#cmakedefine INCOMPLETE_TYPES_BROKEN 1 + + /* Define if your (broken) compiler shifts signed values as if they were + unsigned. */ +-#cmakedefine RIGHT_SHIFT_IS_UNSIGNED ++#cmakedefine RIGHT_SHIFT_IS_UNSIGNED 1 + + /* Define to 1 if type `char' is unsigned and you are not using gcc. */ + #ifndef __CHAR_UNSIGNED__ +- #cmakedefine __CHAR_UNSIGNED__ ++ #cmakedefine __CHAR_UNSIGNED__ 1 + #endif + + /* Define to empty if `const' does not conform to ANSI C. */
