iluvcapra / wavinfo

Probe WAVE Files for all metadata
https://wavinfo.readthedocs.io/
MIT License
33 stars 7 forks source link

Incompatible dependencies with PyTest #9

Closed kakyoism closed 2 years ago

kakyoism commented 2 years ago

First of all, thanks for this little tool! It proves very useful in our projects.

However, we now have a dependency conflict problem, see the error report from Poetry below

  SolverProblemError

  Because no versions of ear match <1.0.0 || >1.0.0,<1.0.1 || >1.0.1,<1.1.0 || >1.1.0,<1.1.1 || >1.1.1,<1.1.2 || >1.1.2,<1.2.0 || >1.2.0,<2.0.0 || >2.0.0
   and ear (1.0.0) depends on attrs (>=17.4,<18.0), ear (<1.0.1 || >1.0.1,<1.1.0 || >1.1.0,<1.1.1 || >1.1.1,<1.1.2 || >1.1.2,<1.2.0 || >1.2.0,<2.0.0 || >2.0.0) requires attrs (>=17.4,<18.0).
  And because ear (1.0.1) depends on attrs (>=17.4,<18.0)
   and ear (1.1.0) depends on attrs (>=17.4,<18.0), ear (<1.1.1 || >1.1.1,<1.1.2 || >1.1.2,<1.2.0 || >1.2.0,<2.0.0 || >2.0.0) requires attrs (>=17.4,<18.0).
  And because ear (1.1.1) depends on attrs (>=17.4,<18.0)
   and ear (1.1.2) depends on attrs (>=17.4,<18.0), ear (<1.2.0 || >1.2.0,<2.0.0 || >2.0.0) requires attrs (>=17.4,<18.0).
  And because ear (1.2.0) depends on attrs (>=17.4,<18.0)
   and ear (2.0.0) depends on attrs (>=17.4,<18.0), every version of ear requires attrs (>=17.4,<18.0).
  And because wavinfo (1.6.3) depends on ear (*)
   and pytest (6.2.0) depends on attrs (>=19.2.0), wavinfo (1.6.3) is incompatible with pytest (6.2.0).
  So, because MyProj depends on both pytest (6.2.0) and wavinfo (1.6.3), version solving failed.

I understand that it's mainly because ear has not been updated for a while. But can we do something here?

Thanks again!

iluvcapra commented 2 years ago

I'll have to check the ear requirements, they've probably both moved on a little from where they were at when I added them. You can try just changing it in the source to the latest version for now, I won't have time to look at this until next week.

As it is ear is only used for the ADM features I'll have to look at wether they're still required or can be worked around.

tomjnixon commented 2 years ago

This should be fixed with the latest EAR release. I've made the attrs requirement more lenient so hopefully this will not happen again.

iluvcapra commented 2 years ago

@tomjnixon Thanks! For the update!