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

mozdownload should grab candidate builds from candidates/ and not nightly/ #218

Closed
whimboo opened this issue Jun 4, 2014 · 5 comments
Closed
Assignees
Milestone

Comments

@whimboo
Copy link
Contributor

whimboo commented Jun 4, 2014

We got informed by @bhearsum that we should not grab candidate builds for Firefox from the firefox/nightly/ folder, but from firefox/candidates.

@jayrajput
Copy link
Contributor

I looked at the code and found that I can fix it in ReleaseCandidateScrapper::candidate_build_list_regex.

Question:

  • What about other applications (b2g and thunderbird)? Application b2g does not have any candidate folder, thunderbird has one.

Since it is application specific, I can either do these two:

  • Add a mapping b/w application (b2g, firefox, thunderbird) and candidate build location folder in the ReleaseCandidateScrapper.
  • Check if candidate folder exists, if yes use it else fallback on the nightly folder. I will need to figure out someway to check if the directory exists

Let me know and I can make changes in the fork I created for #22 (Add a bash-completion script).

@whimboo
Copy link
Contributor Author

whimboo commented Jun 10, 2014

What about other applications (b2g and thunderbird)? Application b2g does not have any candidate folder, thunderbird has one.

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?

@jayrajput
Copy link
Contributor

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.

@jayrajput
Copy link
Contributor

code is available for review. Please see the pull request.

@whimboo
Copy link
Contributor Author

whimboo commented Jun 25, 2014

The PR on issue #221 has been landed. Thanks @jayrajput for working on it!

@whimboo whimboo closed this as completed Jun 25, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants