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 variant conflicts issues #26725

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jhoyt4
Copy link
Contributor

@jhoyt4 jhoyt4 commented Nov 24, 2024

Fix issue where multiple variant conflicts were being set as a single
conflict. Fixing this also allowed for the removal of the
active_vartiants workaround.

  • fix variant conflicts
  • remove active variants workaround\
  • fix typos

Closes: https://trac.macports.org/ticket/60517
Closes: https://trac.macports.org/ticket/71360
Closes: https://trac.macports.org/ticket/71381

Note - I have these broken into 2 commits to allow them to be tracked easier.

Description

Correct improper variant conflicts issue where all conflicts were handled as a single port. Additionally, attempt to address typos (at least the ones I can find) and add back the legacy mariadb10_2 variant.

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

macOS 15.1.1 24B91 arm64
Xcode 16.1 16B40

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?

@jhoyt4 jhoyt4 marked this pull request as draft November 24, 2024 14:57
@jhoyt4 jhoyt4 force-pushed the p5-dbd-mysql branch 3 times, most recently from 9b08de3 to a873787 Compare November 24, 2024 18:19
@jhoyt4 jhoyt4 marked this pull request as ready for review November 24, 2024 19:28
@jhoyt4 jhoyt4 force-pushed the p5-dbd-mysql branch 2 times, most recently from 1f79fa3 to 653f5a7 Compare December 1, 2024 15:15
@jhoyt4 jhoyt4 requested a review from ryandesign December 1, 2024 17:25
@jhoyt4 jhoyt4 force-pushed the p5-dbd-mysql branch 2 times, most recently from 4c7d4ac to 487dc63 Compare January 12, 2025 13:22
@jhoyt4 jhoyt4 force-pushed the p5-dbd-mysql branch 2 times, most recently from 17cb02f to 7851ccf Compare January 26, 2025 11:42
@jhoyt4 jhoyt4 force-pushed the p5-dbd-mysql branch 3 times, most recently from f5a82ea to b344b26 Compare February 22, 2025 04:42
@jhoyt4
Copy link
Contributor Author

jhoyt4 commented Feb 22, 2025

WTF does "Merging is blocked" mean?

@jhoyt4
Copy link
Contributor Author

jhoyt4 commented Feb 22, 2025

I've dropped the perl 3.36 / 3.38 commit in hopes that the more basic p5-dbd-mysql fix will get accepted.

@jhoyt4
Copy link
Contributor Author

jhoyt4 commented Feb 22, 2025

I've also now dropped the mariadb10.2 support patch which I got cc'd in trac to look into. This is again in hopes that this can maybe someday get merged or at least get feedback as to why it is not being accepted.

  Fix issue where multiple variant conflicts were being set as a single
  conflict. Fixing this also allowed for the removal of the
  active_vartiants workaround.

  - fix variant conflicts
  - remove active variants workaround
  - fix typos

Closes: https://trac.macports.org/ticket/60517
Closes: https://trac.macports.org/ticket/71360
Closes: https://trac.macports.org/ticket/71381
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants