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

Align to OTP 24 #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Align to OTP 24 #1

wants to merge 3 commits into from

Conversation

KornelH
Copy link

@KornelH KornelH commented Jul 26, 2021

Fix below compile-time warning. The recommended erlang:crc32/1 was introduced in Erlang/OTP R12.

egd_png.erl:102:11: zlib:crc32/2 is deprecated and will be removed in OTP 27; use erlang:crc32/1 instead

@newmancodes
Copy link

Is it possible to get this change merged? Working through some learning materials and this is a blocker for a few people.

@mcampbell
Copy link

Is it possible to get this change merged? Working through some learning materials and this is a blocker for a few people.

I've run into what appears to be this issue with an Udemy Elixir course using egd, on the latest OTP.

@anthonyhoegberg
Copy link

Im getting this issue aswell can this be merged?

@savs2020
Copy link

savs2020 commented Nov 5, 2024

Reinstall Elixir v1.17 with Erlang/OTP 26.

Fix below compile-time warning. The recommended erlang:crc32/1 was
introduced in Erlang/OTP R12.

egd_png.erl:102:11: zlib:crc32/2 is deprecated and will be removed in
                    OTP 27; use erlang:crc32/1 instead
@jgaik
Copy link

jgaik commented Jan 28, 2025

Any progress on merging that?

@anthonyhoegberg
Copy link

Looks like this repository might be abandoned? any ideas of how to preceded with such a case?

@jgaik
Copy link

jgaik commented Jan 29, 2025

I am currently using the forked version from @KornelH who applied the change there. Maybe there is someone from erlang community who can proceed with this but I am new here 😓

@KornelH
Copy link
Author

KornelH commented Feb 4, 2025

EGD is an old an unmaintained library, but still in use by some. So, I can publish the updated version on hex.pm to be able to use it via rebar3 easily. As far as I understand, the license allows it.

Kornel Horvath added 2 commits February 9, 2025 12:06
Documentation provider is changed to ex_doc [1] that required to:
- add a missing spec line to fix an ex_doc error, and
- convert erl_docgen [2] style XML documents to Markdown.

[1] https://hexdocs.pm/rebar3_ex_doc/
[2] https://www.erlang.org/docs/26/man/erl_docgen_app.html#
@KornelH
Copy link
Author

KornelH commented Feb 9, 2025

Now, it is published at hex.pm as package egd24 and can be included into any rebar3 project by adding below dependency to rebar.config:

{deps, [
    {egd, {pkg, egd24}}
]}.

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.

6 participants