Skip to content

make manager calls nmake /F Makefile.win on Windows#2426

Merged
josevalim merged 1 commit intoelixir-lang:masterfrom
chyndman:mix-make-manager-win-nmake
Jun 18, 2014
Merged

make manager calls nmake /F Makefile.win on Windows#2426
josevalim merged 1 commit intoelixir-lang:masterfrom
chyndman:mix-make-manager-win-nmake

Conversation

@chyndman
Copy link
Copy Markdown
Contributor

Dependencies on Windows should compile with nmake /F Makefile.win. So these commits set :make as the manager if either Makefile OR Makefile.win exist, then makes the platform-appropriate call.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick question: does make ever work on Windows? I am asking because maybe we should check if nmake is available first? Or at least check if Makefile.win is available? Isn't there a chance a makefile will work fine on both OS?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@josevalim make does work on Windows (that's how I build Elixir) but Makefiles written for make typically expect gcc to be installed, which on Windows isn't guaranteed and could create compatibility issues if used. The motivation behind avoiding make is that if people use a precompiled Elixir package don't need to use MinGW.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, I'm still looking into how to implement C interop stuff in MIx directly, so we can avoid makefiles altogether.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok so I think we should definitely check if there is a Makefile.win before invoking nmake \F Makefile.win for now. I will amend it locally.

josevalim pushed a commit that referenced this pull request Jun 18, 2014
make manager calls nmake /F Makefile.win on Windows
@josevalim josevalim merged commit 846703b into elixir-lang:master Jun 18, 2014
@chyndman chyndman deleted the mix-make-manager-win-nmake branch June 18, 2014 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants