Skip to content

test: fix command tests that may hang on linux due to sudo#10107

Open
paulbalandan wants to merge 1 commit intocodeigniter4:developfrom
paulbalandan:fix-linux-hang-on-sudo
Open

test: fix command tests that may hang on linux due to sudo#10107
paulbalandan wants to merge 1 commit intocodeigniter4:developfrom
paulbalandan:fix-linux-hang-on-sudo

Conversation

@paulbalandan
Copy link
Copy Markdown
Member

Description
Ref: #10093 (comment)

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@github-actions github-actions bot added the testing Pull requests that changes tests only label Apr 12, 2026
Copy link
Copy Markdown
Contributor

@neznaika0 neznaika0 left a comment

Choose a reason for hiding this comment

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

Fixed. I just don't know if the test should be run for me (Arch)?

@paulbalandan
Copy link
Copy Markdown
Member Author

does running chattr without the sudo works for you? as i recall, it fails on GHA because of privilege issues, that's why I added that.

@neznaika0
Copy link
Copy Markdown
Contributor

No, this command never seems to work without root.

@paulbalandan
Copy link
Copy Markdown
Member Author

how about chmod($path, 0444);?

@neznaika0
Copy link
Copy Markdown
Contributor

No. Play with chmod(dirname($path), 0555); chmod(dirname($path), 0755) work. 👍

@paulbalandan paulbalandan force-pushed the fix-linux-hang-on-sudo branch from ef8ff67 to 83fe753 Compare April 12, 2026 20:26
@paulbalandan
Copy link
Copy Markdown
Member Author

Thanks! That worked too on mac.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Pull requests that changes tests only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants