Drop macOS ForceActivation workaround #744

pull fanquake wants to merge 1 commits into bitcoin-core:master from fanquake:drop_macos_force_workaround changing 2 files +0 −15
  1. fanquake commented at 6:23 pm on June 30, 2023: member
    Discussion in https://github.com/bitcoin/bitcoin/pull/16720 seems to point to this being no-longer needed after qt 5.6+. We now require 5.11.x+.
  2. gui: drop macOS ForceActivation workaround
    Discussion in #16720 seems to point to this being no-longer needed after
    qt 5.6+. We now required 5.11.x+.
    362e989373
  3. DrahtBot commented at 6:23 pm on June 30, 2023: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    Concept ACK hebasto

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

  4. hebasto renamed this:
    gui: drop macOS ForceActivation workaround
    Drop macOS ForceActivation workaround
    on Jul 3, 2023
  5. hebasto commented at 11:58 am on July 3, 2023: member

    Concept ACK.

    We now require 5.11.x+.

    When building on macOS, Homebrew provides Qt 5.15.10. Our depends does 5.15.5.

    cc @Sjors

  6. hebasto commented at 12:37 pm on July 3, 2023: member
    I think we can drop -DOBJC_OLD_DISPATCH_PROTOTYPES=0 now. At least, the code compiles for me natively on macOS Ventura (arm64) and when cross-building on Ubuntu 23.04.
  7. hebasto added the label macOS on Jul 3, 2023
  8. Sjors commented at 2:36 pm on July 3, 2023: member

    Tested 362e98937356a74a5b70d65348957edf05b1138a but it seems the workaround is still needed.

    This was introduced in https://github.com/bitcoin/bitcoin/pull/14123 as a fix to https://github.com/bitcoin/bitcoin/issues/13829.

    With this PR, built on macOS 13.4.1 (Intel) with qt@5/5.15.10 via Homebrew, when I use ⌘ + H to hide the window, and then e.g. click on “Receive” in the dock, the window doesn’t appear. Without this PR the window does appear.

  9. fanquake closed this on Jul 27, 2023

  10. bitcoin-core locked this on Jul 26, 2024

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/gui. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-21 09:20 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me