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

p5-dbd-mysql: Fix error where wrong database was installed #22702

Merged
merged 1 commit into from
Feb 18, 2024

Conversation

jhoyt4
Copy link
Contributor

@jhoyt4 jhoyt4 commented Feb 17, 2024

Fix all variants to properly parse dependencies.

Closes: https://trac.macports.org/ticket/69359

Description

This issue was caused by missing brackets after the description field in the variants.

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 14.3.1 23D60 x86_64
Xcode 15.2 15C500b

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL in commit message?
  • checked your Portfile with port lint --nitpick?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?
  • checked that the Portfile's most important variants haven't been broken?

  Fix all variants to properly parse dependencies. The main culprit
  was missing start brackes after the description field

Closes: https://trac.macports.org/ticket/69359
@jhoyt4
Copy link
Contributor Author

jhoyt4 commented Feb 17, 2024

% sudo port clean p5.34-dbd-mysql && sudo port install p5.34-dbd-mysql +mysql82
---> Cleaning p5.34-dbd-mysql
---> Computing dependencies for p5.34-dbd-mysql
The following dependencies will be installed: mysql82

% sudo port clean p5.34-dbd-mysql && sudo port install p5.34-dbd-mysql +mysql81
---> Cleaning p5.34-dbd-mysql
---> Computing dependencies for p5.34-dbd-mysql
The following dependencies will be installed: mysql81

% sudo port clean p5.34-dbd-mysql && sudo port install p5.34-dbd-mysql +mysql8
---> Cleaning p5.34-dbd-mysql
---> Computing dependencies for p5.34-dbd-mysql
The following dependencies will be installed: mysql8

% sudo port clean p5.34-dbd-mysql && sudo port install p5.34-dbd-mysql +mariadb10_11
---> Cleaning p5.34-dbd-mysql
---> Computing dependencies for p5.34-dbd-mysql
The following dependencies will be installed:
boost178
judy
mariadb-10.11

% sudo port clean p5.34-dbd-mysql && sudo port install p5.34-dbd-mysql +mariadb10_10
---> Cleaning p5.34-dbd-mysql
---> Computing dependencies for p5.34-dbd-mysql
The following dependencies will be installed:
boost178
judy
mariadb-10.10

% sudo port clean p5.34-dbd-mysql && sudo port install p5.34-dbd-mysql +mariadb10_6
---> Cleaning p5.34-dbd-mysql
---> Computing dependencies for p5.34-dbd-mysql
The following dependencies will be installed:
boost178
judy
mariadb-10.6

% sudo port clean p5.34-dbd-mysql && sudo port install p5.34-dbd-mysql +mariadb10_5
---> Cleaning p5.34-dbd-mysql
---> Computing dependencies for p5.34-dbd-mysql
The following dependencies will be installed:
boost178
judy
mariadb-10.5

% sudo port clean p5.34-dbd-mysql && sudo port install p5.34-dbd-mysql +mariadb10_4
---> Cleaning p5.34-dbd-mysql
---> Computing dependencies for p5.34-dbd-mysql
The following dependencies will be installed:
boost178
judy
mariadb-10.4

% sudo port clean p5.34-dbd-mysql && sudo port install p5.34-dbd-mysql +percona
---> Cleaning p5.34-dbd-mysql
---> Computing dependencies for p5.34-dbd-mysql
The following dependencies will be installed:
percona
protobuf3-cpp

@pmetzger pmetzger merged commit 7aa6191 into macports:master Feb 18, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants