-
Notifications
You must be signed in to change notification settings - Fork 70
ateneo #12
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
Open
dlagazo
wants to merge
28
commits into
morgenroth:master
Choose a base branch
from
ibrdtn:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
ateneo #12
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- missing header for stand alone complitation, - error: stack frame size of N bytes in functions 'CipherStreamTest::aesstream_test01' [-Werror,-Wframe-larger-than=] 'TUN2BundleGateway::process' [-Werror,-Wframe-larger-than=] 'TUN2BundleGateway::received' [-Werror,-Wframe-larger-than=]
error: 'dtn::storage::DataStorage::Callback' has virtual functions but non-virtual destructor [-Werror,-Wnon-virtual-dtor]. error: 'dtn::daemon::Configuration::Extension' has virtual functions but non-virtual destructor [-Werror,-Wnon-virtual-dtor] error: 'dtn::daemon::Configuration::OnChangeListener' has virtual functions but non-virtual destructor [-Werror,-Wnon-virtual-dtor]
Resolving build issues for some platforms
Enabling compilation in parallel for Travis
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.
Since clang builds are restricted, we build a variant without netlink using gcc.
Improve Travis CI tests
std::list.erase(it) invalidates the iterator, the following call to delete results in segfault. http://kera.name/articles/2011/06/iterator-invalidation-rules/
As long as the guidelines are limited to C++ features of C++03, the data() function of std::vector is not available.
DatagramClTest::queueTest line 108 references msg.data[0] which causes 'std::out_of_range' exception where msg.data.size() is 0. DatagramClTest::queueTestterminate called after throwing an instance of 'std::out_of_range' what(): vector::_M_range_check: __n (which is 0) >= this->size() (which is 0)
Fixes issue #242. W/o enable-debug: g++ -DHAVE_CONFIG_H -I.. -O3 -g -pipe -MT SyslogStream.lo -MD -MP -MF .deps/SyslogStream.Tpo -c -o SyslogStream.lo SyslogStream.cpp With enable-debug: g++ -DHAVE_CONFIG_H -I.. -ggdb -g3 -O0 -pipe -Wall -Wextra -pedantic -Wconversion -D__DEVELOPMENT_ASSERTIONS__ -MT SyslogStream.lo -MD -MP -MF .deps/SyslogStream.Tpo -c -o SyslogStream.lo SyslogStream.cpp
Adding option to connect to the daemon via a unix socket similarly to other tools.
This patch adds compatibility to openssl 1.1.0. Signed-off-by: Eneas U de Queiroz <cote2004-github@yahoo.com>
Fixes build with gcc-8.2.0.
This patch adds compatibility with openssl 1.1.0 to ibrdtnd. Signed-off-by: Eneas U de Queiroz <cote2004-github@yahoo.com>
gf_mul is already defined in libcrypto (openssl) so rename it into ibrdtn_gf_mul to fix following build failure: /home/buildroot/autobuild/instance-3/output/host/bin/../arm-buildroot-uclinux-uclibcgnueabi/sysroot/usr/lib/libcrypto.a(f_impl.o): In function `gf_mul': f_impl.c:(.text+0x0): multiple definition of `gf_mul' /home/buildroot/autobuild/instance-3/output/host/arm-buildroot-uclinux-uclibcgnueabi/sysroot/usr/lib/libibrcommon.a(gf128mul.o):gf128mul.cpp:(.text+0x30): first defined here collect2: error: ld returned 1 exit status Makefile:560: recipe for target 'dtnd' failed Fixes: - http://autobuild.buildroot.org/results/1d3b4b6cf043a3e185ce758b617a0a18c3d36cdb Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> Co-authored-by: Johannes Morgenroth <johannes@morgenroth.me>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.