Skip to content
New issue

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

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

Already on GitHub? # to your account

do not replace old docs if rebuild failed #2675

Closed
dhardy opened this issue Nov 18, 2024 · 11 comments · Fixed by #2718
Closed

do not replace old docs if rebuild failed #2675

dhardy opened this issue Nov 18, 2024 · 11 comments · Fixed by #2718

Comments

@dhardy
Copy link

dhardy commented Nov 18, 2024

Crate name

rand

Build failure link

https://docs.rs/crate/rand/latest/builds

Additional details

It seems that rand 0.8.5 docs got rebuilt yesterday? Strange; I haven't seen docs.rs just rebuild old docs before. And unfortunately, the rebuild fails due to a nightly feature which got changed since then.

Can you reinstate the old docs build please?

CC rust-random/rand#1528

@GuillaumeGomez
Copy link
Member

Changing the issue to "do not replace old docs if rebuild failed".

@GuillaumeGomez GuillaumeGomez changed the title rand 0.8.5 docs failed rebuild do not replace old docs if rebuild failed Nov 18, 2024
@syphar
Copy link
Member

syphar commented Nov 18, 2024

this is super odd, I actually tested this case and saw that old docs were kept before I started rebuilding...

@syphar
Copy link
Member

syphar commented Nov 18, 2024

( reinstanting the old docs is a little tricky, a rebuild would be easier if rand would stop using the fature)

@dhardy
Copy link
Author

dhardy commented Nov 18, 2024

I can do. I dread to think how many person-hours will be consumed just checking the changelog...

@syphar
Copy link
Member

syphar commented Nov 18, 2024

( there might be a quicker way on our side, I probably can look into it tomorrow)

@dhardy
Copy link
Author

dhardy commented Nov 18, 2024

Would be good. I opened a PR, but the CI is not happy. I guess I could just ignore that. rust-random/rand#1529

@syphar
Copy link
Member

syphar commented Nov 19, 2024

@dhardy I was able to restore the docs with some direct SQL hackery in the database.

@dhardy
Copy link
Author

dhardy commented Nov 19, 2024

Thanks. I'll let you decide whether to close this issue or not (title?).

@syphar
Copy link
Member

syphar commented Nov 19, 2024

the issue is still an issue, the title is correct.

@syphar
Copy link
Member

syphar commented Nov 20, 2024

@dhardy just a note, we want to do rebuilds so all crates can benefit from the many new rustdoc features,
so there is a value of making rand compile with the current nightly version :)

@dhardy
Copy link
Author

dhardy commented Nov 20, 2024

I wondered, but didn't see that anywhere.

Rand v0.9 should be out soon.

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

Successfully merging a pull request may close this issue.

3 participants