Skip to content

clippy: make the wrapper work if build is not the 1st arg#170

Merged
kdudka merged 1 commit intocsutils:mainfrom
kdudka:clippy-locked-build
May 29, 2024
Merged

clippy: make the wrapper work if build is not the 1st arg#170
kdudka merged 1 commit intocsutils:mainfrom
kdudka:clippy-locked-build

Conversation

@kdudka
Copy link
Member

@kdudka kdudka commented May 28, 2024

... so that we can successfully scan the Rust code in librsvg2-2.50.7-1.el9_0 where cargo is invoked like this:

cd ./librsvg && \
    PKG_CONFIG_ALLOW_CROSS=1 \
    PKG_CONFIG='/usr/bin/x86_64-redhat-linux-gnu-pkg-config' \
    CARGO_TARGET_DIR=/builddir/build/BUILD/librsvg-2.50.7/target \
    cargo --locked build --verbose  --release \
    && cd /builddir/build/BUILD/librsvg-2.50.7 && /bin/sh ./libtool [...]

Related: https://issues.redhat.com/browse/OSH-30

@kdudka kdudka requested a review from lzaoral May 28, 2024 12:02
@kdudka kdudka self-assigned this May 28, 2024
kdudka added a commit to kdudka/csmock that referenced this pull request May 28, 2024
... so that we can successfully scan the Rust code
in `librsvg2-2.50.7-1.el9_0` where `cargo` is invoked
like this:
```
cd ./librsvg && \
    PKG_CONFIG_ALLOW_CROSS=1 \
    PKG_CONFIG='/usr/bin/x86_64-redhat-linux-gnu-pkg-config' \
    CARGO_TARGET_DIR=/builddir/build/BUILD/librsvg-2.50.7/target \
    cargo --locked build --verbose  --release \
    && cd /builddir/build/BUILD/librsvg-2.50.7 && /bin/sh ./libtool [...]
```

Related: https://issues.redhat.com/browse/OSH-30
Closes: csutils#170
@kdudka kdudka force-pushed the clippy-locked-build branch from 75405aa to bcca9da Compare May 28, 2024 12:02
Copy link
Member

@lzaoral lzaoral left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you, @kdudka!

... so that we can successfully scan the Rust code
in `librsvg2-2.50.7-1.el9_0` where `cargo` is invoked
like this:
```
cd ./librsvg && \
    PKG_CONFIG_ALLOW_CROSS=1 \
    PKG_CONFIG='/usr/bin/x86_64-redhat-linux-gnu-pkg-config' \
    CARGO_TARGET_DIR=/builddir/build/BUILD/librsvg-2.50.7/target \
    cargo --locked build --verbose  --release \
    && cd /builddir/build/BUILD/librsvg-2.50.7 && /bin/sh ./libtool [...]
```

Related: https://issues.redhat.com/browse/OSH-30
Closes: csutils#170
@kdudka kdudka force-pushed the clippy-locked-build branch from bcca9da to b037156 Compare May 29, 2024 11:25
@kdudka kdudka requested a review from lzaoral May 29, 2024 11:27
@kdudka
Copy link
Member Author

kdudka commented May 29, 2024

@lzaoral Thanks for review!

@kdudka kdudka merged commit b037156 into csutils:main May 29, 2024
@kdudka kdudka deleted the clippy-locked-build branch May 29, 2024 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants