[ 
https://issues.apache.org/jira/browse/DAEMON-399?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mark Thomas resolved DAEMON-399.
--------------------------------
    Resolution: Fixed

I agree with the view that the quotes should not be removed. That functionality 
has been removed and I have confirmed that the provided test case now works as 
expected.

> apxStrUnQuoteInplaceA removes needed quotes
> -------------------------------------------
>
>                 Key: DAEMON-399
>                 URL: https://issues.apache.org/jira/browse/DAEMON-399
>             Project: Commons Daemon
>          Issue Type: Bug
>          Components: Procrun
>    Affects Versions: 1.1.0
>            Reporter: Camillo Dell'mour
>            Priority: Minor
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> When trying to pass the following argument to the jvm
> {code:java}
> -Xlog:gc:file="d:\logs\gc.log"::filecount=5,filesize=20000
> {code}
> the method apxStrUnQuoteInplaceA removes " from it and the resulting command 
> looks like this:
> {code:java}
> -Xlog:gc:file=d:\logs\gc.log::filecount=5,filesize=2000
> {code}
> Which is an invalid jvm argument.
> Workaround for version 1.1.0 is to add a space into the log filename.
> IMHO the proper fix would be not too use apxStrUnQuoteInplaceA at all.
> Another possible fix would be to replace utils.c:571 with
> {code}
> else if (*p == ' ' || *p == ':') {
> {code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to