[Pcre-svn] [1449] code/trunk: Fix error in README; give an …

Top Page
Delete this message
Author: Subversion repository
Date:  
To: pcre-svn
Subject: [Pcre-svn] [1449] code/trunk: Fix error in README; give an error for invalid --with-pcregrep-bufsize values
Revision: 1449
          http://vcs.pcre.org/viewvc?view=rev&revision=1449
Author:   ph10
Date:     2014-01-17 16:59:52 +0000 (Fri, 17 Jan 2014)


Log Message:
-----------
Fix error in README; give an error for invalid --with-pcregrep-bufsize values
in ./configure.

Modified Paths:
--------------
    code/trunk/ChangeLog
    code/trunk/README
    code/trunk/configure.ac


Modified: code/trunk/ChangeLog
===================================================================
--- code/trunk/ChangeLog    2014-01-13 21:25:53 UTC (rev 1448)
+++ code/trunk/ChangeLog    2014-01-17 16:59:52 UTC (rev 1449)
@@ -80,6 +80,12 @@
     failing. Because of the uncertainty, the script did not used to stop if 
     test 3 failed; it now does. If further versions of a French locale ever 
     come to light, they can now easily be added.
+    
+16. If --with-pcregrep-bufsize was given a non-integer value such as "50K",
+    there was a message during ./configure, but it did not stop. This now 
+    provokes an error. The invalid example in README has been corrected. 
+    If a value less than the minimum is given, the minimum value has always 
+    been used, but now a warning is given.



Version 8.34 15-December-2013

Modified: code/trunk/README
===================================================================
--- code/trunk/README    2014-01-13 21:25:53 UTC (rev 1448)
+++ code/trunk/README    2014-01-17 16:59:52 UTC (rev 1449)
@@ -373,12 +373,12 @@


Of course, the relevant libraries must be installed on your system.

-. The default size of internal buffer used by pcregrep can be set by, for
- example:
+. The default size (in bytes) of the internal buffer used by pcregrep can be
+ set by, for example:

- --with-pcregrep-bufsize=50K
+ --with-pcregrep-bufsize=51200

- The default value is 20K.
+ The value must be a plain integer. The default is 20480.

. It is possible to compile pcretest so that it links with the libreadline
or libedit libraries, by specifying, respectively,
@@ -988,4 +988,4 @@
Philip Hazel
Email local part: ph10
Email domain: cam.ac.uk
-Last updated: 08 January 2014
+Last updated: 17 January 2014

Modified: code/trunk/configure.ac
===================================================================
--- code/trunk/configure.ac    2014-01-13 21:25:53 UTC (rev 1448)
+++ code/trunk/configure.ac    2014-01-17 16:59:52 UTC (rev 1449)
@@ -248,7 +248,7 @@
 # Handle --with-pcregrep-bufsize=N
 AC_ARG_WITH(pcregrep-bufsize,
               AS_HELP_STRING([--with-pcregrep-bufsize=N],
-                             [pcregrep buffer size (default=20480)]),
+                             [pcregrep buffer size (default=20480, minimum=8192)]),
               , with_pcregrep_bufsize=20480)


# Handle --enable-pcretest-libedit
@@ -744,7 +744,12 @@
fi

 if test $with_pcregrep_bufsize -lt 8192 ; then
+  AC_MSG_WARN([$with_pcregrep_bufsize is too small for --with-pcregrep-bufsize; using 8192])
   with_pcregrep_bufsize="8192"
+else
+  if test $? -gt 1 ; then
+  AC_MSG_ERROR([Bad value for  --with-pcregrep-bufsize]) 
+  fi    
 fi


AC_DEFINE_UNQUOTED([PCREGREP_BUFSIZE], [$with_pcregrep_bufsize], [