Skip to content

Fix travis#20

Open
smalls12 wants to merge 7 commits intocrystal-community:masterfrom
smalls12:master
Open

Fix travis#20
smalls12 wants to merge 7 commits intocrystal-community:masterfrom
smalls12:master

Conversation

@smalls12
Copy link

I cleaned up the travis.yml ( simplified ).
libzmq is cloned and built with cmake now.

The error currently breaking travis is that the shared library could not be found, which I addressed by installing zeromq to a location where crystal spec can find it.

That being said, this crystal binding should be built with the static lib to remove that problem entirely.

Also, now that travis can run, crystal spec will timeout on a test.

Clone zeromq from git
Build with cmake
Download ameba
Comment out running ameba for now
Speed up libzmq building
@smalls12
Copy link
Author

Seems to be a couple different errors in the spec

  1. socket_spec.cr
    generate a EFSM error when sending via the REQ socket twice in a row without an intervening receive operation
    This could be due to a different version of libzmq

  2. device_spec.cr
    There is a lockup on one of the tests

  3. nonblocking_socket_spec.cr
    Seems completely broken, albeit probably some small problem with API_Helper maybe

The example in the README doesn't actually work, it would just deadlock
Made an explicit basic example to run and updated README
The change was to remove the loop
bcardiff added a commit that referenced this pull request Sep 16, 2023
bcardiff added a commit that referenced this pull request Sep 16, 2023
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.

1 participant

Comments