/admin/cores?action=STATUS returns weird path info on Windows

[Koji Sekiguchi <ko...@r.email.ne.jp>]

I'm checking MultiCore/CoreAdmin stuff in branch-1.3 and noticed
that /admin/cores?action=STATUS returns weird path info on Windows:

<str name="instanceDir">multicore\core0/</str>
<str name="dataDir">multicore\core0/data/</str>

Backslash and slash characters are mixed in path info. Java client
can easily handle these weird path string, but other clients may not.
I think we should use file.separator system property
before releasing 1.3. Thought?

Koji

Re: /admin/cores?action=STATUS returns weird path info on Windows

[Ryan McKinley <ry...@gmail.com>]

On Sep 14, 2008, at 12:56 PM, Yonik Seeley wrote:

> On Sun, Sep 14, 2008 at 6:45 AM, Koji Sekiguchi <ko...@r.email.ne.jp>  
> wrote:
>> I'm checking MultiCore/CoreAdmin stuff in branch-1.3 and noticed
>> that /admin/cores?action=STATUS returns weird path info on Windows:
>>
>> <str name="instanceDir">multicore\core0/</str>
>> <str name="dataDir">multicore\core0/data/</str>
>>
>> Backslash and slash characters are mixed in path info. Java client
>> can easily handle these weird path string, but other clients may not.
>> I think we should use file.separator system property
>> before releasing 1.3. Thought?
>
> I think clients should be able to handle mixed path separators - they
> need to be able to handle both anyway, and seeing mixed forward and
> backward slashes in windows isn't that uncommon.  We should fix it to
> be consistent (I like standardized forward slashes myself), but it
> doesn't feel like a 1.3 blocker.
>

uggg -- i forgot there are people who use that silly backslash!

I suppose you are right we should support it -- some folks will  
probably want to paste in "c:\path\to\my\dir" from the explorer bar.

But I don't think this is a 1.3 blocker...

ryan

Re: /admin/cores?action=STATUS returns weird path info on Windows

[Yonik Seeley <yo...@apache.org>]

On Sun, Sep 14, 2008 at 6:45 AM, Koji Sekiguchi <ko...@r.email.ne.jp> wrote:
> I'm checking MultiCore/CoreAdmin stuff in branch-1.3 and noticed
> that /admin/cores?action=STATUS returns weird path info on Windows:
>
> <str name="instanceDir">multicore\core0/</str>
> <str name="dataDir">multicore\core0/data/</str>
>
> Backslash and slash characters are mixed in path info. Java client
> can easily handle these weird path string, but other clients may not.
> I think we should use file.separator system property
> before releasing 1.3. Thought?

I think clients should be able to handle mixed path separators - they
need to be able to handle both anyway, and seeing mixed forward and
backward slashes in windows isn't that uncommon.  We should fix it to
be consistent (I like standardized forward slashes myself), but it
doesn't feel like a 1.3 blocker.

-Yonik