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

Add support to xlC_r compiler on AIX #279

Merged
merged 1 commit into from
May 23, 2017

Conversation

eyalg1972
Copy link

Add support for IBM xlC_r compiler for AIX

@@ -127,9 +127,7 @@ amd64.Windows.msvc.res.excludes=

amd64.Windows.msvc.idl.compiler=midl
amd64.Windows.msvc.idl.defines=
amd64.Windows.msvc.idl.options=
Copy link
Contributor

@ctrueden ctrueden May 22, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these removals intended? If so, I think the commit message needs to explain how this is relevant to the addition of xlC_r. Same for the removal of the x86.Windows.gpp lines.

@eyalg1972
Copy link
Author

I just cleaned some duplication.... (You can still find these lines, now just in one place instead of 2)

@ctrueden
Copy link
Contributor

I just cleaned some duplication

Ah, I see, OK. Great. If you can do such things as separate commits, that would be ideal, but in the interest of pragmatism I'll merge as-is.

Thank you!

@ctrueden ctrueden merged commit e60d08f into maven-nar:master May 23, 2017
@eyalg1972
Copy link
Author

eyalg1972 commented May 24, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants