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

travis: add CXXFLAGS=-Wno-psabi at ARM job #14042

Merged
merged 1 commit into from Aug 27, 2018

Conversation

Projects
None yet
5 participants
@ken2812221
Copy link
Member

commented Aug 24, 2018

Disable annoying ABI warnings

@fanquake fanquake added the Tests label Aug 24, 2018

@Empact

This comment has been minimized.

Copy link
Member

commented Aug 24, 2018

Maybe comment as to the specific version message being disabled so it's easier to remove the flag when the cause passes?

@laanwj

This comment has been minimized.

Copy link
Member

commented Aug 27, 2018

utACK 5a0e67c01999873417da4586d409c7909a10e095

Though yes, please add a comment, so that later maintainers know why this was added and can remove it when it's no longer necessary.

@ken2812221 ken2812221 force-pushed the ken2812221:patch-1 branch to 4c530e0 Aug 27, 2018

@ken2812221

This comment has been minimized.

Copy link
Member Author

commented Aug 27, 2018

@Empact @laanwj Comment added.

@MarcoFalke MarcoFalke merged commit 4c530e0 into bitcoin:master Aug 27, 2018

1 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

MarcoFalke added a commit that referenced this pull request Aug 27, 2018

Merge #14042: travis: add CXXFLAGS=-Wno-psabi at ARM job
4c530e0 travis: add  CXXFLAGS=-Wnopsabi at ARM job (Chun Kuan Lee)

Pull request description:

  Disable annoying ABI warnings

Tree-SHA512: 2f98c67c2a1a67f7e05c81ce82783b44083876709209f569d0b1162355b9bb0bd354769c36289bea7f707b002b42d71274f7d9f3c18401b67b404eb7b76c6eea

@ken2812221 ken2812221 deleted the ken2812221:patch-1 branch Sep 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.