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

do not replace old docs if rebuild failed #2675

Open
dhardy opened this issue Nov 18, 2024 · 11 comments
Open

do not replace old docs if rebuild failed #2675

dhardy opened this issue Nov 18, 2024 · 11 comments

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.

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

No branches or pull requests

3 participants