Uploaded image for project: 'RHEL'
  1. RHEL
  2. RHEL-67426

regression in rust-toolset 1.79: %cargo_install macro no longer accepts arguments

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Undefined Undefined
    • None
    • rhel-8.10, CentOS Stream 8, CentOS Stream 9, rhel-9.5
    • rust-toolset
    • Yes
    • None
    • rhel-sst-pt-llvm-rust-go
    • ssg_platform_tools
    • None
    • False
    • Hide

      None

      Show
      None
    • None
    • None
    • None
    • None
    • All
    • None

      What were you trying to do that didn't work?

      Packages in EPEL 8 and 9 that use the `%cargo_install` macro from rust-toolset and pass arguments to this macro fail to build on top of RHEL 8.10, RHEL 9.5, and in CentOS Stream 9.

      The introduction of a subshell inside the `%cargo_install` macro implementation without explicitly passing through arguments causes a parsing failure for any passed arguments instead of them being passed through to `cargo`.

      What is the impact of this issue to you?

      Packages that need to pass arguments to `%cargo_install` fail to build. Personally, this affects many of my Rust packages in EPEL 8 (for example, rust-ripgrep and rust-tokei).

      Please provide the package NVR for which the bug is seen:

      rust-toolset-1.79.0-2.module+el8.10.0+22210+e7efad26 (for RHEL 8.10 / EPEL 8)

      How reproducible is this bug?:

      Always.

      Steps to reproduce

      1. fedpkg clone rust-tokei
      2. cd rust-tokei
      3. fedpkg switch-branch epel8
      4. fedpkg mockbuild

      Expected results

      Arguments passed to `%cargo_install` are passed through to `cargo`.

      Actual results

      Arguments passed to `%cargo_install` fail the build due to a syntax error.

       

              jistone@redhat.com Josh Stone
              decathorpe_gmail Fabio Valentini
              Josh Stone Josh Stone
              Jesus Checa Hidalgo Jesus Checa Hidalgo
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Created:
                Updated: