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

Improve Travis CI tests #236

Merged
merged 9 commits into from
Oct 31, 2017

Conversation

morgenroth
Copy link

More dependencies added to cover more code during build.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 63.766% when pulling aa62195 on morgenroth:dependencies-and-travisci into e59e3c4 on ibrdtn:master.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 6022d01 on morgenroth:dependencies-and-travisci into ** on ibrdtn:master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 6022d01 on morgenroth:dependencies-and-travisci into ** on ibrdtn:master**.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 63.774% when pulling cd421f0 on morgenroth:dependencies-and-travisci into 7f702b6 on ibrdtn:master.


./configure CXXFLAGS="-ggdb"
./configure --enable-debug

Choose a reason for hiding this comment

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

actually this produces Makefiles with "-O2", that's hard to debug.

Copy link
Author

Choose a reason for hiding this comment

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

Yes. This is the default behavior of gcc and not the responsibility of this PR. Please open a ticket if you want to mention this as a missing feature of the "--enable-debug" option.

The package version of vmime is deprecated in Travis CI builds. As documented
in [1], this version does not work with clang builds. For now, we disable
that feature for clang tests.

[1] kisli/vmime#92
The dtndht feature is disabled for clang builds because libtool sorts
the linker options wrong and leads to unresolved functions.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 63.75% when pulling 21f42ac on morgenroth:dependencies-and-travisci into 5b227ef on ibrdtn:master.

Since clang builds are restricted, we build a variant without netlink
using gcc.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 63.766% when pulling 9f93e97 on morgenroth:dependencies-and-travisci into 5b227ef on ibrdtn:master.

@morgenroth morgenroth merged commit 3a9c0de into ibrdtn:master Oct 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants