Skip to content

Revert "Merge #116"#220

Merged
bors[bot] merged 1 commit intoRIOT-OS:masterfrom
kaspar030:revert_116
Dec 19, 2022
Merged

Revert "Merge #116"#220
bors[bot] merged 1 commit intoRIOT-OS:masterfrom
kaspar030:revert_116

Conversation

@kaspar030
Copy link
Contributor

It pains me a lot but the bad debian newlib package won't be fixed in time for the hard freeze, and neither will we sort out the issues with picolibc as default, in the week before x-mas. :(

This reverts commit 20529a8 (#116), reversing changes made to 604bc23.

This reverts commit 20529a8, reversing
changes made to 604bc23.
@aabadie
Copy link
Contributor

aabadie commented Dec 19, 2022

I would prefer to only revert the commits related to #116, e.g c49165b, dc6e40f, 1afc188 and 1d52cae

@aabadie
Copy link
Contributor

aabadie commented Dec 19, 2022

Hmm, this PR does this (same diff), so OK :)

@kaspar030
Copy link
Contributor Author

kaspar030 commented Dec 19, 2022

I would prefer to only revert the commits related to #116, e.g c49165b, dc6e40f, 1afc188 and 1d52cae

This is git revert -m1 20529a8aa6996c784e2ca2921e1356ee1fcdd5ee, then manually fixing a conflict in the packages.
What else did get removed? Was it the installation of the c++ packages?

edit simultaneous typing

@aabadie
Copy link
Contributor

aabadie commented Dec 19, 2022

Could you test locally if that fixes the size increase ? (I'm still building the Docker image...)

Copy link
Contributor

@aabadie aabadie left a comment

Choose a reason for hiding this comment

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

I tested locally and I confirm that the Docker images generated with the changes in this PR fixes the code size increase.

ACK

@aabadie
Copy link
Contributor

aabadie commented Dec 19, 2022

bors merge

@bors
Copy link
Contributor

bors bot commented Dec 19, 2022

Build succeeded:

  • Build and Test

@bors bors bot merged commit 5efc102 into RIOT-OS:master Dec 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants