[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#66327: PARTED_CHECK_LIBPARTED shipped with parted 3.6 fails to detec
From: |
Brian C. Lane |
Subject: |
bug#66327: PARTED_CHECK_LIBPARTED shipped with parted 3.6 fails to detect it |
Date: |
Fri, 6 Oct 2023 08:15:23 -0700 |
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