pkgsrc-Changes archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
CVS commit: pkgsrc/devel/jq
Module Name: pkgsrc
Committed By: leot
Date: Sun Aug 19 18:08:06 UTC 2018
Modified Files:
pkgsrc/devel/jq: Makefile distinfo
Added Files:
pkgsrc/devel/jq/patches: patch-builtin.c patch-compile.c
patch-configure.ac patch-inject__errors.c patch-util.c
Log Message:
jq: Avoid SIGSEGVs when using date builtins
Apply patch written by <soda> and shared with upstream as part of
<https://github.com/stedolan/jq/pull/1458> in order to fix crashes of jq when
using date builtin functions.
Also add a SUBST class intended to avoid invoking git as part of pre-configure
phase when invoking autoreconf.
While here take the MAINTAINERship and reformat a bit the Makefile
(NFCI).
Thanks a lot to <soda>, Takehiko NOZAKI and <nonaka> for investigating
the issue, writing the patches, sharing them with upstream and as
part of PR pkg/52460!
Bump PKGREVISION
To generate a diff of this commit:
cvs rdiff -u -r1.15 -r1.16 pkgsrc/devel/jq/Makefile
cvs rdiff -u -r1.9 -r1.10 pkgsrc/devel/jq/distinfo
cvs rdiff -u -r0 -r1.1 pkgsrc/devel/jq/patches/patch-builtin.c \
pkgsrc/devel/jq/patches/patch-compile.c \
pkgsrc/devel/jq/patches/patch-inject__errors.c \
pkgsrc/devel/jq/patches/patch-util.c
cvs rdiff -u -r0 -r1.3 pkgsrc/devel/jq/patches/patch-configure.ac
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: pkgsrc/devel/jq/Makefile
diff -u pkgsrc/devel/jq/Makefile:1.15 pkgsrc/devel/jq/Makefile:1.16
--- pkgsrc/devel/jq/Makefile:1.15 Wed May 30 16:03:48 2018
+++ pkgsrc/devel/jq/Makefile Sun Aug 19 18:08:05 2018
@@ -1,26 +1,32 @@
-# $NetBSD: Makefile,v 1.15 2018/05/30 16:03:48 ginsbach Exp $
+# $NetBSD: Makefile,v 1.16 2018/08/19 18:08:05 leot Exp $
DISTNAME= jq-1.5
-PKGREVISION= 4
+PKGREVISION= 5
CATEGORIES= devel
MASTER_SITES= ${MASTER_SITE_GITHUB:=stedolan/}
+GITHUB_TAG= ${DISTNAME}
-MAINTAINER= pkgsrc-users%NetBSD.org@localhost
+MAINTAINER= leot%NetBSD.org@localhost
HOMEPAGE= https://github.com/stedolan/jq/
COMMENT= Command-line JSON processor
LICENSE= mit
-GITHUB_TAG= ${DISTNAME}
-
USE_LANGUAGES= c99
USE_LIBTOOL= yes
-USE_TOOLS+= bison flex gmake
+USE_TOOLS+= autoreconf autoconf automake bison flex gmake
GNU_CONFIGURE= yes
-.include "options.mk"
-
-CFLAGS.SunOS+= -D_XPG6 -D__EXTENSIONS__
+SUBST_CLASSES+= tag
+SUBST_STAGE.tag= pre-configure
+SUBST_MESSAGE.tag= Avoid to run `git describe' to fetch the tag
+SUBST_FILES.tag= configure.ac scripts/version
+SUBST_SED.tag+= -e 's/git describe .*|/echo "jq-1.5-1-g940132e-dirty" |/'
TEST_TARGET= check
+pre-configure:
+ cd ${WRKSRC} && autoreconf -fi
+
+.include "options.mk"
+
.include "../../mk/bsd.pkg.mk"
Index: pkgsrc/devel/jq/distinfo
diff -u pkgsrc/devel/jq/distinfo:1.9 pkgsrc/devel/jq/distinfo:1.10
--- pkgsrc/devel/jq/distinfo:1.9 Wed May 30 16:03:48 2018
+++ pkgsrc/devel/jq/distinfo Sun Aug 19 18:08:05 2018
@@ -1,9 +1,14 @@
-$NetBSD: distinfo,v 1.9 2018/05/30 16:03:48 ginsbach Exp $
+$NetBSD: distinfo,v 1.10 2018/08/19 18:08:05 leot Exp $
SHA1 (jq-1.5.tar.gz) = 664638b560d9e734178e8cafb21d98817af5b5f3
RMD160 (jq-1.5.tar.gz) = 33ac77ac93e0539f6d66d29cd717013cdab8cf61
SHA512 (jq-1.5.tar.gz) = 3f9049321e8430b04dbccf81d7f2ab691b89383e91526eaf585bbeeb67698dea73a36b3aa179f6a95ef97cd73a8a9b6dc53cb1a5b0398b6422c11289b705de7b
Size (jq-1.5.tar.gz) = 1118086 bytes
SHA1 (patch-Makefile.in) = 446be0fa3517fb6fc1e2f5761d1f8fb28339c79c
+SHA1 (patch-builtin.c) = ab8572304d39680b19337aec525beaa88e6506da
+SHA1 (patch-compile.c) = a9d2f8eadaddcd6bd3e467740f6826073b40720e
+SHA1 (patch-configure.ac) = 5ac2b2c744f554a716189239446a5c5a13c58bea
+SHA1 (patch-inject__errors.c) = 863086a96be3b81011a141bdf293d3ef5693ef10
SHA1 (patch-src_jv__parse.c) = efca86e70daf27291a01bf538487b745b7bd600c
SHA1 (patch-src_jv__print.c) = 4d53dc1a1f0bb1cc827dd1adb62ecef2b7393970
+SHA1 (patch-util.c) = c6b9062312206267b0e83bc7284646f64e133f58
Added files:
Index: pkgsrc/devel/jq/patches/patch-builtin.c
diff -u /dev/null pkgsrc/devel/jq/patches/patch-builtin.c:1.1
--- /dev/null Sun Aug 19 18:08:06 2018
+++ pkgsrc/devel/jq/patches/patch-builtin.c Sun Aug 19 18:08:05 2018
@@ -0,0 +1,22 @@
+$NetBSD: patch-builtin.c,v 1.1 2018/08/19 18:08:05 leot Exp $
+
+Defining _GNU_SOURCE, _BSD_SOURCE etc. in C sources is problematic,
+because the result of the configure command may be inconsistent with it.
+to be consistent, such macros have to be defined in the early stage of
+the configure command, and the AC_USE_SYSTEM_EXTENSIONS macro does the job.
+
+Part of pull request 1458, commit id `df9a0963f8fa6fca773b059dce22c598152f3edb':
+
+ <https://github.com/stedolan/jq/pull/1458>
+
+Also shared via PR pkg/52460.
+
+--- builtin.c.orig 2015-08-18 04:25:04.000000000 +0000
++++ builtin.c
+@@ -1,6 +1,3 @@
+-#define _BSD_SOURCE
+-#define _GNU_SOURCE
+-#define _XOPEN_SOURCE
+ #include <sys/time.h>
+ #include <stdlib.h>
+ #include <stddef.h>
Index: pkgsrc/devel/jq/patches/patch-compile.c
diff -u /dev/null pkgsrc/devel/jq/patches/patch-compile.c:1.1
--- /dev/null Sun Aug 19 18:08:06 2018
+++ pkgsrc/devel/jq/patches/patch-compile.c Sun Aug 19 18:08:05 2018
@@ -0,0 +1,22 @@
+$NetBSD: patch-compile.c,v 1.1 2018/08/19 18:08:05 leot Exp $
+
+Defining _GNU_SOURCE, _BSD_SOURCE etc. in C sources is problematic,
+because the result of the configure command may be inconsistent with it.
+to be consistent, such macros have to be defined in the early stage of
+the configure command, and the AC_USE_SYSTEM_EXTENSIONS macro does the job.
+
+Part of pull request 1458, commit id `df9a0963f8fa6fca773b059dce22c598152f3edb':
+
+ <https://github.com/stedolan/jq/pull/1458>
+
+Also shared via PR pkg/52460.
+
+--- compile.c.orig 2015-08-18 04:25:04.000000000 +0000
++++ compile.c
+@@ -1,6 +1,3 @@
+-#ifndef _GNU_SOURCE
+-#define _GNU_SOURCE // for strdup
+-#endif
+ #include <assert.h>
+ #include <math.h>
+ #include <string.h>
Index: pkgsrc/devel/jq/patches/patch-inject__errors.c
diff -u /dev/null pkgsrc/devel/jq/patches/patch-inject__errors.c:1.1
--- /dev/null Sun Aug 19 18:08:06 2018
+++ pkgsrc/devel/jq/patches/patch-inject__errors.c Sun Aug 19 18:08:05 2018
@@ -0,0 +1,21 @@
+$NetBSD: patch-inject__errors.c,v 1.1 2018/08/19 18:08:05 leot Exp $
+
+Defining _GNU_SOURCE, _BSD_SOURCE etc. in C sources is problematic,
+because the result of the configure command may be inconsistent with it.
+to be consistent, such macros have to be defined in the early stage of
+the configure command, and the AC_USE_SYSTEM_EXTENSIONS macro does the job.
+
+Part of pull request 1458, commit id `df9a0963f8fa6fca773b059dce22c598152f3edb':
+
+ <https://github.com/stedolan/jq/pull/1458>
+
+Also shared via PR pkg/52460.
+
+--- inject_errors.c.orig 2015-08-18 04:25:04.000000000 +0000
++++ inject_errors.c
+@@ -1,5 +1,3 @@
+-
+-#define _GNU_SOURCE /* for RTLD_NEXT */
+ #include <assert.h>
+ #include <dlfcn.h>
+ #include <errno.h>
Index: pkgsrc/devel/jq/patches/patch-util.c
diff -u /dev/null pkgsrc/devel/jq/patches/patch-util.c:1.1
--- /dev/null Sun Aug 19 18:08:06 2018
+++ pkgsrc/devel/jq/patches/patch-util.c Sun Aug 19 18:08:05 2018
@@ -0,0 +1,24 @@
+$NetBSD: patch-util.c,v 1.1 2018/08/19 18:08:05 leot Exp $
+
+Defining _GNU_SOURCE, _BSD_SOURCE etc. in C sources is problematic,
+because the result of the configure command may be inconsistent with it.
+to be consistent, such macros have to be defined in the early stage of
+the configure command, and the AC_USE_SYSTEM_EXTENSIONS macro does the job.
+
+Part of pull request 1458, commit id `df9a0963f8fa6fca773b059dce22c598152f3edb':
+
+ <https://github.com/stedolan/jq/pull/1458>
+
+Also shared via PR pkg/52460.
+
+--- util.c.orig 2015-08-18 04:25:04.000000000 +0000
++++ util.c
+@@ -1,8 +1,3 @@
+-
+-#ifdef HAVE_MEMMEM
+-#define _GNU_SOURCE
+-#endif
+-
+ #include <sys/types.h>
+ #include <sys/stat.h>
+ #include <assert.h>
Index: pkgsrc/devel/jq/patches/patch-configure.ac
diff -u /dev/null pkgsrc/devel/jq/patches/patch-configure.ac:1.3
--- /dev/null Sun Aug 19 18:08:06 2018
+++ pkgsrc/devel/jq/patches/patch-configure.ac Sun Aug 19 18:08:05 2018
@@ -0,0 +1,23 @@
+$NetBSD: patch-configure.ac,v 1.3 2018/08/19 18:08:05 leot Exp $
+
+Defining _GNU_SOURCE, _BSD_SOURCE etc. in C sources is problematic,
+because the result of the configure command may be inconsistent with it.
+to be consistent, such macros have to be defined in the early stage of
+the configure command, and the AC_USE_SYSTEM_EXTENSIONS macro does the job.
+
+Part of pull request 1458, commit id `df9a0963f8fa6fca773b059dce22c598152f3edb':
+
+ <https://github.com/stedolan/jq/pull/1458>
+
+Also shared via PR pkg/52460.
+
+--- configure.ac.orig 2015-08-18 04:25:04.000000000 +0000
++++ configure.ac
+@@ -11,6 +11,7 @@ AC_PREREQ([2.64])
+ AC_CONFIG_AUX_DIR([config])
+ AM_INIT_AUTOMAKE([1.11.2 parallel-tests foreign -Wall])
+ AM_SILENT_RULES([yes])
++AC_USE_SYSTEM_EXTENSIONS
+ AM_PROG_AR
+ AM_MAINTAINER_MODE([enable])
+ AC_PROG_CC
Home |
Main Index |
Thread Index |
Old Index