Skip to content
This repository was archived by the owner on Aug 1, 2025. It is now read-only.

Fix #3 - absolute path issue on OS X - #7

Open
msambarino wants to merge 1 commit into
Psvensso:masterfrom
msambarino:master
Open

Fix #3 - absolute path issue on OS X#7
msambarino wants to merge 1 commit into
Psvensso:masterfrom
msambarino:master

Conversation

@msambarino

Copy link
Copy Markdown

I came across this issue and decided to fix it as it seemed like a quick task. I based it off of @vorce's work.

A couple screenshots working on OS X:

screen shot 2015-12-09 at 6 01 49 pm

screen shot 2015-12-09 at 6 50 07 pm

@msambarino

Copy link
Copy Markdown
Author

I realize now that @vorce opened #4 for this, i will point however that i believe theres an issue with the proposed changes, specifically, the "-S" arg is missing for running iex -S mix in the args definition.

@vorce

vorce commented Dec 9, 2015

Copy link
Copy Markdown

Actually running iex /path/to/mix is the same as iex -S mix :)
Regardless it would be cool to get this or #4 merged.
👍

@msambarino

Copy link
Copy Markdown
Author

@vorce Ha! silly me, i did a quick test with iex mix instead of iex path/to/mix. Sorry for the confusion.
I'll keep this open in case further discussion is warranted but my vote goes to merging #4 as my PR is pretty much based on the work you did.

@mgwidmann

Copy link
Copy Markdown
Contributor

Any chance of this getting finished, merged and out in a release?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants