Remotequeue does not accept some svn changelist names

[Gabriel Petrovay]

I have a changelist called: test\rbkt\ExpQueryResults\zorba\schemas\any.xml.res

I try to submit only my changelist to the patchqueue with:
cmake -D "ZORBA_BUILD_DIR=%CD%\builds\debug10" -D "changelist=test\rbkt\ExpQueryResults\zorba\schemas\any.xml.res" -P test\remote\RunRemoteTest.cmake

Seems like this case is not captured in CMake:

-- Zorba source dir: C:/Users/Gabriel/Work/28msec/zorba
CMake Warning (dev) at test/remote/SvnPackage.cmake:96 (file):
  Syntax error in cmake code at

    C:/Users/Gabriel/Work/28msec/zorba/test/remote/SvnPackage.cmake:96

  when parsing string

    test\rbkt\ExpQueryResults\zorba\schemas\any.xml.res

  Invalid escape sequence \E

  Policy CMP0010 is not set: Bad variable reference syntax is an error. Run
  "cmake --help-policy CMP0010" for policy details. Use the cmake_policy
  command to set the policy and suppress this warning.
Call Stack (most recent call first):
  test/remote/RunRemoteTest.cmake:62 (svn_package)
This warning is for project developers. Use -Wno-dev to suppress it.

CMake Warning (dev) at test/remote/SvnPackage.cmake:101 (get_files_with_status):
  Syntax error in cmake code at

    C:/Users/Gabriel/Work/28msec/zorba/test/remote/SvnPackage.cmake:101

  when parsing string

    test\rbkt\ExpQueryResults\zorba\schemas\any.xml.res

  Invalid escape sequence \E

  Policy CMP0010 is not set: Bad variable reference syntax is an error. Run
  "cmake --help-policy CMP0010" for policy details. Use the cmake_policy
  command to set the policy and suppress this warning.
Call Stack (most recent call first):
  test/remote/RunRemoteTest.cmake:62 (svn_package)
This warning is for project developers. Use -Wno-dev to suppress it.

CMake Warning (dev) at test/remote/SvnPackage.cmake:54 (if):
  Syntax error in cmake code at

    C:/Users/Gabriel/Work/28msec/zorba/test/remote/SvnPackage.cmake:54

  when parsing string

    test\rbkt\ExpQueryResults\zorba\schemas\any.xml.res

  Invalid escape sequence \E

  Policy CMP0010 is not set: Bad variable reference syntax is an error. Run
  "cmake --help-policy CMP0010" for policy details. Use the cmake_policy
  command to set the policy and suppress this warning.
Call Stack (most recent call first):
  test/remote/SvnPackage.cmake:101 (get_files_with_status)
  test/remote/RunRemoteTest.cmake:62 (svn_package)
This warning is for project developers. Use -Wno-dev to suppress it.

CMake Warning (dev) at test/remote/SvnPackage.cmake:57 (set):
  Syntax error in cmake code at

    C:/Users/Gabriel/Work/28msec/zorba/test/remote/SvnPackage.cmake:57

  when parsing string

    changelist[@name=\"test\rbkt\ExpQueryResults\zorba\schemas\any.xml.res\"]

  Invalid escape sequence \E

  Policy CMP0010 is not set: Bad variable reference syntax is an error. Run
  "cmake --help-policy CMP0010" for policy details. Use the cmake_policy
  command to set the policy and suppress this warning.
Call Stack (most recent call first):
  test/remote/SvnPackage.cmake:101 (get_files_with_status)
  test/remote/RunRemoteTest.cmake:62 (svn_package)
This warning is for project developers. Use -Wno-dev to suppress it.

Error in C:\Users\Gabriel\Work\28msec\zorba\src\compiler\api\compiler_api.cpp:221. Query: <.>, line 1, column 54: [XPST0003] syntax error, unexpected character
"\"
 Submitting job changes-2010-12-15T03-07-30.996Z.tgz....

[Chris Hillery (ceejatec)]

I'm going to refuse this one, unless you can provide me a strong use-case for supporting it. It would take at least a little bit of effort to solve, and it seems like a very odd thing to do. Could you just use / instead of \ in your changelists, if you need to have such complex names?

[Gabriel Petrovay (gabipetrovay)]

Then, we should leave it open I guess. "Later" is a much more positive thinking resolution than "Wont Fix" :)

And "Could you just use / instead of \" is also not the right question. One can easily find such question for every bug.

The priority is also low enough such that one can see that this is not critical. As a bonus I have even decremented it now to 2 ;)

If you refuse to make it (a.k.a Wont Fix), assign it back to me.

Thanks!

[Chris Hillery (ceejatec)]

No longer relevant as the old remote queue is dead. (Well, ok, it's still there for non-core modules, but that should be taken care of in the next week or so too.)