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

remove text example from tests (2nd option for fixing #222) #225

Closed
wants to merge 1 commit into from

Conversation

shashi
Copy link
Collaborator

@shashi shashi commented Oct 19, 2016

No description provided.

@shashi
Copy link
Collaborator Author

shashi commented Oct 19, 2016

what should I do about the nightly test fails? @tkelman

@@ -2,7 +2,20 @@
cachedout = joinpath(testdir, "data")
differentfiles = AbstractString[]
const creator_producer = r"(Creator|Producer)"
for output in readdir(cachedout)

const KERNEL = VERSION < v"0.5.0-dev" ? OS_NAME : Sys.KERNEL
Copy link
Contributor

Choose a reason for hiding this comment

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

use Compat, this cutoff isn't right

@tkelman
Copy link
Contributor

tkelman commented Oct 19, 2016

looks like JuliaLang/julia#18839 may have broken precompilation of Iterators.jl. I suspected we shouldn't export the Base.Iterators module. Reduce and report to base, or see if removing Iterators from base/exports.jl is enough to fix it locally

@tlnagy
Copy link
Member

tlnagy commented Nov 15, 2016

Any progress on this? Gadfly is also having a lot of warnings due to JuliaLang/julia#18839 and I'm not sure how to patch everything given the reliance on both Iterators and Base.Iterators.

@tlnagy tlnagy closed this Sep 8, 2018
@tlnagy tlnagy deleted the s/text_workaround-2 branch September 8, 2018 22:12
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.

3 participants