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

Fixes xcode version finder #382

Merged
merged 1 commit into from
Apr 2, 2020

Conversation

robertwijas
Copy link
Contributor

It was finding a beta version despite the release one being already available (11.4 beta instead of 11.4).
Now it tries to match the name first and when nothing matches it falls back to matching version.

Resolves #381

Solution proposed here: #381 (comment).

It was finding a beta version despite the release one being already available (11.4 beta instead of 11.4).
Now it tries to match the name first and when nothing matches it falls back to matching version.

Resolves xcpretty#381

Solution proposed here: xcpretty#381 (comment).
@xavierLowmiller
Copy link

This probably fixes this (closed) issue as well: #374

Copy link
Collaborator

@jpsim jpsim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding tests!

@OdNairy
Copy link

OdNairy commented Mar 30, 2020

Any updates on this PR?

@hiragram
Copy link

hiragram commented Apr 1, 2020

Any update? 👀

Can't wait for release.

@robertwijas
Copy link
Contributor Author

You guys probably know that - but just in case - you can use the following in your Gemfile until it gets released:

gem 'xcode-install' , git: 'https://github.com/robertwijas/xcode-install', branch: 'fix/find-xcode-version'

@jpsim
Copy link
Collaborator

jpsim commented Apr 2, 2020

I'll be merging this because it looks like the e2e tests are failing for unrelated reasons.

RuntimeError: Missing username, and running in non-interactive shell

@Kesin11 @milch looks like both of you were involved in adding these failing e2e tests in #365. Can you look into how to fix them or otherwise disable them?

@jpsim jpsim merged commit 11822ca into xcpretty:master Apr 2, 2020
@Kesin11
Copy link
Contributor

Kesin11 commented Apr 2, 2020

@jpsim I created issue about e2e tests. #383
It is a limitation of Github Action secret. So I want to remove that test until the GitHub team resolves it.

@milch
Copy link
Collaborator

milch commented Apr 2, 2020

Hey everyone, I just released 2.6.4 which includes this fix: https://rubygems.org/gems/xcode-install/versions/2.6.4

Let me know if there are any issues 👍🏼

@Kesin11
Copy link
Contributor

Kesin11 commented Apr 4, 2020

@jpsim
I disabled E2E test at #385. Could you review it?

@nysander
Copy link

nysander commented Apr 7, 2020

when this change will be marked as release so gem update catches it?

@OdNairy
Copy link

OdNairy commented Apr 7, 2020

when this change will be marked as release so gem update catches it?

It's already available as the 2.6.4 version.

@nysander
Copy link

nysander commented Apr 7, 2020

am I missing something?

image

@OdNairy
Copy link

OdNairy commented Apr 7, 2020

Just try to update your gem. I have no idea why there is no linked Github Release.

@nysander
Copy link

nysander commented Apr 7, 2020

I have updated. It catches v2.6.4 but problem exists as in #381

$ xcversion --version
2.6.4
$ xcversion list
...
11.3.1 (installed)
11.4 beta 3 (installed)
11.4 (installed)
11.4 beta 2 (installed)
11.4 beta (installed)
$ ls /Applications/Xcode
Xcode-11.3.1.app/ Xcode-11.4.app/   Xcode.app/

where Xcode-11.4.app is in fact beta 3 and Xcode.app is version from App Store

@robertwijas
Copy link
Contributor Author

@nysander looks like this PR doesn't fix the update flow. I only tested it on a machine without 11.4 beta installed. Another PR #386 is ready though.

@nysander
Copy link

nysander commented Apr 7, 2020

I have removed all 11.4 versions and tried to install only GM but it installs 11.4 b3 every time

@robertwijas
Copy link
Contributor Author

@nysander :( try the branch from #386. Sorry, looks like my PR didn't fix it... but somehow it was enough for me to install proper version locally.

@robertwijas
Copy link
Contributor Author

@nysander xcversion cleanup might also be worth trying

@iosdevben
Copy link

iosdevben commented Jun 2, 2020

After upgrading to xcode-install 2.6.4 I was able to xcversion install 11.5 and not get 11.5 beta 2. Success.

The confusion for me was that I typed gem update xcversion, expecting to get the latest version of xcversion. Of course it did not update xcode-install.

@FeodorFitsner
Copy link

xcversion install 11.5 installed 11.5 only, but xcversion list still shows other 11.5 as installed:

$ xcversion list | grep '(installed)'
11.5 beta (installed)
11.5 GM Seed (installed)
11.5 (installed)
11.5 beta 2 (installed)

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

Successfully merging this pull request may close these issues.

Xcode 11.4 beta 3 installs instead of 11.4 GM
10 participants