search

ietf-wg-bpf / ebpf-docs

eBPF Standard Documentation
42 stars 5 forks source link

Github: installing xml2rfc fails #23

Closed dthaler closed 1 year ago

dthaler commented 1 year ago

https://github.com/ietf-wg-bpf/ebpf-docs/actions/runs/5471462362/jobs/9962624761

hawkinsw commented 1 year ago

Not sure if it would help or not, but I have made/published a GitHub action that can do some/all of the post-processing: https://github.com/marketplace/actions/ietf-publisher

Otherwise, let me know if/how I can help!

dthaler commented 1 year ago

Not sure if it would help or not, but I have made/published a GitHub action that can do some/all of the post-processing: https://github.com/marketplace/actions/ietf-publisher

Otherwise, let me know if/how I can help!

That repository says it works with markdown. BPF docs are converted from RST not markdown, so I don't think it will help. Something recently broke, this was working fine earlier.

hawkinsw commented 1 year ago

Okay! I am happy to add an RST mode to that action if it would help!

On Thu, Jul 6, 2023 at 1:35 PM Dave Thaler @.***> wrote:

Not sure if it would help or not, but I have made/published a GitHub action that can do some/all of the post-processing: https://github.com/marketplace/actions/ietf-publisher

Otherwise, let me know if/how I can help!

That repository says it works with markdown. BPF docs are converted from RST not markdown, so I don't think it will help. Something recently broke, this was working fine earlier.

— Reply to this email directly, view it on GitHub https://github.com/ietf-wg-bpf/ebpf-docs/issues/23#issuecomment-1624067973, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCP2CQJXB2AHG6CAJW6ME3XO3ZOJANCNFSM6AAAAAAZ7Z7HPM . You are receiving this because you commented.Message ID: @.***>

hawkinsw commented 1 year ago

Now that I am looking at the error, I recall dealing with a very similar issue when building my action. It had to do with the fact that the runner had an old version of pip, by default. Adding

python3 -m pip install --upgrade pip

fixed the issue for me. I will fork and attempt to submit a PR.

Will

dthaler commented 1 year ago

Fixed by PR #25