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.

Sign up for GitHub

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

Jump to bottom

cpu/atmega_common: include time.h for time_t #5974

Merged
merged 1 commit into from
Oct 25, 2016
Merged

cpu/atmega_common: include time.h for time_t #5974

merged 1 commit into from
Oct 25, 2016

Conversation

cgundogan
Copy link
Member

building applications that use timeval for atmega_common breaks, because time_t is not found. This commit includes avr-libc's time.h for time_t.

@rfuentess is this fixing your problem with tinydtls?

@cgundogan cgundogan added the Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) label Oct 20, 2016
@cgundogan cgundogan added this to the Release 2016.10 milestone Oct 20, 2016
@rfuentess
Copy link
Contributor

rfuentess commented Oct 21, 2016 •
edited
Loading

@cgundogan This fix issues with the boards cc2538dk, msbiot and telosb

Edit:

And partially the waspmote-pro but remains this issue:

RIOT/examples/dtls-echo/bin/pkg/waspmote-pro/tinydtls/crypto.c:40:21: fatal error: pthread.h: No such file or directory
 #include <pthread.h>
                     ^

PeterKietzmann
PeterKietzmann approved these changes Oct 22, 2016
View reviewed changes
Copy link
Member

@PeterKietzmann PeterKietzmann left a comment

Choose a reason for hiding this comment

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

ACK

@PeterKietzmann PeterKietzmann added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 22, 2016
@rfuentess rfuentess mentioned this pull request Oct 22, 2016
Merged
@kaspar030 kaspar030 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Oct 24, 2016
@PeterKietzmann PeterKietzmann added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Oct 24, 2016
@PeterKietzmann PeterKietzmann merged commit d8d658f into RIOT-OS:master Oct 25, 2016
@cgundogan cgundogan deleted the pr/atmega_common_time branch October 26, 2016 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PeterKietzmann PeterKietzmann PeterKietzmann approved these changes

Assignees

@kYc0o kYc0o

Labels
CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Milestone
Release 2016.10
Development

Successfully merging this pull request may close these issues.
5 participants
@cgundogan @rfuentess @PeterKietzmann @kaspar030 @kYc0o