Closed Bug 952623 Opened 11 years ago Closed 9 years ago

Error "use of undeclared identifier" if building with macos-target=10.9

Categories

(Toolkit :: Crash Reporting, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1123356

People

(Reporter: Nomis101, Assigned: Nomis101)

Details

Attachments

(1 file)

As noted in https://bugzilla.mozilla.org/show_bug.cgi?id=909914#c6 if building on Mac with macos-target=10.9 you see two errors.


/Volumes/Developer/mozilla-central/toolkit/crashreporter/google-breakpad/src/client/mac/crash_generation/crash_generation_server.cc:67:30: error: 
      use of undeclared identifier 'pthread_create'
  int thread_create_result = pthread_create(&server_thread_, NULL,
                             ^
/Volumes/Developer/mozilla-central/toolkit/crashreporter/google-breakpad/src/client/mac/crash_generation/crash_generation_server.cc:83:30: error: 
      use of undeclared identifier 'pthread_join'; did you mean 'pthread_kill'?
    int thread_join_result = pthread_join(server_thread_, NULL);
                             ^~~~~~~~~~~~
                             pthread_kill
/Developer/SDKs/MacOSX10.9.sdk/usr/include/signal.h:80:5: note: 'pthread_kill'
      declared here
int     pthread_kill(pthread_t, int);
Attached patch FixSplinter Review
This will fix the two errors.
Attachment #8350758 - Flags: review?(ted)
Comment on attachment 8350758 [details] [diff] [review]
Fix

Review of attachment 8350758 [details] [diff] [review]:
-----------------------------------------------------------------

I'll land this upstream for you.
Attachment #8350758 - Flags: review?(ted) → review+
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #2)
> Comment on attachment 8350758 [details] [diff] [review]
> Fix
> 
> Review of attachment 8350758 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I'll land this upstream for you.
Thanks!
Sorry, I ended up rewriting this.  :-)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: