-
Notifications
You must be signed in to change notification settings - Fork 77
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
mozdownload should grab candidate builds from candidates/ and not nightly/ #218
Comments
I looked at the code and found that I can fix it in ReleaseCandidateScrapper::candidate_build_list_regex. Question:
Since it is application specific, I can either do these two:
Let me know and I can make changes in the fork I created for #22 (Add a bash-completion script). |
We do not support downloading of b2g builds yet. That is still an open issue, where we already have a PR which needs to be done. So for Firefox and Thunderbird it should be fine. @bhearsum, just to make sure... we can access the candidate builds in that folder at the time when they have been created? Means they will be available at the same time in candidates/ as in /nightly? |
I have updated the candidate_build_list_regex of the ReleaseCandidateScraper class to use candidates in place of nightly. The tests are also updated and all passed. I also tried manually downloading candidate build for v30.0 and it worked fine. |
code is available for review. Please see the pull request. |
The PR on issue #221 has been landed. Thanks @jayrajput for working on it! |
We got informed by @bhearsum that we should not grab candidate builds for Firefox from the firefox/nightly/ folder, but from firefox/candidates.
The text was updated successfully, but these errors were encountered: