Opened 14 years ago
Closed 14 years ago
#725 closed defect (fixed)
Drop regex dependencies from chicken-core scripts
| Reported by: | Mario Domenech Goulart | Owned by: | felix winkelmann |
|---|---|---|---|
| Priority: | not urgent at all | Milestone: | |
| Component: | core tools | Version: | 4.7.x |
| Keywords: | regex, core scripts | Cc: | |
| Estimated difficulty: |
Description
Make the following scripts use irregex instead of regex:
- dpkg-eggs.scm
- make-eggdoc.scm
- wiki2html.scm
- tools.scm
(are they even used?)
Attachments (1)
Change History (7)
comment:1 by , 14 years ago
| Owner: | set to |
|---|---|
| Status: | new → assigned |
comment:2 by , 14 years ago
| Milestone: | 4.8.0 |
|---|
comment:3 by , 14 years ago
comment:4 by , 14 years ago
You are right, Jim. Boy, what a mess I've made!
I have no idea why I thought/saw string-match in tools.scm. I'm really sorry for all the confusion.
by , 14 years ago
| Attachment: | 0001-Remove-unneeded-dependencies-on-regex-in-scripts-725.patch added |
|---|
comment:5 by , 14 years ago
Attached will fix by removing unused regex deps. Should I push it myself?
Note that dpkg-eggs.scm is still broken afterward because it uses ##sys#pathname-directory-separator, which no longer exists. It could be changed to make-pathname but it's probably safer for Ivan to fix that (if it is still in use).
comment:6 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Note:
See TracTickets
for help on using tickets.

This change appears to be unnecessary.
Overall, it seems that (use regex) was added to tools.scm just to provide string-match, but string-match is only used in wiki2html.scm, which already depends on regex.