archinstall icon indicating copy to clipboard operation
archinstall copied to clipboard

downgrade pyparted due to "unrelease"

Open alan-barzilay opened this issue 1 year ago • 5 comments

somehow and for some reason, pyparted 3.13 is no longer available in pypi only in its git repository. To deal with this I downgraded the version requirement and updated to >= instead of the == exact version match

  • This fix issue:

PR Description:

Tests and Checks

  • [ ] I have tested the code!

alan-barzilay avatar Dec 02 '23 20:12 alan-barzilay

relevant links: https://pypi.org/project/pyparted/ image

https://github.com/dcantrell/pyparted/ image

alan-barzilay avatar Dec 02 '23 20:12 alan-barzilay

If I recall correctly, we had issues with older versions of pyparted (libparted), so I'm wondering if this will maybe ensure that pyparted gets installed yes - but breaks calls later?

I'll have to verify this first. Better odds are probably to get the packager of PyParted's attention and push the release as they should. It's a pretty minor thing as they only need to run ~1 command more or less.

Torxed avatar Dec 04 '23 08:12 Torxed

Adding a cross reference to your issue for traceability: https://github.com/dcantrell/pyparted/issues/106

Torxed avatar Dec 06 '23 22:12 Torxed

@Torxed , since it has been a few months and nothing happened on the pyparted side, could we move on with the downgrade here? I'm unable to use archinstall in production do to this.. If they release newer versions in the future we can still upgrade the requirement, the downgrade would be just a band-aid to fix the pip installation (which, as far as I understand, is fundamentally broken now) while pyparted does not break their inertia and start moving

alan-barzilay avatar Mar 07 '24 13:03 alan-barzilay

@Torxed up haha

alan-barzilay avatar Mar 31 '24 22:03 alan-barzilay

I'm not closing this one just yet, but I have submitted a PR that could potentially replace this one. Instead of downgrading, I'm exploring just running around the problem completely. Because.. frustrations are a bit high atm with this dependency :P

Torxed avatar May 09 '24 21:05 Torxed

Ok review went quick, closing this. But if we bump into issues we can revisit this one.

Torxed avatar May 09 '24 21:05 Torxed