Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 328889

Summary: trying to run "ln" during core.resources tests on Windows 7 slave at eclipse.org
Product: [Eclipse Project] Platform Reporter: Kim Moir <kim.moir>
Component: ResourcesAssignee: Malgorzata Janczarska <malgorzata.tomczyk>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: Szymon.Brandys
Version: 3.7Flags: Szymon.Brandys: review+
Target Milestone: 3.7 M4   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Bug Depends on:    
Bug Blocks: 295393, 335864    
Attachments:
Description Flags
Fix "ln" problem
Szymon.Brandys: iplog+
Fix volumes problem Szymon.Brandys: iplog+

Description Kim Moir CLA 2010-10-27 16:38:54 EDT 
It looks like the core.resources tests are running "ln" on an unsupported platform when I run them on the Windows 7 machine at the foundation.

https://hudson.eclipse.org/hudson/view/Eclipse%20and%20Equinox/job/eclipse-JUnit/lastCompletedBuild/testReport/org.eclipse.core.tests.internal.localstore/SymlinkResourceTest/testBug232426/

junit.framework.AssertionFailedError: createSymLink: java.io.IOException: Cannot run program "ln" (in directory "C:\Users\HUDSONBUILD\hudson\workspace\eclipse-JUnit\2010-10-26_11-53-11\eclipse-testing\test-eclipse\eclipse\core_resources_sniff_folder\Project0\a"): CreateProcess error=2, The system cannot find the file specified
	at org.eclipse.core.tests.harness.CoreTest.fail(CoreTest.java:63)

Another thing, it looks like the testCopyAcrossVolumes test fails when both D:\temp and c:\temp don't exist. I talked to John about this and he suggested that the test should just not run if another volume doesn't exist. (I opened bug 328851 asking the foundation to create this temp directory)

https://hudson.eclipse.org/hudson/view/Eclipse%20and%20Equinox/job/eclipse-JUnit/lastCompletedBuild/testReport/org.eclipse.core.tests.filesystem/FileStoreTest/testCopyAcrossVolumes/
Comment 1 Kim Moir CLA 2010-10-28 17:00:50 EDT 
The foundation webmaster has created e:\temp on the build machine, so could you update the tests to also check this directory when running the testCopyAcrossVolumes tests?
Comment 2 Malgorzata Janczarska CLA 2010-11-02 09:56:53 EDT 
Created attachment 182204 [details]
Fix "ln" problem

Uses "mklink" on Windows7
Comment 3 Malgorzata Janczarska CLA 2010-11-02 09:58:04 EDT 
Created attachment 182205 [details]
Fix volumes problem

Searches for volumes between c: and z: to copy between volumes
Comment 4 Malgorzata Janczarska CLA 2010-11-02 10:15:05 EDT 
Kim, those are two separate errors, so I made two patches.
1. Check patch_328889_LN.txt on Wondows7 in should fix the "ln" problem.
2. patch_328889_Volumes.txt should fix problem if c:\temp of d:\temp could not be created. If there are any volumes between C: and Z: where \temp dir can be created and is not readonly the test will use those two volumes. If there are none the test is not performed. This should work as well for e:/temp directory created on the build machine you've mentioned.
Comment 5 Szymon Brandys CLA 2010-11-04 07:17:31 EDT 
Both patches released to HEAD. Thanks Gosia.