Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change chp.regex to regex in all files #250

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jaxford
Copy link

@jaxford jaxford commented Oct 12, 2015

build_html.pl fails to create html files with this error:

Unknown link regex

Replacing chp.regex with regex in these files corrects the problem:

sections/autoload.pod
sections/chapter_06.pod
sections/implicit_ideas.pod
sections/values.pod

build_html.pl fails to create html files with this error:

Unknown link regex

Replacing chp.regex with regex in these files corrects the problem:

   sections/autoload.pod
   sections/chapter_06.pod
   sections/implicit_ideas.pod
   sections/values.pod
@davorg
Copy link

davorg commented Oct 22, 2015

I just came here to submit a very similar PR. I'm guessing that somewhere in the code, the link references are constructed using (\w+) in a regex. So chp.regex is created as regex and you get the error that @jaxford mentioned above (regex is the name of a missing link reference).

I fixed it by changing chp.regex to chp_regex - but this fix works just as well.

@sd43
Copy link
Contributor

sd43 commented Oct 24, 2015

I should have seen this earlier. The problem is with the regex not being able to extract the complete link name as @davorg mention. I made a fix by using the start_L and end_L callbacks to get the position of the text having the link name.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants