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

V2.7.2 - MariaDB failing to install #328

Merged
merged 2 commits into from
Nov 15, 2023
Merged

Conversation

mayushko26
Copy link
Contributor

@mayushko26 mayushko26 commented Nov 15, 2023

Description of changes:

MariaDB fails to be installed in v2.7.2 due to a bad path being provided. Additionally, there is a hardcoded instance of amd64 in the path which should be parametrized instead by the user's current architecture.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

image (5)

Description of testing:

  • Tested that it builds locally (macOS)
  • Automated tests show it builds for x86_64

@mayushko26 mayushko26 changed the title V2.7.2 V2.7.2 - MariaDB failing to install Nov 15, 2023
@mayushko26 mayushko26 merged commit 6bbbc46 into aws:v2.7.2 Nov 15, 2023
1 check passed
wget https://mirror.mariadb.org/yum/11.1.2/fedora38-amd64/rpms/MariaDB-shared-11.1.2-1.fc38.$(uname -p).rpm -P /mariadb_rpm
wget https://mirror.mariadb.org/yum/11.1.2/fedora38-amd64/rpms/MariaDB-devel-11.1.2-1.fc38.$(uname -p).rpm -P /mariadb_rpm
if [[ $(uname -p) == "aarch64" ]]; then
wget https://mirror.mariadb.org/yum/11.1/fedora38-aarch64/rpms/MariaDB-common-11.1.2-1.fc38.$(uname -p).rpm -P /mariadb_rpm

Choose a reason for hiding this comment

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

These links don't work neither. I've found a workaround using fedora/38/$(uname -p) instead of fedora38-aarch64 in all the lines.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Which OS and architecture are you using?

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.

4 participants