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

link to git.eclipse.org submodule broken #531

Conversation

elasticdotventures
Copy link

Please update dtls submodule to
https://github.com/eclipse/tinydtls

Breaks platform.io / espressif esp-idf 4.0 toolchain installation
(which still doesn't work, because they point at a specific commit, but the maintainers should update to point at the release-v4.2.0 once the submodule link is corrected)

@mrdeep1
Copy link
Collaborator

mrdeep1 commented Jul 22, 2020

@elasticdotventures Any reason as to why you cannot use release 4.2.1?

@obgm obgm added the tinydtls label Jul 23, 2020
obgm added a commit that referenced this pull request Jul 23, 2020
This change points the submodule to the changed tinydtls repository
which now lives at https://github.com/eclipse/tinydtls and updates
to the latest development HEAD.

Closes PR #531. [1]

[1] #531
obgm added a commit that referenced this pull request Jul 23, 2020
This change points the submodule to the changed tinydtls repository
which now lives at https://github.com/eclipse/tinydtls and updates
to the latest development HEAD.

Closes PR #531. [1]

[1] #531
obgm added a commit that referenced this pull request Jul 23, 2020
This change points the submodule to the changed tinydtls repository
which now lives at https://github.com/eclipse/tinydtls and updates
to the latest development HEAD.

Closes PR #531. [1]

[1] #531
@obgm
Copy link
Owner

obgm commented Jul 23, 2020

Thank you for pointing this out. I have manually applied the change and updated to the latest tinydtls development HEAD in the release branches and the develop branch because your commit message was a bit unspecific.

@mrdeep1
Copy link
Collaborator

mrdeep1 commented Aug 12, 2020

@elasticdotventures Is your issue now fixed (the change was put in 017528d
for release 4.2.0) ? If so, can you please close off this PR.

@obgm
Copy link
Owner

obgm commented Oct 14, 2020

Closing as this seems to have been resolved.

@obgm obgm closed this Oct 14, 2020
espressif-bot pushed a commit to espressif/esp-idf that referenced this pull request Oct 21, 2020
Though IDF v4.0 does not have dependency on `tinydtls` repository
within libcoap, it created problem in recursive clone. This commit
picks up change from upstream libcoap release-v4.2.0 to update
`tinydtls` repository pointer.

More information at:
obgm/libcoap#531

Merges #5626
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants