Skip to content

add -f flag to cp in install target#73

Merged
ViralBShah merged 2 commits intoJuliaMath:masterfrom
tkelman:cp-dashf
Jan 7, 2015
Merged

add -f flag to cp in install target#73
ViralBShah merged 2 commits intoJuliaMath:masterfrom
tkelman:cp-dashf

Conversation

@tkelman
Copy link
Copy Markdown
Contributor

@tkelman tkelman commented Jan 7, 2015

Would anyone be opposed to this? I get occasional build failures in Julia due to "unable to cp, file exists" or similar errors. This was probably triggered by a different change I made in one of Julia's makefiles, but that was fixing a different problem.

@tkelman
Copy link
Copy Markdown
Contributor Author

tkelman commented Jan 7, 2015

Looks like the osx tests got turned on, and they fail. cc @ViralBShah

ViralBShah added a commit that referenced this pull request Jan 7, 2015
add -f flag to cp in install target
@ViralBShah ViralBShah merged commit b44ec54 into JuliaMath:master Jan 7, 2015
@ViralBShah
Copy link
Copy Markdown
Member

I wrote to them as you suggested and they did it. The tests have been in that condition for a while - a few ULPs here and there. Could be a gcc/clang difference or something else. At least enough digging has to now be done to make the tests pass.

@tkelman tkelman deleted the cp-dashf branch January 7, 2015 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants