Skip to content

TinyDTLS Upgrades (02b)#6319

Closed
rfuentess wants to merge 17 commits intoRIOT-OS:masterfrom
rfuentess:TinyDTLS-Upgrades02b
Closed

TinyDTLS Upgrades (02b)#6319
rfuentess wants to merge 17 commits intoRIOT-OS:masterfrom
rfuentess:TinyDTLS-Upgrades02b

Conversation

@rfuentess
Copy link
Copy Markdown
Contributor

Hi people!

I have been working on my TinyDTLS branch on the last days.

This PR brings 3 things:

  1. RIOT will uses my TinyDTLS master branch by default instead of a specific commit.
  2. The default example has a series of fixes and is saving a little more of memory now.
  3. The current configuration (only PSK mode and TinyDTLS' debug messages disabled) is more stable. Probably due to other PRs related to hardware performance.

I have been speaking with @kYc0o about what is next and for now is increasing the stability for ECC. But probably, that will be reflected only on my TinyDTLS fork. After that, I'll try again with sock.


The merge conflict for this PR is in pkg/tinydtls/Makefile:

PKG_NAME=tinydtls
PKG_URL=https://github.com/rfuentess/TinyDTLS.git
<<<<<<< HEAD
# PKG_VERSION=RIOT-OS
PKG_VERSION=d06ad84d3e8bd3e86f7557faee34b68d5f8c0129
=======
PKG_VERSION=master
# (Official) Branch with tested(?) updates before merging to remote master
#PKG_VERSION=RIOT-Tested-Upgrades

>>>>>>> TinyDTLS-Upgrades02b
PKG_LICENSE=EPL-1.0,EDL-1.0

.PHONY: all

all: git-download
       $(MAKE) -C $(PKG_BUILDDIR)

include $(RIOTBASE)/pkg/pkg.mk

@OlegHahm OlegHahm added Area: crypto Area: Cryptographic libraries Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: network Area: Networking labels Jan 10, 2017
@OlegHahm OlegHahm added this to the Release 2017.01 milestone Jan 10, 2017
@OlegHahm OlegHahm added the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Jan 10, 2017
@rfuentess rfuentess force-pushed the TinyDTLS-Upgrades02b branch from cab5d3a to 371a25d Compare January 18, 2017 13:29
Removed gnrc_pktdump due it is useless for this example
Currently RawPublicKey mode (ECC) has issues with pthread for other boards than native
Variables related only to TinyDTLS moved from application Makefile
to TinyDTLS's Makefile.dep
The last PRs related to drivers have fixed the issue related to
retransmissions.

NOTE: The 802.15.4 frames for DTLS datagrams are still marking the
      ACK flag as set.
The mistake done with PKG_URL with git:// instead of https:// is
fixed.

The license is added back.
@rfuentess
Copy link
Copy Markdown
Contributor Author

I'm closing this PR in favor of #6430 that is free of the merge conflict (and squashed).

@rfuentess rfuentess closed this Jan 19, 2017
@rfuentess rfuentess deleted the TinyDTLS-Upgrades02b branch May 19, 2017 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: crypto Area: Cryptographic libraries Area: network Area: Networking CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants