View Issue Details

IDProjectCategoryView StatusLast Update
0000814LDMud 3.5Compilation, Installationpublic2013-06-21 19:33
ReporterLeonidas Assigned Tozesstra  
PrioritynormalSeveritytrivialReproducibilityalways
Status closedResolutionno change required 
Summary0000814: Morgengrauen-Settings miss PCRE
DescriptionIn settings/morgengrauen enable_use_pcre is no. MGs mudlib depends on PCRE.
TagsNo tags attached.
Attached Files
fix_morgengrauen_settings.patch (707 bytes)   
From eca0d3095d2622e68fc4f8014b88a216a6f3dd92 Mon Sep 17 00:00:00 2001
From: Christian Georg Becker <krischan@flunzel.de>
Date: Thu, 10 Jan 2013 20:55:45 +0100
Subject: [PATCH] enabled pcre for morgengrauen

---
 src/settings/morgengrauen |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/settings/morgengrauen b/src/settings/morgengrauen
index 830444d..4a9a98d 100755
--- a/src/settings/morgengrauen
+++ b/src/settings/morgengrauen
@@ -25,7 +25,7 @@ enable_use_deprecated=no
 enable_use_set_light=no
 enable_use_parse_command=no
 enable_erq=no
-enable_use_pcre=no
+enable_use_pcre=yes
 enable_synchronous_heart_beat=yes
 with_alarm_time=1
 with_heart_beat_interval=2
-- 
1.7.9

Activities

zesstra

2013-02-23 19:45

administrator   ~0002184

Now, that I thought about it: I think, the settings for MG are correct.

--enable-use-pcre changes the default PCRE engine to PCRE instead of the traditional one. So that what is used if nothing is selected by the programmer. However many of our old REs are not PCRE and will fail if interpreted as PCRE.

However, even without that flag, configure searches for libpcre and we try to link against it. Then you select for each call to regexp efuns what RE type should be used.

At some distant point in the future, maybe all of our RE are PCRE...

Issue History

Date Modified Username Field Change
2013-01-10 20:09 Leonidas New Issue
2013-01-10 20:09 Leonidas File Added: fix_morgengrauen_settings.patch
2013-02-21 19:41 zesstra Assigned To => zesstra
2013-02-21 19:41 zesstra Status new => assigned
2013-02-23 19:45 zesstra Note Added: 0002184
2013-06-21 19:33 zesstra Status assigned => closed
2013-06-21 19:33 zesstra Resolution open => no change required