View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0000658 | LDMud 3.5 | Compilation, Installation | public | 2009-06-12 14:51 | 2009-06-16 15:43 |
| Reporter | zesstra | Assigned To | zesstra | ||
| Priority | normal | Severity | minor | Reproducibility | N/A |
| Status | resolved | Resolution | fixed | ||
| Target Version | 3.5.0 | Fixed in Version | 3.5.0 | ||
| Summary | 0000658: RfC: Remove/Update supplied version of pcre | ||||
| Description | The pcre version we supply is dreadfully out-dated. We should either update or remove it. I wanted to update it for a long time, but the longer I think about it, the more I would like to remove it and just link against the system libpcre. That should save us time to concentrate on our core development. I think, nearly nobody currently uses the supplied pcre package and it seems to be ubiquitary available on POSIX systems. Please comment. ;-) | ||||
| Tags | No tags attached. | ||||
|
|
I agree with removing the bundled libpcre. Even cygwin has a pcre-devel package. |
|
|
The driver currently relies on libpcre's availability (because it is bundled). Removing it leads to a non-optional requirement for libpcre on the system. It seems OK for me, but I could alternatively insert some #ifdef HAS_PCRE into mregex.c, so that without a system libpcre only the traditional RE implementation is available. Gnomi and me don't like #ifdefs too much, but that way we would have libpcre still optional. |
|
|
I agree, too, that the libpcre sources should be removed from the driver sources, and I think that pcre support should be made optional. |
|
|
I am just preparing a patch which removes our builtin pcre package and encapsulates the references to it in mregex.c by #ifdef HAS_PCRE. I plan to commit the patch directly to 3.5. because that is anyway expected to be unstable. ;-) |
|
|
Ok, I removed the built-in PCRE package in r2646. The existence of libpcre on the host system should be completely optional. Of course, without it, no PCREs in the driver. I hope, I did not miss some manpage... ;-) |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2009-06-12 14:51 | zesstra | New Issue | |
| 2009-06-14 17:33 | fufu | Note Added: 0001203 | |
| 2009-06-14 23:55 | zesstra | Note Added: 0001204 | |
| 2009-06-15 04:05 | Gnomi | Note Added: 0001205 | |
| 2009-06-16 13:15 | zesstra | Status | new => assigned |
| 2009-06-16 13:15 | zesstra | Assigned To | => zesstra |
| 2009-06-16 13:18 | zesstra | Note Added: 0001210 | |
| 2009-06-16 15:43 | zesstra | Note Added: 0001211 | |
| 2009-06-16 15:43 | zesstra | Status | assigned => resolved |
| 2009-06-16 15:43 | zesstra | Fixed in Version | => 3.5.0 |
| 2009-06-16 15:43 | zesstra | Resolution | open => fixed |
