scalameta / scalameta

Library to read, analyze, transform and generate Scala programs
http://scalameta.org/
BSD 3-Clause "New" or "Revised" License
1.11k stars 223 forks source link

Remove dependency to geny < 1.0.0 or shading geny #3106

Closed tanishiking closed 1 year ago

tanishiking commented 1 year ago

Related issues

Current scalameta parser depends on forked version of fastparse, which depends on geny 0.6.5. Since, geny < 1.0.0, and geny 1.0.0 are binary incompatible, apps that depends on scalameta cannot update have geny >= 1.0.0.

Not sure how much scalameta fork is aprt from the original one, but it would be awesone if wr can go back to li-hayoi fastparse, I'll take a quick look We did that to maintain a binary compatibility and avoid dependency issues such as with Geny, so there are three solutions here:

So, isn't going back to li-haoyi/fastparse 3.0.1 (that depends on geny 1.0.0) and bump scalameta/scalameta as 5.0.0 (because it's not binary compatible by geny) an option?

tgodzik commented 1 year ago

So, isn't going back to li-haoyi/fastparse 3.0.1 (that depends on geny 1.0.0) and bump scalameta/scalameta as 5.0.0 (because it's not binary compatible by geny) an option?

We actually spend a considerable amount of effort to keep the binary compatibility, so I would say that currently it's not an option.

nafg commented 1 year ago

How much effort would it be to shade geny, then?

tgodzik commented 1 year ago

How much effort would it be to shade geny, then?

Should be doable in a day I think when anyone has the time