On Sat, Jan 16, 2016 at 04:09:20PM -0500, Michael McConville wrote:
> This works for me. It unconditionally vforks at some point so, unlike
> grep, it needs proc. Also, the --pager flag lets the user specify a
> pager to display the output in, so it needs exec in that case.
>
>

Thanks a lot. Here is one with revision bump:

I will test this but so far it seems to work.

Regards,
Florian

Index: Makefile
===================================================================
RCS file: /cvs/ports/textproc/the_silver_searcher/Makefile,v
retrieving revision 1.19
diff -u -p -u -r1.19 Makefile
--- Makefile    9 Oct 2015 12:48:22 -0000       1.19
+++ Makefile    16 Jan 2016 21:29:15 -0000
@@ -3,6 +3,7 @@
 COMMENT =      code searching tool, with a focus on speed (ag)
 
 DISTNAME =     the_silver_searcher-0.31.0
+REVISION =     0
 CATEGORIES =   textproc
 
 HOMEPAGE =     https://github.com/ggreer/the_silver_searcher
Index: patches/patch-src_options_c
===================================================================
RCS file: patches/patch-src_options_c
diff -N patches/patch-src_options_c
--- /dev/null   1 Jan 1970 00:00:00 -0000
+++ patches/patch-src_options_c 16 Jan 2016 21:29:15 -0000
@@ -0,0 +1,19 @@
+$OpenBSD$
+--- src/options.c.orig Sun Sep  6 02:20:35 2015
++++ src/options.c      Sat Jan 16 15:52:17 2016
+@@ -543,6 +543,15 @@ void parse_options(int argc, char **argv, char **base_
+         }
+     }
+ 
++    if (opts.pager == NULL) {
++        if (pledge("stdio rpath proc", NULL) == -1) {
++            err(2, "pledge");
++        }
++    } else {
++        if (pledge("stdio rpath proc exec", NULL) == -1) {
++            err(2, "pledge");
++        }
++    }
+ 
+     if (ext_index[0]) {
+         num_exts = combine_file_extensions(ext_index, lang_num, &extensions);

Reply via email to