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

Download 5.22.0 when .travis.yml asks for '5.22' and similar #25

Open
autarch opened this issue Nov 13, 2015 · 2 comments
Open

Download 5.22.0 when .travis.yml asks for '5.22' and similar #25

autarch opened this issue Nov 13, 2015 · 2 comments

Comments

@autarch
Copy link
Contributor

autarch commented Nov 13, 2015

It'd be nice if the helpers were smart enough to just download the latest "X.Y.Z" when the config asks for "X.Y".

I'll try to get to this if I find some round tuits, but feel free to beat me to it ;)

@haarg
Copy link
Member

haarg commented Nov 14, 2015

The reason I don't do this currently is that I didn't want to change the behavior of the existing version targets travis provides. But if for example, you request 5.12-thr, it will give you a 5.12.5 build with threads enabled.

Do you think it would be better to always try to select the most recent version, even if travis provides an existing version under the same name?

@autarch
Copy link
Contributor Author

autarch commented Nov 15, 2015

Good question. Here's how I think this should all work ...

If someone asks for a specific patchlevel, like 5.22.0, they should get exactly that. If possible, it should come from the Travis environment (since that's faster).

If someone asks for just a major/minor version, like 5.22, they should get the first of ...

  • Perl from Travis that is 5.22.X. If there are multiple patchlevels available, use the most recent.
  • Pre-built Perl from travis-perl. If there are multiple patchlevels available, use the most recent.

Basically, I'm saying it should optimize for speed, and if someone always wants the latest version of a given release, they should ask for it explicitly.

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

2 participants