Skip to content

Conversation

AlfredWJ
Copy link

Busybox-bash does not support "[]", fix it by "cut"

issue: #466

@mergify mergify bot added the Fedora label Jun 25, 2025
Busybox-bash does not support "[]", fix it by "cut"

Signed-off-by: wangjun <wangjun445@h-partners.com>
@jamacku jamacku requested a review from Copilot June 25, 2025 10:11
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

A change to the functions file to avoid using unsupported array indices in Busybox-bash by reading process stat values using cut instead of array slicing.

  • Replace array indexing on process stat data with cut-based extraction.
  • Adjust both pid and self stat retrieval accordingly.

AlfredWJ and others added 2 commits June 26, 2025 10:30
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@AlfredWJ
Copy link
Author

AlfredWJ commented Jul 2, 2025

image
is it stuck?

@jamacku jamacku requested a review from Copilot July 2, 2025 06:04
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@AlfredWJ AlfredWJ closed this Aug 12, 2025
@AlfredWJ AlfredWJ reopened this Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant