Skip to content

Conversation

@mhdawson
Copy link
Member

Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

core

Description of change

Fix compile failure in backtrace_posix.c on AIX introduced by #6734

Fix compile failure in backtrace_posix.c on AIX
@nodejs-github-botnodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Jul 5, 2016
@mhdawson
Copy link
MemberAuthor

@bnoordhuis

@mhdawson
Copy link
MemberAuthor

@cjihrig
Copy link
Contributor

LGTM

@mhdawson
Copy link
MemberAuthor

Going to land this once the CI run completes since it fixes a build break and the change is small and have one LGTM altready.

@bnoordhuis
Copy link
Member

LGTM

1 similar comment
@mscdex
Copy link
Contributor

LGTM

@mhdawson
Copy link
MemberAuthor

CI is good landing

@mhdawson
Copy link
MemberAuthor

Landed as 95a3c8f

mhdawson added a commit that referenced this pull request Jul 5, 2016
Fix compile failure in backtrace_posix.c on AIX Fixes: #7539 PR-URL: #7544 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Brian White <[email protected]>
@mhdawsonmhdawson closed this Jul 5, 2016
Fishrock123 pushed a commit that referenced this pull request Jul 6, 2016
Fix compile failure in backtrace_posix.c on AIX Fixes: #7539 PR-URL: #7544 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Brian White <[email protected]>
@Fishrock123Fishrock123 mentioned this pull request Jul 6, 2016
@MylesBorins
Copy link
Contributor

@mhdawson lts?

@MylesBorins
Copy link
Contributor

needs to be backported if #6734 is backported

@mhdawsonmhdawson deleted the aixstack branch March 15, 2017 21:56
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@mhdawson@cjihrig@bnoordhuis@mscdex@MylesBorins@nodejs-github-bot