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

modification for compatibility (AIX, Android NDK), for smartversion compile #27

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gvollant
Copy link

some compatibility patch are from previous pull request

Pull requet #25 was accidentelly closed, so I reopen this PR

for smartversion compile (including android), some compatibility patch are from previous pull request
@gvollant
Copy link
Author

@janisozaur @richgel999 can you look at #27

@gvollant
Copy link
Author

@richgel999 I made a minor update to compile on macos 11 on arm64

I'll be happy if you take a look !

@gvollant
Copy link
Author

@janisozaur you can also look

@janisozaur
Copy link

My objection regarding improper commit message still stands

@janisozaur
Copy link

modification for compatibility for smartversion compile (including android), some compatibility patch are from previous pull request

is not something I would ever want to see as a commit. It does not explain what the "modification for compatibility" is; "modification", as in "code change" is already implied; the "previous pull request" doesn't refer to anything in particular.

The subject line should be glanceable and short, written in imperative mode and above all, should actually explain what it does.

E.g.:

Improve Android support

or

Extend Android support with better memory allocation

or something in that vein

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