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

Cpoll fix #1016

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@msmith-techempower
Member

msmith-techempower commented Aug 15, 2014

The CPLUS_INCLUDE_PATH is definitely needed.

As for using postgresql-server-dev-9.3 - this might not work on Ubuntu 12 - we shall find out.

@hamiltont

This comment has been minimized.

Show comment
Hide comment
@hamiltont

hamiltont Aug 15, 2014

Contributor

postgresql-server-dev-9.3

If this only needed for this one test, we should put a call to apt-get in the install.sh

Contributor

hamiltont commented Aug 15, 2014

postgresql-server-dev-9.3

If this only needed for this one test, we should put a call to apt-get in the install.sh

@msmith-techempower

This comment has been minimized.

Show comment
Hide comment
@msmith-techempower

msmith-techempower Aug 15, 2014

Member

Good point

Member

msmith-techempower commented Aug 15, 2014

Good point

@hamiltont

This comment has been minimized.

Show comment
Hide comment
@hamiltont

hamiltont Aug 16, 2014

Contributor

This "fixes" cpoll in that it now compiles and runs correctly. It does have some minor validation errors (json array instead of dict sort of things). Let me know if you want to fix those with this PR, or if we should merge this into master as "better than current"

Contributor

hamiltont commented Aug 16, 2014

This "fixes" cpoll in that it now compiles and runs correctly. It does have some minor validation errors (json array instead of dict sort of things). Let me know if you want to fix those with this PR, or if we should merge this into master as "better than current"

@msmith-techempower

This comment has been minimized.

Show comment
Hide comment
@msmith-techempower

msmith-techempower Aug 18, 2014

Member

@hamiltont is Travis treating both errors and warnings as failure state? It should just be errors, correct?

EDIT: ugh, morning. Accidentally hit "close and comment"

Member

msmith-techempower commented Aug 18, 2014

@hamiltont is Travis treating both errors and warnings as failure state? It should just be errors, correct?

EDIT: ugh, morning. Accidentally hit "close and comment"

@hamiltont

This comment has been minimized.

Show comment
Hide comment
@hamiltont

hamiltont Aug 18, 2014

Contributor

errors and warnings as failure state

To nitpick on terminology for a second - travis only reports errors or failures. You can print warnings all day and they don't care - only the return status is important. Failure and Error both indicate a non-zero return status from some line listed in the .travis.yml (a failure if the offender is under the script category, else an error). An error for any job marks the build as error. A failure for any job marks the build as fail.

The main source of our errors at the moment is #989 . In fact, if you just "restart" the errored jobs, they will normally launch without these network issues, complete without incident, and the entire travis build will then be marked properly.

Contributor

hamiltont commented Aug 18, 2014

errors and warnings as failure state

To nitpick on terminology for a second - travis only reports errors or failures. You can print warnings all day and they don't care - only the return status is important. Failure and Error both indicate a non-zero return status from some line listed in the .travis.yml (a failure if the offender is under the script category, else an error). An error for any job marks the build as error. A failure for any job marks the build as fail.

The main source of our errors at the moment is #989 . In fact, if you just "restart" the errored jobs, they will normally launch without these network issues, complete without incident, and the entire travis build will then be marked properly.

@msmith-techempower msmith-techempower deleted the msmith-techempower:cpoll branch Jan 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment