Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 328889 - trying to run "ln" during core.resources tests on Windows 7 slave at eclipse.org
Summary: trying to run "ln" during core.resources tests on Windows 7 slave at eclipse.org
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Resources (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.7 M4   Edit
Assignee: Malgorzata Janczarska CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 295393 335864
  Show dependency tree
 
Reported: 2010-10-27 16:38 EDT by Kim Moir CLA
Modified: 2011-01-31 11:16 EST (History)
1 user (show)

See Also:
Szymon.Brandys: review+

Attachments
Fix "ln" problem (1.38 KB, patch)
2010-11-02 09:56 EDT, Malgorzata Janczarska CLA 		Szymon.Brandys: iplog+
Details | Diff
Fix volumes problem (8.17 KB, patch)
2010-11-02 09:58 EDT, Malgorzata Janczarska CLA 		Szymon.Brandys: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.