Skip to content

clar: define CLAR_MAX_PATH even w/o PATH_MAX#132

Merged
ethomson merged 1 commit intoclar-test:mainfrom
pinotree:path_max
Dec 30, 2025
Merged

clar: define CLAR_MAX_PATH even w/o PATH_MAX#132
ethomson merged 1 commit intoclar-test:mainfrom
pinotree:path_max

Conversation

@pinotree
Copy link
Copy Markdown
Contributor

PATH_MAX is an optional constant in POSIX, and at least on GNU/Hurd it is not provided on purpose.

Define CLAR_MAX_PATH to a static value in case PATH_MAX is not available, using the same value used for the "Windows with long paths" case. In the end, 4096 should not be a problem for a test framework.

PATH_MAX is an optional constant in POSIX, and at least on GNU/Hurd
it is not provided on purpose.

Define CLAR_MAX_PATH to a static value in case PATH_MAX is not available,
using the same value used for the "Windows with long paths" case. In the
end, 4096 should not be a problem for a test framework.
@ethomson
Copy link
Copy Markdown
Member

Thanks! Makes sense.

@ethomson ethomson merged commit da87d48 into clar-test:main Dec 30, 2025
7 checks passed
@pinotree pinotree deleted the path_max branch December 31, 2025 08:01
@pinotree
Copy link
Copy Markdown
Contributor Author

@ethomson I see your update in libgit2/libgit2#7173 -- thanks!

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