Giter VIP home page Giter VIP logo

Comments (8)

barkingfoodog avatar barkingfoodog commented on June 12, 2024

Sorry for the delay in getting back to you...I've been swamped lately!

I added a test to oOmnisCli.$_parseArguments() to try and demonstrate this error, but the test correctly sees 'foo bar' 'baz biff' as two arguments.

There could be some command-line parsing causing the issue. What platform and shell are you using?

from omniscli.

Frogli avatar Frogli commented on June 12, 2024

No problem, we all have busy schedules... 😉
I use MacOs 10.12.6 and tested it in Bash and zsh. Both give the same results. I noticed it does work if you wrap the arguments with single and double quotes.
eg
"/Applications/Omnis Studio Runtime 8.0.3.5 x64.app/Contents/MacOS/bin/omniscli" Hello "'foo bar'" "'baz biff'"

from omniscli.

barkingfoodog avatar barkingfoodog commented on June 12, 2024

I found the problem. The omniscli and omniscli.ps1 scripts were consuming the single quotes. I checked in a fix, so grab a new copy of omniscli and see if that fixes the issue.

from omniscli.

Frogli avatar Frogli commented on June 12, 2024

from omniscli.

barkingfoodog avatar barkingfoodog commented on June 12, 2024

Both scripts needed a fix. The script takes its arguments and stores them in an environment variable for Omnis to read. The quotes kept the arguments grouped properly in the script, but the script didn’t re-add them for Omnis to see. Both scripts now quote each argument to ensure Omnis parses them as the user intended.

from omniscli.

Frogli avatar Frogli commented on June 12, 2024

from omniscli.

barkingfoodog avatar barkingfoodog commented on June 12, 2024

Yeah...I squashed the commits for the Mac and Windows scripts and went one revision too far for the release notes. If you check the code for omniscli you'll see a shift to get the command, then a for loop to wrap the arguments in single quotes. This is the fix for this issue. Can you give it a try on your box and verify?

from omniscli.

Frogli avatar Frogli commented on June 12, 2024

Ah sorry, I was confused by the information GitHub gave me. When I compared the revisions of the omniscli file I saw the change and it also works at my computer!

Thanks!

from omniscli.

Related Issues (6)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.