Skip to content

Symlinks failing on windows bots #45981

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

Closed
dcharkes opened this issue May 11, 2021 · 1 comment
Closed

Symlinks failing on windows bots #45981

dcharkes opened this issue May 11, 2021 · 1 comment
Labels
area-infrastructure Use area-infrastructure for SDK infrastructure issues, like continuous integration bot changes. gardening

Comments

@dcharkes
Copy link
Contributor

There are new test failures on Don't report deprecated hints from deprecated constructors....[vm/concurrency] Track on [Thread] whether VM can deopt a mutator.

The tests

co19/LibTest/io/Link/resolveSymbolicLinksSync_A01_t01 RuntimeError (expected Pass)
co19/LibTest/io/Link/resolveSymbolicLinksSync_A01_t01 RuntimeError (expected Pass)
co19/LibTest/io/Link/resolveSymbolicLinks_A01_t01 RuntimeError (expected Pass)
co19/LibTest/io/Link/resolveSymbolicLinks_A01_t01 RuntimeError (expected Pass)

are failing on all windows bots with the following error:

set DART_CONFIGURATION=ReleaseX64 & set DART_SUPPRESS_WER=1 & set DART_CRASHPAD_HANDLER=C:\b\s\w\ir\out\ReleaseX64\crashpad_handler.exe & out\ReleaseX64\dart.exe --enable_asserts --no-sound-null-safety --ignore-unrecognized-flags --packages=C:\b\s\w\ir\.packages C:\b\s\w\ir\tests\co19\src\LibTest\io\Link\resolveSymbolicLinksSync_A01_t01.dart

exit code:
255

stderr:
Unhandled exception:
FileSystemException: Cannot resolve symbolic links, path = 'C:\b\s\w\iton6qvt\co19-resolveSymbolicLinksSync_A01_t01_7b6bedce\co19-resolveSymbolicLinksSync_A01_t01_18db5e28\resolveSymbolicLinksSync_A01_t01_3_8269.lnk' (OS Error: Access is denied.

, errno = 5)
#0      FileSystemEntity._throwIfError (dart:io/file_system_entity.dart:818:7)
#1      FileSystemEntity.resolveSymbolicLinksSync (dart:io/file_system_entity.dart:348:5)
#2      _main (file:///C:/b/s/w/ir/tests/co19/src/LibTest/io/Link/resolveSymbolicLinksSync_A01_t01.dart:66:34)
#3      inSandbox (file:///C:/b/s/w/ir/tests/co19/src/LibTest/io/file_utils.dart:24:22)
#4      main (file:///C:/b/s/w/ir/tests/co19/src/LibTest/io/Link/resolveSymbolicLinksSync_A01_t01.dart:32:9)
#5      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:283:19)
#6      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)

--- Re-run this test:
python3 tools/test.py -n dartk-weak-asserts-win-release-x64 co19/LibTest/io/Link/resolveSymbolicLinksSync_A01_t01

One of the logs.

This does not look like it is related to any of the mentioned commits.

@athomas @whesse did something change with the Windows bot setup related to symlinks?

@dcharkes dcharkes added gardening area-infrastructure Use area-infrastructure for SDK infrastructure issues, like continuous integration bot changes. labels May 11, 2021
@whesse
Copy link
Contributor

whesse commented May 11, 2021

The Windows bots updated to a more recent version of Windows 10. This may have changed the behavior, or it may have exposed some problem which was potentially there before also.

The actual failure seems to be when dereferencing a symbolic link that starts with "..". This is "link3" in the tests, and links between two sibling directories inside a temporary directory.

There are bug reports of many existing failures and differences in behavior between Windows and other OSs in the dart:io file system libraries, adding this report to those: #29096

copybara-service bot pushed a commit that referenced this issue Feb 21, 2024
… Windows.

Also modified `File::GetType` on Windows to correctly report the type of broken links as notFound.

Windows fixes:
co19/LibTest/io/Link/createSync_A04_t08
co19/LibTest/io/Link/createSync_A04_t10
co19/LibTest/io/Link/createSync_A04_t12
co19/LibTest/io/Link/createSync_A04_t14
co19/LibTest/io/Link/createSync_A04_t16
co19/LibTest/io/Link/createSync_A06_t01
co19/LibTest/io/Link/createSync_A06_t03
co19/LibTest/io/Link/createSync_A06_t06
co19/LibTest/io/Link/createSync_A06_t08
co19/LibTest/io/Link/create_A04_t08
co19/LibTest/io/Link/create_A04_t10
co19/LibTest/io/Link/create_A04_t12
co19/LibTest/io/Link/create_A04_t14
co19/LibTest/io/Link/create_A04_t16
co19/LibTest/io/Link/create_A06_t01
co19/LibTest/io/Link/create_A06_t03
co19/LibTest/io/Link/create_A06_t06
co19/LibTest/io/Link/create_A06_t08
co19/LibTest/io/Link/resolveSymbolicLinksSync_A01_t01
co19/LibTest/io/Link/resolveSymbolicLinks_A01_t01


Bug:#53848
Bug:#45981
Change-Id: I3d156f38540089d8adb12dbb79d0477330d9eb07
Tested: updated unit tests plus fixes existing tests
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/335940
Commit-Queue: Brian Quinlan <[email protected]>
Reviewed-by: Siva Annamalai <[email protected]>
copybara-service bot pushed a commit that referenced this issue Feb 22, 2024
The new library dependency added to File io on Windows should be
added to the libdart_builtin targets, not the downstream
executables, so that Flutter builds that use libdart_builtin
will inherit the dependency.

Follow-up to https://dart-review.googlesource.com/c/sdk/+/335940

TEST=monorepo_build,dart_ci,flutter_engine_ci

Bug: #45981
Bug: #53848
Change-Id: I90407a0d702683cadb4f2d2c57bb6c598b6e052d
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/353680
Commit-Queue: William Hesse <[email protected]>
Reviewed-by: Slava Egorov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Use area-infrastructure for SDK infrastructure issues, like continuous integration bot changes. gardening
Projects
None yet
Development

No branches or pull requests

3 participants