-
Notifications
You must be signed in to change notification settings - Fork 272
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change max_port_file_count to warn rather than failing to calculate an ABI hash. #1104
Change max_port_file_count to warn rather than failing to calculate an ABI hash. #1104
Conversation
The reason for this max file count was to avoid appearing to "hang" if users accidentally cloned source repos into the port directory. We will recurse into .git directories and attempt to hash tens or hundreds of thousands of files. Would it be more appropriate to raise this limit to 1,000 instead of removing it entirely? |
I think we should address this with a warning rather than just refusing to calculate the correct cache information. I'll restore the limit but make it trigger a warning rather than refusing to generate a cache entry. |
HashPortManyFiles, | ||
(msg::package_name, msg::count), | ||
"", | ||
"The {package_name} contains a {count} files. Hashing these contents may take a long time when determining the ABI " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"The {package_name} contains a {count} files. Hashing these contents may take a long time when determining the ABI " | |
"The {package_name} contains {count} files. Hashing these contents may take a long time when determining the ABI " |
(msg::package_name, msg::count), | ||
"", | ||
"The {package_name} contains a {count} files. Hashing these contents may take a long time when determining the ABI " | ||
"hash for binary caching. You should probably consider reducing the number of files. Common causes of this are " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"hash for binary caching. You should probably consider reducing the number of files. Common causes of this are " | |
"hash for binary caching. Consider reducing the number of files. Common causes of this are " |
"", | ||
"The {package_name} contains a {count} files. Hashing these contents may take a long time when determining the ABI " | ||
"hash for binary caching. You should probably consider reducing the number of files. Common causes of this are " | ||
"accidentally checking out source or build trees into a port's directory.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"accidentally checking out source or build trees into a port's directory.") | |
"accidentally checking out source or build files into a port's directory.") |
I missed CR comments in microsoft#1104
I missed CR comments in #1104
Resolves microsoft/vcpkg#31975 .