Skip to content
This repository has been archived by the owner on Feb 19, 2019. It is now read-only.

Fix for multiple argument expansion under bash #383

Closed
wants to merge 1 commit into from

Conversation

arcaartem
Copy link

Current implementation expands provided arguments and passes it as a single argument to the executable.

@ferventcoder
Copy link
Contributor

Thanks!

On Wednesday, November 27, 2013, Arca Artem wrote:

Current implementation expands provided arguments and passes it as a

single argument to the executable.

You can merge this Pull Request by running

git pull https://github.com/ArcaArtem/chocolatey master

Or view, comment on, or merge it at:

#383
Commit Summary

  • Fix for multiple argument expansion under bash

File Changes

Patch Links:


Rob
"Be passionate in all you do"

http://devlicio.us/blogs/rob_reynolds
http://ferventcoder.com
http://twitter.com/ferventcoder

@geraldcombs
Copy link

The generated files use DOS line endings, which makes them unusable from a standard Cygwin environment. Would it be possible to insert something like

% {$_.replace("`r`n", "`n")}

before Out-File in the pipeline?

@ferventcoder
Copy link
Contributor

@geraldcombs we can probably do something like that as well.

@ferventcoder
Copy link
Contributor

Fixed both items at b0e5fe6

ferventcoder added a commit to ferventcoder/chocolatey that referenced this pull request Jun 17, 2014
This provides the ideas from the original request but adds the correct line
endings to the file that is produced. Out-File unfortunately always adds an
additional CRLF to the end of the file and that cannot be removed with a
-replace, so it became easier just to drop down to .NET code to write the file.
ferventcoder added a commit to ferventcoder/chocolatey that referenced this pull request Jun 17, 2014
* stable:
  (chocolatey-archiveGH-383) Fix for multiple arg expansion - bash
  (doc) Chocolatey-Help makeover

Conflicts:
	src/functions/Chocolatey-Help.ps1
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants