Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [orion-dev] orion.file.allowedPaths path separator

Hi Gabriel,

Sorry for the slow response - many of the people who know this part of the code have been on holiday. Can you enter a bugzilla for this, and attach the patch in bugzilla please? Bugzilla is how we keep track of contributions, no matter how small. As for the change itself, switching separators seems perfectly reasonable to me. Since : is also a valid character in Unix paths, we would also need an escape mechanism to distinguish separators from internal path characters.

John




Gabriel Petrovay <gabipetrovay@xxxxxxxxx>
Sent by: orion-dev-bounces@xxxxxxxxxxx

07/28/2011 03:38 PM

Please respond to
Orion developer discussions <orion-dev@xxxxxxxxxxx>

To
Orion developer discussions <orion-dev@xxxxxxxxxxx>
cc
Subject
[orion-dev] orion.file.allowedPaths path separator





Hi,

I see that the path separator for the orion.file.allowedPaths option
is "," which can cause problems since the comma is allowed in path. Is
there a reason for it? If not, can this be changed to the OS specific
path separator? If yes, apply the one line patch atttached.


Thanks!

--
MSc Gabriel Petrovay
Mobile: +41(0)787978034
www.28msec.com
_______________________________________________
orion-dev mailing list
orion-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/orion-dev

Attachment: pathSep.diff
Description: Binary data


Back to the top