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

filter xcversion list by range of versions #426

Open
Pe-te opened this issue Apr 20, 2021 · 5 comments
Open

filter xcversion list by range of versions #426

Pe-te opened this issue Apr 20, 2021 · 5 comments

Comments

@Pe-te
Copy link

Pe-te commented Apr 20, 2021

Hi, I'd like to always have all versions from Xcode 10 upwards installed, so I use this regex to find the missing ones:

xcversion list | grep -E '^[0-9]{2,}(.[0-9]+)*$'

But it would be really great, if we could have those as parameters for the list command instead:

  • Minimum version, usually we don't care about very old versions (I use two digits at the beginning for 10 and higher, but one day I might want to install something like 14 and higher)
  • Maybe even maximum version, if you have a build server that only handles old apps
  • Skipping all the "beta|seed|candidate|installed" versions (I do this by parsing for digits and dots only)
  • Skipping all already installed versions, we only want to know what is missing so we can install it
@rogerluan
Copy link
Collaborator

I like this a lot! Perhaps using semantic versioning to compare versions 👌

Feel free to submit a pull request to implement this improvement 👍

@rogerluan rogerluan changed the title Improvements for xcversion list filter xcversion list by range of versions Apr 21, 2021
@Pe-te
Copy link
Author

Pe-te commented Apr 22, 2021

Feel free to submit a pull request to implement this improvement 👍

Awesome, my ruby-fu is just not strong enough. 😢 Hopefully someone can make this happen.

crazytonyli added a commit to crazytonyli/xcode-install that referenced this issue Jun 22, 2021
crazytonyli added a commit to crazytonyli/xcode-install that referenced this issue Jun 22, 2021
@Pe-te
Copy link
Author

Pe-te commented Jun 13, 2022

As far as I understand the code, Tony Li added the missing option? How do we get it into master?

@rogerluan
Copy link
Collaborator

@Pe-te , we'd need a PR to be open containing @crazytonyli's changes so we can review, test and merge them 😊

@crazytonyli
Copy link

FYI, I've opened a PR here: #438

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

3 participants