SCIENTIFIC-LINUX-DEVEL Archives

November 2019

SCIENTIFIC-LINUX-DEVEL@LISTSERV.FNAL.GOV

Options: Use Monospaced Font
Show Text Part by Default
Show All Mail Headers

Message: [<< First] [< Prev] [Next >] [Last >>]
Topic: [<< First] [< Prev] [Next >] [Last >>]
Author: [<< First] [< Prev] [Next >] [Last >>]

Print Reply
Subject:
From:
"Patrick J. LoPresti" <[log in to unmask]>
Reply To:
Patrick J. LoPresti
Date:
Sun, 3 Nov 2019 07:02:16 -0800
Content-Type:
text/plain
Parts/Attachments:
text/plain (69 lines)
That fixed it.

Thank you, Pat!

 - Pat

On Fri, Nov 1, 2019 at 6:51 AM Pat Riehecky <[log in to unmask]> wrote:
>
> Can you clear you cache and test again?
>
> Pat
>
> On 11/1/19 8:20 AM, Pat Riehecky wrote:
> > Can do.
> >
> > Good catch!
> >
> > On 10/31/19 3:28 PM, Patrick J. LoPresti wrote:
> >> I decided to try playing with dnf-4.0.9.2-1.el7_6.noarch from
> >> 7x/external_products/extras/x86_64. But whenever I attempt any dnf
> >> operation with the main SL7 repository configured (e.g. 7x/x86_64/os),
> >> dnf fails to load the repository with the following error:
> >>
> >>    basic_string::_S_construct null not valid, ignoring this repo.
> >>
> >> I tracked this down to these lines in libdnf (see
> >> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_rpm-2Dsoftware-2Dmanagement_libdnf_blob_ad6d26fb0b9c92838b98513be47c2837ca165ce7_libdnf_repo_Repo.cpp-23L1025&d=DwIBaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=gd8BzeSQcySVxr0gDWSEbBYBgsCHS7vSr83lHQ-aa58eXICk1KkzZlAHgohLRRrX&m=oV6hpXkb7j1aO27zXIgqbe1G5TT-orJBbynPDrTir4w&s=rOy-wFxOo11xo1NKNy5gv6hwcthwhz46XSDw6bhztxI&e=
> >> ):
> >>
> >>    if (distroTag->tag)
> >>      distro_tags.emplace_back(distroTag->cpeid, distroTag->tag);
> >>
> >> The problem is that distroTag->tag is non-NULL, but distroTag->cpeid
> >> *is* NULL. So this code tries to create a std::string from a NULL
> >> pointer, which throws a C++ runtime_error exception.
> >>
> >> These values originate from the repomd.xml file. The CentOS 8
> >> repomd.xml has this <distro> tag (note "cpeid" attribute):
> >>
> >>      <distro cpeid="cpe:/o:centos:centos:8">CentOS 8</distro>
> >>
> >> The Centos 7 repomd.xml has no <distro> tag at all. Both of these
> >> repositories work fine.
> >>
> >> But SL7 has this <distro> tag (note missing "cpeid" attribute):
> >>
> >> <distro>cpe:/o:scientificlinux:scientificlinux:7.7:GA</distro>
> >>
> >> Since the <distro> tag is present but the "cpeid" attribute is NULL,
> >> the failure path in libdnf is triggered.
> >>
> >> When I manually edit the SL7 repomd.xml to add "cpeid" attribute to
> >> the <distro> tag, or when I simply remove the <distro> tag entirely,
> >> dnf becomes happy with the repository.
> >>
> >> Could you modify repomd.xml to make it compatible with dnf?
> >>
> >> Thanks!
> >>
> >>   - Pat
> >
>
> --
> Pat Riehecky
>
> Fermi National Accelerator Laboratory
> http://www.fnal.gov
> https://urldefense.proofpoint.com/v2/url?u=http-3A__www.scientificlinux.org&d=DwIFaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=gd8BzeSQcySVxr0gDWSEbBYBgsCHS7vSr83lHQ-aa58eXICk1KkzZlAHgohLRRrX&m=wUjGL5z6Pzg4QooQb3z91jAdjUztGh0EPH2CkuvQLis&s=-ydBkBl2jifsbpANCBZfxJGQEJVYKwRQcFcrIi2tJ6A&e= 

ATOM RSS1 RSS2