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

Compute both 32 and 64 bit hashes on the same arch #817

Merged
merged 5 commits into from Jul 10, 2013
Merged

Compute both 32 and 64 bit hashes on the same arch #817

merged 5 commits into from Jul 10, 2013

Conversation

ghost
Copy link

@ghost ghost commented Jul 10, 2013

This allows one to generate 32bits/phalcon.c and 64bits/phalcon.c on the same computer, without having to switch to a VM.

_Rebased #813 against 1.2.1_

phalcon pushed a commit that referenced this pull request Jul 10, 2013
Compute both 32 and 64 bit hashes on the same arch
@phalcon phalcon merged commit 71453c5 into phalcon:1.2.1 Jul 10, 2013
@phalcon
Copy link
Collaborator

phalcon commented Jul 10, 2013

This is very useful!

@ghost ghost deleted the hash_32_64 branch July 10, 2013 03:47
@ghost
Copy link
Author

ghost commented Jul 10, 2013

Please check that it works on Windows — I don't have a Windows box at the moment and not sure it will compile properly.

@phalcon
Copy link
Collaborator

phalcon commented Jul 10, 2013

ok, regarding the simplified travis.yml, I merged the pull request into master, are those changes in 1.2.1 too?

@ghost
Copy link
Author

ghost commented Jul 10, 2013

Yes

@phalcon
Copy link
Collaborator

phalcon commented Jul 10, 2013

thanks man

@ghost
Copy link
Author

ghost commented Jul 10, 2013

Sorry, actually no - you will need to merge them to 1.2.1 as well

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.

3 participants