Skip to content

Copy parse_rule since it was removed in werkzeug 2.2#61

Merged
ashb merged 2 commits intosphinx-contrib:mainfrom
Klikini:patch-1
Nov 14, 2022
Merged

Copy parse_rule since it was removed in werkzeug 2.2#61
ashb merged 2 commits intosphinx-contrib:mainfrom
Klikini:patch-1

Conversation

@Klikini
Copy link
Copy Markdown
Contributor

@Klikini Klikini commented Aug 10, 2022

@vLabayen
Copy link
Copy Markdown

Also needed to remove the import in translate_werkzeug_rule function.

def translate_werkzeug_rule(rule):
    from werkzeug.routing import parse_rule
    ...

With that, my build errors are fixed.
I'm using sphinx==v5.1.1 & sphinxcontrib.httpdomain==v.1.8.0

@Klikini
Copy link
Copy Markdown
Contributor Author

Klikini commented Nov 1, 2022

@vLabayen Good catch! I have applied that change to this PR

@kasium
Copy link
Copy Markdown

kasium commented Nov 9, 2022

Any update on this issue? It would be great to use the latest version of flask/werkzeug

@Klikini
Copy link
Copy Markdown
Contributor Author

Klikini commented Nov 9, 2022

@kasium we're just waiting for a maintainer to merge this PR. I'm using the code from my fork and it works for me, but I don't know if anyone else has tested it.

@kasium
Copy link
Copy Markdown

kasium commented Nov 10, 2022

@ashb would somebody be so kind to check this PR and do a release?

@ashb
Copy link
Copy Markdown
Collaborator

ashb commented Nov 11, 2022

@kasium Yes of course, sorry for not seeing it!

@ashb ashb merged commit 07302c8 into sphinx-contrib:main Nov 14, 2022
@ashb
Copy link
Copy Markdown
Collaborator

ashb commented Nov 14, 2022

1.8.1 released! Better late then never right?

MonkeyDo added a commit to metabrainz/listenbrainz-server that referenced this pull request Mar 21, 2023
Since werkzeug 2.2 a missing method is causing autohttp to break the build.
New version fixes the issue: sphinx-contrib/httpdomain#61 (comment)
amCap1712 pushed a commit to metabrainz/listenbrainz-server that referenced this pull request Mar 21, 2023
Since werkzeug 2.2 a missing method is causing autohttp to break the build.
New version fixes the issue: sphinx-contrib/httpdomain#61 (comment)
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.

4 participants