feat(psmisc): package psmisc version 23.7 - #500
Conversation
📝 WalkthroughWalkthroughAdds a Changespsmisc package
Estimated code review effort: 3 (Moderate) | ~20 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/psmisc/build.ncl`:
- Around line 49-54: Remove the entire source_provenance block from the package
configuration, including its category, host, owner, and repo fields; do not
replace Gitlab with another provider.
In `@packages/psmisc/build.sh`:
- Around line 9-10: Update the autotools build flow in build.sh to set ARFLAGS
to Drc, include --enable-deterministic-archives in the ./configure invocation,
and remove generated *.la files after installation.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 17bf3796-90d9-4841-af6e-dda7ef14618f
📒 Files selected for processing (2)
packages/psmisc/build.nclpackages/psmisc/build.sh
| source_provenance = { | ||
| category = 'Gitlab, | ||
| host = "gitlab.com", | ||
| owner = "psmisc", | ||
| repo = "psmisc", | ||
| }, |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🔴 Critical | ⚡ Quick win
Use a supported source_provenance category.
Gitlab isn’t a valid source_provenance.category here, so this package will fail schema validation. Remove the block until GitLab provenance is supported; don’t remap it to another provider.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@packages/psmisc/build.ncl` around lines 49 - 54, Remove the entire
source_provenance block from the package configuration, including its category,
host, owner, and repo fields; do not replace Gitlab with another provider.
| export CFLAGS="$MARCH -O2 -pipe -gno-record-gcc-switches -ffile-prefix-map=$(pwd)=/builddir" | ||
| export CXXFLAGS="$CFLAGS" |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
# Map the package files and inspect the build script
git ls-files packages/psmisc
echo '--- build.sh ---'
cat -n packages/psmisc/build.sh
echo '--- search for deterministic archive settings in nearby packages ---'
rg -n --hidden --glob 'packages/*/build.sh' "ARFLAGS|enable-deterministic-archives|\\.la' -delete|\\.la\" -delete|--enable-deterministic-archives" packagesRepository: gominimal/pkgs
Length of output: 3366
🏁 Script executed:
#!/bin/bash
set -euo pipefail
# Read only the relevant build script in smaller slices if needed
wc -l packages/psmisc/build.sh
sed -n '1,120p' packages/psmisc/build.shRepository: gominimal/pkgs
Length of output: 964
Add deterministic archive handling for autotools. Set ARFLAGS='Drc', pass --enable-deterministic-archives to ./configure, and remove *.la files after install.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@packages/psmisc/build.sh` around lines 9 - 10, Update the autotools build
flow in build.sh to set ARFLAGS to Drc, include --enable-deterministic-archives
in the ./configure invocation, and remove generated *.la files after
installation.
Source: Coding guidelines
Because i want
killalllolSummary by CodeRabbit
New Features
psmiscpackage, providing system process-management utilities such askillall.psmisc23.7 release.Tests