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

[BUG] on macOS arm64, compiling Bitcoin is throwing warnings #544

Open
bitcoin-tools opened this issue Apr 25, 2024 · 1 comment
Open

[BUG] on macOS arm64, compiling Bitcoin is throwing warnings #544

bitcoin-tools opened this issue Apr 25, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@bitcoin-tools
Copy link
Owner

bitcoin-tools commented Apr 25, 2024

Describe the bug

This is an upstream issue, but nodebuilder can suppress it without suppressing all of stdout.

Will require some grep -A -B -C magic.

The following lines are printing in the terminal output.

+ make --jobs 4
ld: warning: -single_module is obsolete
qt/macnotificationhandler.mm:27:9: warning: 'NSUserNotification' is deprecated: first deprecated in macOS 11.0 - All NSUserNotifications API should be replaced with UserNotifications.frameworks API [-Wdeprecated-declarations]
        NSUserNotification* userNotification = [[NSUserNotification alloc] init];
        ^
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/Foundation.framework/Headers/NSUserNotification.h:24:12: note: 'NSUserNotification' has been explicitly marked deprecated here
@interface NSUserNotification : NSObject <NSCopying> {
           ^
qt/macnotificationhandler.mm:27:50: warning: 'NSUserNotification' is deprecated: first deprecated in macOS 11.0 - All NSUserNotifications API should be replaced with UserNotifications.frameworks API [-Wdeprecated-declarations]
        NSUserNotification* userNotification = [[NSUserNotification alloc] init];
                                                 ^
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/Foundation.framework/Headers/NSUserNotification.h:24:12: note: 'NSUserNotification' has been explicitly marked deprecated here
@interface NSUserNotification : NSObject <NSCopying> {
           ^
qt/macnotificationhandler.mm:30:11: warning: 'NSUserNotificationCenter' is deprecated: first deprecated in macOS 11.0 - All NSUserNotifications API should be replaced with UserNotifications.frameworks API [-Wdeprecated-declarations]
        [[NSUserNotificationCenter defaultUserNotificationCenter] deliverNotification: userNotification];
          ^
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/Foundation.framework/Headers/NSUserNotification.h:118:12: note: 'NSUserNotificationCenter' has been explicitly marked deprecated here
@interface NSUserNotificationCenter : NSObject {
           ^
3 warnings generated.
ld: warning: ignoring duplicate libraries: '-levent'
ld: warning: ignoring duplicate libraries: '-levent'
ld: warning: ignoring duplicate libraries: '-levent'
ld: warning: ignoring duplicate libraries: '-levent'
ld: warning: ignoring duplicate libraries: '-levent', 'libbitcoin_util.a'
ld: warning: ignoring duplicate libraries: '-levent', 'libbitcoin_util.a'
+ printf '%s\n  %s' ok. 'Running compile checks, please wait... '
  Compiling source code, please wait... ok.

Steps to Reproduce

Run with -c option on macOS 14 / arm64. Or check the CI jobs.

Expected behavior

Should show Compiling source code, please wait... ok. without warnings.

Desktop (please complete the following information):

  • OS: macOS 14
  • Environment: bare metal
  • Release: master

Additional context
Add any other context about the problem here.

@bitcoin-tools bitcoin-tools added the bug Something isn't working label Apr 25, 2024
@bitcoin-tools bitcoin-tools changed the title [BUG] on macOS 14 [arm64], compiling Bitcoin is throwing warnings [BUG] on macOS arm64, compiling Bitcoin is throwing warnings Apr 25, 2024
@bitcoin-tools
Copy link
Owner Author

Resolved temporarily by suppressing stderr - in order to be able to merge the test-compile feature.

The goal is to not suppress all of stderr, and instead to only do it selectively for certain warnings.

Leaving this issue open until then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant