search

rpm-software-management / dnf5

Next-generation RPM package management system
Other
233 stars 76 forks source link

dnf5 segfaults when --forcearch contains unsupported architecture #869

Closed m-blaha closed 11 months ago

m-blaha commented 11 months ago 
❯ dnf5 --repoid=dummyrepo install --forcearch=unknown_arch acpi
terminate called after throwing an instance of 'std::logic_error'
  what():  basic_string: construction from null is not valid
Aborted (core dumped)
jan-kolarik commented 11 months ago

Isn't this already fixed in upstream? I've tested the given prompt:

# dnf5 --repoid=dummyrepo install --forcearch=unknown_arch acpi
Unsupported architecture "unknown_arch". Please choose one from "aarch64", "alpha", "alphaev4", "alphaev45", "alphaev5", "alphaev56", "alphaev6", "alphaev67", "alphaev68", "alphaev7", "alphapca56", "amd64", "armv5tejl", "armv5tel", "armv5tl", "armv6hl", "armv6l", "armv7hl", "armv7hnl", "armv7l", "armv8hcnl", "armv8hl", "armv8hnl", "armv8l", "athlon", "geode", "i386", "i486", "i586", "i686", "ia32e", "ia64", "loongarch64", "mips", "mips64", "mips64el", "mipsel", "noarch", "ppc", "ppc64", "ppc64iseries", "ppc64le", "ppc64p7", "ppc64pseries", "riscv128", "riscv32", "riscv64", "s390", "s390x", "sh3", "sh4", "sh4a", "sparc", "sparc64", "sparc64v", "sparcv8", "sparcv9", "sparcv9v", "x86_64". Add "--help" for more information about the arguments.
m-blaha commented 11 months ago

Yes. I forgot to mention it in PR https://github.com/rpm-software-management/dnf5/pull/856