On Fri, Oct 06, 2023 at 12:12:44AM +0200, Jan Palus wrote: > On 04.10.2023 13:52, Brian C. Lane wrote: > > On Tue, Oct 03, 2023 at 07:01:52PM +0200, Jan Palus wrote: > > > autoconf macro PARTED_CHECK_LIBPARTED requires ped_get_version() to > > > return 3 part version string or otherwise it fails with "3.6, bad > > > version string" > > > > > > if ( !(version = ped_get_version ()) ) > > > exit(EXIT_FAILURE); > > > if (sscanf(version, "%d.%d.%d", &major, &minor, µ) != 3) { > > > printf("%s, bad version string\n", version); > > > exit(EXIT_FAILURE); > > > } > > > > > > Therefore it fails for two part version string like "3.6". > > > > > > > > > > > > > Thanks for the report. Where is this causing problems for you? As far as > > I can tell that code has been there since the start of the git history, > > and I've never seen a failure. I'm actually not sure it ever gets called > > by anything -- nothing uses the PARTED_CHECK_LIBPARTED macro in the > > upstream code. > > > > I'm inclined to just delete it. > > Noticed the issue when checking original fatresize codebase. Don't get > me wrong I'm completely fine with either fixing or dropping it entirely. > I just don't think it should be shipped in current shape. >
Ah, ok. Wanted to make sure it wasn't something you were depending on somehow :) This codebase is *really* old at this point so there are bound to be unused corners like this -- especially if it isn't used and this looks like it never would have worked, the first release in git is 1.8 Thanks, Brian -- Brian C. Lane (PST8PDT) - weldr.io - lorax - parted - pykickstart