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

Sandbox limits increase request for stm32ral #616

Closed
adamgreig opened this issue Feb 16, 2020 · 3 comments
Closed

Sandbox limits increase request for stm32ral #616

adamgreig opened this issue Feb 16, 2020 · 3 comments
Labels
C-limits-increase Category: A crate needs a limits increase to build successfully

Comments

@adamgreig
Copy link
Member

Crate name: stm32ral
Build failure link: https://docs.rs/crate/stm32ral/0.4.1/builds

Requested RAM limit: 6GB
Requested timeout: 15min is OK

Why your crate needs the resource increases:

Related to #608, stm32ral is an alternative register access crate for all STM32 microcontrollers in one package. As with those crates, it documents thousands of hardware registers and so is quite resource-intensive to build. I measured the doc build for the latest version as taking 1m39s and 5432MB RAM, so a 6GB RAM limit should work OK. 8GB might be more conservative if the crate continues to grow.

On balance the #608 crates are more important as they are much more widely used, but if it would be possible to get the RAM limit increased for stm32ral too, I'd really appreciate it.

Thanks!

@adamgreig adamgreig added the C-limits-increase Category: A crate needs a limits increase to build successfully label Feb 16, 2020
@adamgreig
Copy link
Member Author

Any chance of getting this limit raised alongside #608 now that #632 is in?

Thanks!

@jyn514
Copy link
Member

jyn514 commented Mar 15, 2020

Just updated the memory limit for stm32ral to 8 GB. Could you also set targets = [] for this crate to avoid unnecessary load on the server? Thanks again for your patience.

@adamgreig
Copy link
Member Author

Great, thank you! I have done so in adamgreig/stm32ral@cd0bfa3 so will be in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-limits-increase Category: A crate needs a limits increase to build successfully
Projects
None yet
Development

No branches or pull requests

2 participants