Skip to content

AppVeyor - Fix freetds install errors by updating packages #483

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

Merged
merged 1 commit into from
Jan 11, 2021

Conversation

bvogelzang
Copy link
Contributor

AppVeyor builds have been failing during freetds installation due to outdated packages:

warning: no /var/cache/pacman/pkg/ cache exists, creating...
error: failed retrieving file 'mingw-w64-i686-freetds-1.00.100-1-any.pkg.tar.xz' from repo.msys2.org : The requested URL returned error: 404
error: failed retrieving file 'mingw-w64-i686-freetds-1.00.100-1-any.pkg.tar.xz' from sourceforge.net : The requested URL returned error: 404
error: failed retrieving file 'mingw-w64-i686-freetds-1.00.100-1-any.pkg.tar.xz' from www2.futureware.at : The requested URL returned error: 404
error: failed retrieving file 'mingw-w64-i686-freetds-1.00.100-1-any.pkg.tar.xz' from mirror.yandex.ru : The requested URL returned error: 404
warning: failed to retrieve some files
error: failed to commit transaction (unexpected error)

Adds install commands to the appveyor.yml so that freetds installs successfully.

I would expect running something similar to pacman -Syu would do the job, however, it appears there were recent changes to MSYS2 that require more setup. These commands were borrowed from https://github.com/oneclick/rubyinstaller2/blob/566169328a88c4960c75f0b25ef0c6fb5327fb46/appveyor.yml#L14-L31

@aharpervc aharpervc self-requested a review December 18, 2020 14:44
Copy link
Contributor

@aharpervc aharpervc left a comment

Choose a reason for hiding this comment

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

LGTM

@aharpervc
Copy link
Contributor

Can you rebase your other two PRs on this branch to confirm that with these changes, they pass CI?

@aharpervc aharpervc merged commit 32690ff into rails-sqlserver:master Jan 11, 2021
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.

None yet

2 participants