inflex icon indicating copy to clipboard operation
inflex copied to clipboard

Make regexes OTP28 compatible

Open warmwaffles opened this issue 7 months ago • 14 comments

This makes the library compatible with OTP28

Closes: #98

warmwaffles avatar May 21 '25 19:05 warmwaffles

@nurugger07 anything that we can help to let you merge this PR?

ardhitama avatar Jun 09 '25 11:06 ardhitama

How are we going with merging that?

ludwikbukowski avatar Jun 11 '25 18:06 ludwikbukowski

Anybody has permission to merge? This seems to be an important elixir library but it looks abandoned.

Maybe we could move it to a community organization?

kuon avatar Jun 19 '25 14:06 kuon

Others have just taken the entire library and inlined it into their projects or depend on forks. I have no idea who to contact to merge this and get a new version published.

warmwaffles avatar Jun 19 '25 17:06 warmwaffles

@josevalim I am sorry to poke you, but this is a very popular library and I was wondering if you had any idea where to fork it to keep it maintained by the community if we cannot get PR merged to this repo.

kuon avatar Jun 19 '25 19:06 kuon

@warmwaffles thank you for the fix! Until this PR is resolved, my temporary solution is to add the following line to my deps:

{:inflex, git: "https://github.com/warmwaffles/inflex.git", branch: "master", override: true}

Disclaimer: I'm an Elixir beginner.

timonweb avatar Jun 24 '25 19:06 timonweb

@timonweb in case you want to have it shorter, you could also do it like this:

{:inflex, github: "warmwaffles/inflex", branch: "master", override: true}

That's how I have it in my project.

gmile avatar Jun 24 '25 19:06 gmile

I'll be sure to leave this branch alone until this is merged upstream or until a new package takes over.

warmwaffles avatar Jun 24 '25 20:06 warmwaffles

thanks for the patch @warmwaffles!

afomi avatar Jul 06 '25 14:07 afomi

Btw, we will likely have a fix for this in Erlang/OTP 28.1, so stay on 27 until 28.1 is out and you may be able to upgrade without issues.

josevalim avatar Jul 06 '25 15:07 josevalim

for the sake of completeness: https://github.com/erlang/otp/pull/9976

ardhitama avatar Jul 10 '25 14:07 ardhitama

Hmm... This is not fixed with OTP 28.1, is it?

mruoss avatar Sep 19 '25 19:09 mruoss

It requires a new Elixir version.

josevalim avatar Sep 19 '25 22:09 josevalim

I've pushed a hex build that works on otp28, please send and PRs to noizu-labs-scaffolding/inflex28

def deps do [ {:inflex28, "~> 2.1.0"} ] end

noizu avatar Sep 30 '25 02:09 noizu