Opened 6 years ago

Closed 6 years ago

#725 closed defect (fixed)

Drop regex dependencies from chicken-core scripts

Reported by: mario Owned by: felix
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)

0001-Remove-unneeded-dependencies-on-regex-in-scripts-725.patch (1.3 KB) - added by zbigniew 6 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 6 years ago by felix

  • Owner set to felix
  • Status changed from new to assigned

comment:2 Changed 6 years ago by felix

  • Milestone 4.8.0 deleted

comment:3 Changed 6 years ago by zbigniew

This change appears to be unnecessary.

  • dpkg-eggs.scm -- Claims dependency on regex, but actually has none; (use regex) can be removed.
  • make-eggdoc.scm -- Claims dependency on regex, but actually has none; (use regex) can be removed.
  • tools.scm -- Was already converted to irregex in 8a516a0425.
  • wiki2html.scm -- Also depends on htmlprag and matchable, so converting to irregex gains you nothing except verbosity.

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.

comment:4 Changed 6 years ago by mario

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.

comment:5 Changed 6 years ago by zbigniew

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 Changed 6 years ago by felix

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.