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

Fixes for Xcode 6.3 #559

Merged
merged 2 commits into from Apr 10, 2015
Merged

Fixes for Xcode 6.3 #559

merged 2 commits into from Apr 10, 2015

Conversation

quantumsteve
Copy link
Contributor

This Fixes #11509.

Xcode 6.3 was released today and two small changes are required for Mantid to build without errors or warnings.

testing: Code review should be sufficient since our official OS X builds will remain on OSX 10.9 and Xcode 6.2.

@quantumsteve quantumsteve added the Bug Issues and pull requests that are regressions or would be considered a bug by users (e.g. crashing) label Apr 10, 2015
@quantumsteve quantumsteve added this to the Release 3.4 milestone Apr 10, 2015
@quantumsteve
Copy link
Contributor Author

OS X 10.9 now generates a warning "unknown warning group"...

Maybe we can instead generate a seg fault via NULL pointer dereference?

int *ptr = NULL;
*ptr = 1;

http://en.wikipedia.org/wiki/Segmentation_fault

@quantumsteve
Copy link
Contributor Author

The NULL pointer dereference code mentioned above generates a seg fault on OS X. Once the build servers finish, this ticket is ready for testing.

@stuartcampbell stuartcampbell self-assigned this Apr 10, 2015
stuartcampbell added a commit that referenced this pull request Apr 10, 2015
@stuartcampbell stuartcampbell merged commit fa6491d into master Apr 10, 2015
@stuartcampbell stuartcampbell deleted the 11509_xcode_6p3_fixes branch April 10, 2015 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues and pull requests that are regressions or would be considered a bug by users (e.g. crashing)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants