Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change print_escaped to escape_string #136

Closed
wants to merge 4 commits into from

Conversation

DrKrar
Copy link

@DrKrar DrKrar commented Apr 30, 2017

print_escaped is depreciated in Julia v0.5 and give error in Julia v0.6-dev. i think this will help the pakage to run in Julia v0.6-dev.

`print_escaped` is depreciated in Julia v0.5 and give error in Julia  v0.6-dev. i think this will help the pakage to run in Julia  v0.6-dev.
@DrKrar
Copy link
Author

DrKrar commented Apr 30, 2017

This PR is good for windows only as the code is conditionally executed in windows.

@tkelman
Copy link
Contributor

tkelman commented Apr 30, 2017

Thanks! We should probably set up appveyor for windows CI testing here

DrKrar added 3 commits May 1, 2017 00:55
correct for Julia v0.5 enviroment
re-correct the v.05 enviroment
- JULIAVERSION: "julialang/bin/winnt/x86/0.4/julia-0.4-latest-win32.exe"
- JULIAVERSION: "julialang/bin/winnt/x64/0.4/julia-0.4-latest-win64.exe"
- JULIAVERSION: "julialang/bin/winnt/x86/0.5/julia-0.5-latest-win32.exe"
- JULIAVERSION: "julialang/bin/winnt/x64/0.5/julia-0.5-latest-win64.exe"
Copy link
Author

Choose a reason for hiding this comment

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

I think this is the correct directory as it had worked for my package GeoEfficiency

@bkamins
Copy link
Contributor

bkamins commented Oct 20, 2017

Sorry for duplicating the above PR with #139.
EDIT: #139 adds an additional fix - I can leave only the latter in this PR is merged.

@aviks
Copy link
Collaborator

aviks commented Feb 5, 2020

Fixed by #147

@aviks aviks closed this Feb 5, 2020
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.

4 participants