Skip to content

Conversation

cvybhu
Copy link
Contributor

@cvybhu cvybhu commented Dec 17, 2022

The book workflow suddenly started failing at the mdbook test step.

After a brief investigation it looks like mdbook doesn't work properly when flags are passed before the main argument.

Because of this the old command:

mdbook test -L target/debug/deps docs

Started failing as if the docs argument wasn't passed at all. It used the default book directory and couldn't find a book there because the book is in the docs directory.
Error: Couldn't open SUMMARY.md in "/home/runner/work/scylla-rust-driver/scylla-rust-driver/src

On the other hand this works properly:

mdbook test docs -L target/debug/deps

So I changed the order of arguments to fix the problem.

The book workflow suddenly started failing
at the `mdbook test` step.

After a brief investigation it looks like
`mdbook` doesn't work properly when flags
are passed before the main argument.

Because of this the old command:
```bash
mdbook test -L target/debug/deps docs
```

Started failing as if the `docs` argument wasn't passed at all.
It used the default book directory and couldn't find a book there
because the book is in the `docs` directory.
Error: Couldn't open SUMMARY.md in "/home/runner/work/scylla-rust-driver/scylla-rust-driver/src

On the other hand this works properly:
```bash
mdbook test docs -L target/debug/deps
```

So I changed the order of arguments to fix the problem.

Signed-off-by: Jan Ciolek <[email protected]>
@cvybhu
Copy link
Contributor Author

cvybhu commented Dec 17, 2022

Opened rust-lang/mdBook#1958 because this looks like a mdbook bug

@cvybhu cvybhu requested a review from piodul December 17, 2022 12:15
@cvybhu
Copy link
Contributor Author

cvybhu commented Dec 19, 2022

The newest verison of the book has fixed the bug, so this PR is no longer needed. The CI passes on master without modifications.

@cvybhu cvybhu closed this Dec 19, 2022
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.

1 participant