Skip to content

Conversation

@kikairoya
Copy link
Contributor

Cygwin doesn't support ulimit -f because Windows doesn't provide such functionality.

Cygwin doesn't support `ulimit -f` because Windows doesn't provide such functionality.
@llvmbot
Copy link
Member

llvmbot commented Oct 31, 2025

@llvm/pr-subscribers-testing-tools

Author: Tomohiro Kashiwada (kikairoya)

Changes

Cygwin doesn't support ulimit -f because Windows doesn't provide such functionality.


Full diff: https://github.com/llvm/llvm-project/pull/165849.diff

1 Files Affected:

  • (modified) llvm/utils/lit/tests/shtest-ulimit-nondarwin.py (+1-1)
diff --git a/llvm/utils/lit/tests/shtest-ulimit-nondarwin.py b/llvm/utils/lit/tests/shtest-ulimit-nondarwin.py
index 286fd3d7e173e..d81cde0159792 100644
--- a/llvm/utils/lit/tests/shtest-ulimit-nondarwin.py
+++ b/llvm/utils/lit/tests/shtest-ulimit-nondarwin.py
@@ -2,7 +2,7 @@
 
 # ulimit does not work on non-POSIX platforms.
 # These tests are specific to options that Darwin does not support.
-# UNSUPPORTED: system-windows, system-darwin, system-aix, system-solaris
+# UNSUPPORTED: system-windows, system-cygwin, system-darwin, system-aix, system-solaris
 
 # RUN: not %{lit} -a -v %{inputs}/shtest-ulimit-nondarwin | FileCheck %s
 

@kikairoya
Copy link
Contributor Author

Similar to #164604.
This addresses the new test added in 160058f.
cc: @jeremyd2019 @mstorsjo

Copy link
Member

@mstorsjo mstorsjo left a comment

Choose a reason for hiding this comment

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

LGTM

@mstorsjo mstorsjo merged commit dec6e73 into llvm:main Nov 4, 2025
13 checks passed
@kikairoya kikairoya deleted the cygwin-ulimit-f branch November 4, 2025 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants