-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[mono][ios] System.Formats.Tar.Tests are failing with System.ArgumentException #88049
Comments
Tagging subscribers to 'os-tvos': @steveisok, @akoeplinger Issue DetailsBuild InformationBuild: https://dev.azure.com/dnceng-public/public/_build/results?buildId=320749&view=results Error MessageFill the error message using step by step known issues guidance. {
"ErrorMessage": "System.ArgumentException : The field 'Name' exceeds the maximum allowed length for this format",
"ErrorPattern": "",
"BuildRetry": false,
"ExcludeConsoleLog": false
}
|
@carlossanlop It seems that the regression was introduced in #87534. The issue appears to be specific to the ios and tvos simulators. |
This fails only on apple mobile simulators that run Mono Interpreter. My assumption is that the /cc: @BrzVlad const ushort limit = 100;
ReadOnlySpan<char> name = _name;
int encodedLength = GetUtf8TextLength(name);
if (encodedLength > FieldLengths.Name)
{
if (_format is TarEntryFormat.V7)
{
throw new ArgumentException(SR.Format(SR.TarEntryFieldExceedsMaxLength, nameof(TarEntry.Name)), ArgNameEntry);
}
int utf16NameTruncatedLength = GetUtf16TruncatedTextLength(name, FieldLengths.Name);
name = name.Slice(0, utf16NameTruncatedLength);
} |
This issue is also occurring on iOS/tvOS devices. For instance in: https://dev.azure.com/dnceng-public/public/_build/results?buildId=357141&view=results. The length of fields exceeds the values set in:
@carlossanlop Should we update the fields length to match the following examples? runtime/src/libraries/System.Formats.Tar/tests/TarFile/TarFile.ExtractToDirectory.Base.cs Line 11 in 7b2716b
|
Hi - The Path length used for V7 should stay as is, it's what the spec indicates and it should be respected. The test could be modified so that the temporary path directory is created with a much shorter path. Would that work? |
Yes, definitely. |
@kotlarmilos @steveisok the folder used to store files needs to have a shorter path for the test to work (the failure is expected for this tar format). Can you please change it to a shorter directory for this platform? |
Build Information
Build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=320749&view=results
Build error leg or test failing: iossimulator-x64 Release AllSubsets_Mono
Pull request: #88042
Error Message
Fill the error message using step by step known issues guidance.
Known issue validation
Build: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=320749
Result validation: ✅ Known issue matched with the provided build.
Report
Summary
The text was updated successfully, but these errors were encountered: