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

Fix esp32sim erlang eunit #1420

Merged
merged 6 commits into from
Dec 26, 2024
Merged

Conversation

bettio
Copy link
Collaborator

@bettio bettio commented Dec 26, 2024

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Just keep one list of atoms, and use the old good X macro trick
https://en.wikipedia.org/wiki/X_macro

Signed-off-by: Davide Bettio <davide@uninstall.it>
No need to make a lot of checks: default atoms insertion is not error
prone anymore.

Signed-off-by: Davide Bettio <davide@uninstall.it>
Using same approach used in defaultatoms also for ESP32 platform ones.

Signed-off-by: Davide Bettio <davide@uninstall.it>
Using same approach used in defaultatoms also for generic_unix platform ones.

Signed-off-by: Davide Bettio <davide@uninstall.it>
…able

Make defaultatoms.h/.c easier to maintain

Just keep one list of atoms, and use the old good X macro trick
https://en.wikipedia.org/wiki/X_macro

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later
Install erlang-eunit as it has been already done for esp32-build.yaml.

Signed-off-by: Davide Bettio <davide@uninstall.it>
@bettio bettio merged commit 6fa7546 into atomvm:main Dec 26, 2024
93 of 98 checks passed
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.

1 participant