You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Kevin Zhou (JIRA)" <ji...@apache.org> on 2008/11/03 10:20:44 UTC

[jira] Created: (HARMONY-6006) [classlib] [luni] Cannot process platform paths using backslash

[classlib] [luni] Cannot process platform paths using backslash
---------------------------------------------------------------

                 Key: HARMONY-6006
                 URL: https://issues.apache.org/jira/browse/HARMONY-6006
             Project: Harmony
          Issue Type: Bug
          Components: Classlib
    Affects Versions: 5.0M7
            Reporter: Kevin Zhou
             Fix For: 5.0M8


Defect Reproduce Steps:
1. Make folders like /ffs0/niagara/bin
2. Make a symbolic prefix link "/niagara" using the command:
ln -s /ffs0/niagara /niagara
3. Run CanonicalPathTest [1] using the command:
java CanonicalPathTest /niagara /ffs0/niagara /niagara/bin /ffs0/niagara/bin
4. Defect Result shown as [2], the correct result should  be [3].

[1] CanonicalPathTest.java
import java.io.File;
import java.io.IOException;
public class CanonicalPathTest {
public static void main(String[] args) throws IOException {
File file = null;
for (int index = 0; index < args.length; index++) { file = new File(args[index]); System.out.println(args[index] + "->" + file.getCanonicalPath()); }
}
}

[2] Defect Result:
/niagara->/ffs0/niagara
/ffs0/niagara->/ffs0/niagara
/niagara/bin->/niagara/bin   // it's wrong
/ffs0/niagara/bin->/ffs0/niagara/bin

[3] Correct Result:
/niagara->/ffs0/niagara
/ffs0/niagara->/ffs0/niagara
/niagara/bin->/ffs0/niagara/bin
/ffs0/niagara/bin->/ffs0/niagara/bi

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-6006) [classlib] [luni] Cannot process platform paths using backslash

Posted by "Kevin Zhou (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6006?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kevin Zhou updated HARMONY-6006:
--------------------------------

    Attachment: HARMONY-6006.diff

Add a test case for your convenience. Would you please help to try it?

> [classlib] [luni] Cannot process platform paths using backslash
> ---------------------------------------------------------------
>
>                 Key: HARMONY-6006
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6006
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M7
>            Reporter: Kevin Zhou
>            Assignee: Sean Qiu
>             Fix For: 5.0M8
>
>         Attachments: HARMONY-6006.diff
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Defect Reproduce Steps:
> 1. Make folders like /ffs0/niagara/bin
> 2. Make a symbolic prefix link "/niagara" using the command:
> ln -s /ffs0/niagara /niagara
> 3. Run CanonicalPathTest [1] using the command:
> java CanonicalPathTest /niagara /ffs0/niagara /niagara/bin /ffs0/niagara/bin
> 4. Defect Result shown as [2], the correct result should  be [3].
> [1] CanonicalPathTest.java
> import java.io.File;
> import java.io.IOException;
> public class CanonicalPathTest {
> public static void main(String[] args) throws IOException {
> File file = null;
> for (int index = 0; index < args.length; index++) { file = new File(args[index]); System.out.println(args[index] + "->" + file.getCanonicalPath()); }
> }
> }
> [2] Defect Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/niagara/bin   // it's wrong
> /ffs0/niagara/bin->/ffs0/niagara/bin
> [3] Correct Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/ffs0/niagara/bin
> /ffs0/niagara/bin->/ffs0/niagara/bi

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-6006) [classlib] [luni] Cannot process platform paths using backslash

Posted by "Kevin Zhou (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6006?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kevin Zhou updated HARMONY-6006:
--------------------------------

    Attachment:     (was: HY-1003.diff)

> [classlib] [luni] Cannot process platform paths using backslash
> ---------------------------------------------------------------
>
>                 Key: HARMONY-6006
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6006
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M7
>            Reporter: Kevin Zhou
>            Assignee: Sean Qiu
>             Fix For: 5.0M8
>
>         Attachments: HARMONY-6006.diff
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Defect Reproduce Steps:
> 1. Make folders like /ffs0/niagara/bin
> 2. Make a symbolic prefix link "/niagara" using the command:
> ln -s /ffs0/niagara /niagara
> 3. Run CanonicalPathTest [1] using the command:
> java CanonicalPathTest /niagara /ffs0/niagara /niagara/bin /ffs0/niagara/bin
> 4. Defect Result shown as [2], the correct result should  be [3].
> [1] CanonicalPathTest.java
> import java.io.File;
> import java.io.IOException;
> public class CanonicalPathTest {
> public static void main(String[] args) throws IOException {
> File file = null;
> for (int index = 0; index < args.length; index++) { file = new File(args[index]); System.out.println(args[index] + "->" + file.getCanonicalPath()); }
> }
> }
> [2] Defect Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/niagara/bin   // it's wrong
> /ffs0/niagara/bin->/ffs0/niagara/bin
> [3] Correct Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/ffs0/niagara/bin
> /ffs0/niagara/bin->/ffs0/niagara/bi

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-6006) [classlib] [luni] Cannot process platform paths using backslash

Posted by "Kevin Zhou (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6006?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kevin Zhou updated HARMONY-6006:
--------------------------------

    Attachment:     (was: HARMONY-6006.diff)

> [classlib] [luni] Cannot process platform paths using backslash
> ---------------------------------------------------------------
>
>                 Key: HARMONY-6006
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6006
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M7
>            Reporter: Kevin Zhou
>            Assignee: Sean Qiu
>             Fix For: 5.0M8
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Defect Reproduce Steps:
> 1. Make folders like /ffs0/niagara/bin
> 2. Make a symbolic prefix link "/niagara" using the command:
> ln -s /ffs0/niagara /niagara
> 3. Run CanonicalPathTest [1] using the command:
> java CanonicalPathTest /niagara /ffs0/niagara /niagara/bin /ffs0/niagara/bin
> 4. Defect Result shown as [2], the correct result should  be [3].
> [1] CanonicalPathTest.java
> import java.io.File;
> import java.io.IOException;
> public class CanonicalPathTest {
> public static void main(String[] args) throws IOException {
> File file = null;
> for (int index = 0; index < args.length; index++) { file = new File(args[index]); System.out.println(args[index] + "->" + file.getCanonicalPath()); }
> }
> }
> [2] Defect Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/niagara/bin   // it's wrong
> /ffs0/niagara/bin->/ffs0/niagara/bin
> [3] Correct Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/ffs0/niagara/bin
> /ffs0/niagara/bin->/ffs0/niagara/bi

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (HARMONY-6006) [classlib] [luni] Cannot process platform paths using backslash

Posted by "Sean Qiu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6006?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sean Qiu reassigned HARMONY-6006:
---------------------------------

    Assignee: Sean Qiu

> [classlib] [luni] Cannot process platform paths using backslash
> ---------------------------------------------------------------
>
>                 Key: HARMONY-6006
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6006
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M7
>            Reporter: Kevin Zhou
>            Assignee: Sean Qiu
>             Fix For: 5.0M8
>
>         Attachments: HY-1003.diff
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Defect Reproduce Steps:
> 1. Make folders like /ffs0/niagara/bin
> 2. Make a symbolic prefix link "/niagara" using the command:
> ln -s /ffs0/niagara /niagara
> 3. Run CanonicalPathTest [1] using the command:
> java CanonicalPathTest /niagara /ffs0/niagara /niagara/bin /ffs0/niagara/bin
> 4. Defect Result shown as [2], the correct result should  be [3].
> [1] CanonicalPathTest.java
> import java.io.File;
> import java.io.IOException;
> public class CanonicalPathTest {
> public static void main(String[] args) throws IOException {
> File file = null;
> for (int index = 0; index < args.length; index++) { file = new File(args[index]); System.out.println(args[index] + "->" + file.getCanonicalPath()); }
> }
> }
> [2] Defect Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/niagara/bin   // it's wrong
> /ffs0/niagara/bin->/ffs0/niagara/bin
> [3] Correct Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/ffs0/niagara/bin
> /ffs0/niagara/bin->/ffs0/niagara/bi

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-6006) [classlib] [luni] Cannot process platform paths using backslash

Posted by "Kevin Zhou (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6006?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kevin Zhou updated HARMONY-6006:
--------------------------------

    Attachment: HY-1003.diff

Would you please help to try it?

> [classlib] [luni] Cannot process platform paths using backslash
> ---------------------------------------------------------------
>
>                 Key: HARMONY-6006
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6006
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M7
>            Reporter: Kevin Zhou
>             Fix For: 5.0M8
>
>         Attachments: HY-1003.diff
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Defect Reproduce Steps:
> 1. Make folders like /ffs0/niagara/bin
> 2. Make a symbolic prefix link "/niagara" using the command:
> ln -s /ffs0/niagara /niagara
> 3. Run CanonicalPathTest [1] using the command:
> java CanonicalPathTest /niagara /ffs0/niagara /niagara/bin /ffs0/niagara/bin
> 4. Defect Result shown as [2], the correct result should  be [3].
> [1] CanonicalPathTest.java
> import java.io.File;
> import java.io.IOException;
> public class CanonicalPathTest {
> public static void main(String[] args) throws IOException {
> File file = null;
> for (int index = 0; index < args.length; index++) { file = new File(args[index]); System.out.println(args[index] + "->" + file.getCanonicalPath()); }
> }
> }
> [2] Defect Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/niagara/bin   // it's wrong
> /ffs0/niagara/bin->/ffs0/niagara/bin
> [3] Correct Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/ffs0/niagara/bin
> /ffs0/niagara/bin->/ffs0/niagara/bi

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HARMONY-6006) [classlib] [luni] Cannot process platform paths using backslash

Posted by "Sean Qiu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HARMONY-6006?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12644877#action_12644877 ] 

Sean Qiu commented on HARMONY-6006:
-----------------------------------

Thank you, Kevin.
Since we're in the stage of M8, the repository is freezed.
I'll apply it after the M8 is completed.

> [classlib] [luni] Cannot process platform paths using backslash
> ---------------------------------------------------------------
>
>                 Key: HARMONY-6006
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6006
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M7
>            Reporter: Kevin Zhou
>            Assignee: Sean Qiu
>             Fix For: 5.0M8
>
>         Attachments: HARMONY-6006.diff
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Defect Reproduce Steps:
> 1. Make folders like /ffs0/niagara/bin
> 2. Make a symbolic prefix link "/niagara" using the command:
> ln -s /ffs0/niagara /niagara
> 3. Run CanonicalPathTest [1] using the command:
> java CanonicalPathTest /niagara /ffs0/niagara /niagara/bin /ffs0/niagara/bin
> 4. Defect Result shown as [2], the correct result should  be [3].
> [1] CanonicalPathTest.java
> import java.io.File;
> import java.io.IOException;
> public class CanonicalPathTest {
> public static void main(String[] args) throws IOException {
> File file = null;
> for (int index = 0; index < args.length; index++) { file = new File(args[index]); System.out.println(args[index] + "->" + file.getCanonicalPath()); }
> }
> }
> [2] Defect Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/niagara/bin   // it's wrong
> /ffs0/niagara/bin->/ffs0/niagara/bin
> [3] Correct Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/ffs0/niagara/bin
> /ffs0/niagara/bin->/ffs0/niagara/bi

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (HARMONY-6006) [classlib] [luni] Cannot process platform paths using backslash

Posted by "Sean Qiu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HARMONY-6006?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12644674#action_12644674 ] 

Sean Qiu commented on HARMONY-6006:
-----------------------------------

Though the patch seems all right, could you add some tests to indicate the issue?
Thank you in advance for your work.

> [classlib] [luni] Cannot process platform paths using backslash
> ---------------------------------------------------------------
>
>                 Key: HARMONY-6006
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6006
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M7
>            Reporter: Kevin Zhou
>            Assignee: Sean Qiu
>             Fix For: 5.0M8
>
>         Attachments: HARMONY-6006.diff
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Defect Reproduce Steps:
> 1. Make folders like /ffs0/niagara/bin
> 2. Make a symbolic prefix link "/niagara" using the command:
> ln -s /ffs0/niagara /niagara
> 3. Run CanonicalPathTest [1] using the command:
> java CanonicalPathTest /niagara /ffs0/niagara /niagara/bin /ffs0/niagara/bin
> 4. Defect Result shown as [2], the correct result should  be [3].
> [1] CanonicalPathTest.java
> import java.io.File;
> import java.io.IOException;
> public class CanonicalPathTest {
> public static void main(String[] args) throws IOException {
> File file = null;
> for (int index = 0; index < args.length; index++) { file = new File(args[index]); System.out.println(args[index] + "->" + file.getCanonicalPath()); }
> }
> }
> [2] Defect Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/niagara/bin   // it's wrong
> /ffs0/niagara/bin->/ffs0/niagara/bin
> [3] Correct Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/ffs0/niagara/bin
> /ffs0/niagara/bin->/ffs0/niagara/bi

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (HARMONY-6006) [classlib] [luni] Cannot process platform paths using backslash

Posted by "Kevin Zhou (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6006?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kevin Zhou updated HARMONY-6006:
--------------------------------

    Attachment: HARMONY-6006.diff

> [classlib] [luni] Cannot process platform paths using backslash
> ---------------------------------------------------------------
>
>                 Key: HARMONY-6006
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6006
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M7
>            Reporter: Kevin Zhou
>            Assignee: Sean Qiu
>             Fix For: 5.0M8
>
>         Attachments: HARMONY-6006.diff
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Defect Reproduce Steps:
> 1. Make folders like /ffs0/niagara/bin
> 2. Make a symbolic prefix link "/niagara" using the command:
> ln -s /ffs0/niagara /niagara
> 3. Run CanonicalPathTest [1] using the command:
> java CanonicalPathTest /niagara /ffs0/niagara /niagara/bin /ffs0/niagara/bin
> 4. Defect Result shown as [2], the correct result should  be [3].
> [1] CanonicalPathTest.java
> import java.io.File;
> import java.io.IOException;
> public class CanonicalPathTest {
> public static void main(String[] args) throws IOException {
> File file = null;
> for (int index = 0; index < args.length; index++) { file = new File(args[index]); System.out.println(args[index] + "->" + file.getCanonicalPath()); }
> }
> }
> [2] Defect Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/niagara/bin   // it's wrong
> /ffs0/niagara/bin->/ffs0/niagara/bin
> [3] Correct Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/ffs0/niagara/bin
> /ffs0/niagara/bin->/ffs0/niagara/bi

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (HARMONY-6006) [classlib] [luni] Cannot process platform paths using backslash

Posted by "Sean Qiu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HARMONY-6006?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sean Qiu resolved HARMONY-6006.
-------------------------------

       Resolution: Fixed
    Fix Version/s:     (was: 5.0M8)
                   5.0M9

Applied at r721071 with minor modification.
Would you please veirfy it whether it works as expected?
Thanks very much for your patch.

> [classlib] [luni] Cannot process platform paths using backslash
> ---------------------------------------------------------------
>
>                 Key: HARMONY-6006
>                 URL: https://issues.apache.org/jira/browse/HARMONY-6006
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M7
>            Reporter: Kevin Zhou
>            Assignee: Sean Qiu
>             Fix For: 5.0M9
>
>         Attachments: HARMONY-6006.diff
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Defect Reproduce Steps:
> 1. Make folders like /ffs0/niagara/bin
> 2. Make a symbolic prefix link "/niagara" using the command:
> ln -s /ffs0/niagara /niagara
> 3. Run CanonicalPathTest [1] using the command:
> java CanonicalPathTest /niagara /ffs0/niagara /niagara/bin /ffs0/niagara/bin
> 4. Defect Result shown as [2], the correct result should  be [3].
> [1] CanonicalPathTest.java
> import java.io.File;
> import java.io.IOException;
> public class CanonicalPathTest {
> public static void main(String[] args) throws IOException {
> File file = null;
> for (int index = 0; index < args.length; index++) { file = new File(args[index]); System.out.println(args[index] + "->" + file.getCanonicalPath()); }
> }
> }
> [2] Defect Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/niagara/bin   // it's wrong
> /ffs0/niagara/bin->/ffs0/niagara/bin
> [3] Correct Result:
> /niagara->/ffs0/niagara
> /ffs0/niagara->/ffs0/niagara
> /niagara/bin->/ffs0/niagara/bin
> /ffs0/niagara/bin->/ffs0/niagara/bi

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.