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

[release-0.5] final backports for 0.5.0 #18546

Merged
merged 32 commits into from
Sep 19, 2016
Merged

Conversation

tkelman
Copy link
Contributor

@tkelman tkelman commented Sep 16, 2016

closes #17418

will run pkgeval, and
@nanosoldier runbenchmarks(ALL, vs = ":release-0.5")

vtjnash and others added 22 commits September 11, 2016 23:14
…lizer

fix #18449
(the real, non-buggy fix is in #18413 for v0.6-dev master)
Also adds a broken test for #18307

(cherry picked from commit e242d46)
ref #18352
(cherry picked from commit e531600)
ref #18333
* Add accuracy note for 2-arg log

Addresses #18298.

* update based on comments

(cherry picked from commit 5f74d16)
Preserve/update `NewvarNode`.

(cherry picked from commit 7dc06b0)
ref #18386
we want cache=false to disable putting the method in mt->cache (possibly with a wider signature)
but it should still be doing the expected unique-ing through the method->specializations cache

fix #18404

(cherry picked from commit 9402af3)
ref #18415
I am testing julia on a machine where my home directory is a
symlink.  Unfortunately, this means the repl tests hang, as they
wait for the homedir to be given at the shell prompt, but instead
the pwd() is printed.  `pwd()` calls `uv_cwd()` which calls
`getcwd()`, which [will always resolve symbolic links](http://stackoverflow.com/a/1545838/1558890),
thus printing something other than `homedir()` on my system.
This changes the repl tests to wait for `realpath(homedir())`
instead of `homedir()`.

(cherry picked from commit f26aee2)
ref #18416
(cherry picked from commit 59c1cda)
ref #18393
(cherry picked from commit df99e20)
ref #18393
also avoid an expensive loop in the common case of the hotpath

fix #18399

(cherry picked from commit a188f95)
ref #18428
fix #18080

(cherry picked from commit a78ad7b)
ref #18430
It is getting installed correctly

(cherry picked from commit 5f91c39)
ref #18418
Fixes 18395
(cherry picked from commit fa4c02c)
Has been broken since 172fdb2.

(cherry picked from commit 46773af)
ref #18536
…iarc

Pass through `ENV["PATH"]` in the `setenv` call since it is read from in
https://github.com/JuliaLang/julia/blob/v0.5.0-rc4/contrib/mac/juliarc.jl
(cherry picked from commit db01a8f)
ref #18545

also make the test a little more robust against contents of /tmp
(cherry picked from commit ffbb0eb)
@musm
Copy link
Contributor

musm commented Sep 16, 2016

I would add #18368
Since the docs for that are wrong

@MichaelHatherly
Copy link
Member

#18229 should be fine to backport as well — if it's not too late?

Fixes #11249 to follow the CommonMark spec for lists which does not
require a blank line prior to the start of a list.

(cherry picked from commit d8da082)
ref #18229
@nanosoldier
Copy link
Collaborator

Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. cc @jrevels

@tkelman
Copy link
Contributor Author

tkelman commented Sep 17, 2016

Failures were in Cairo, Mamba, and Gadfly. Cairo we've seen before, might be fixed in that package but need a tag I think? Mamba and Gadfly should be fixed by #18452 which I'll probably merge into this branch but does need a test.

edit: forgot this part https://gist.github.com/106fdba4f38658b0622742f96798a840

vtjnash and others added 6 commits September 17, 2016 04:10
this ensures that the writes are completed before flushing the icache
otherwise, platforms like ARM would potentially fail to see some of the writes
and were especially likely to miss the relocations applied to the JIT code

fix #17577

(cherry picked from commit 6f57c40)
ref #18516
…on and promotion man (#18368)

* Remove uneccesary promote_rule from doc for conversion and promotion

* remove promote_op reference since it's dep

* Fixed oversights.

(cherry picked from commit 7423d72)
Fixes #18531.
(cherry picked from commit adcbf1e)
* fix docs for how to change colors in REPL

* double backticks

(cherry picked from commit 07cf50f)
[release 0.5] hack in a fix to getting the right lambda for the incremental deserializer
Fixes #18271. Allow multidimensional logical arrays as indices into a particular dimension so long as there is only one non-singleton dimension.

(cherry picked from commit 5fa5b7f)
ref #18401

Fixup inline comment

(cherry picked from commit 2bbcd80)

Update devdocs for bounds checking following #18401

I had forgotten that this behavior was documented in the developer section of the manual when authoring #18401.

(cherry picked from commit b8406ff)
ref #18552
@tkelman
Copy link
Contributor Author

tkelman commented Sep 17, 2016

@nanosoldier runbenchmarks(ALL, vs = ":release-0.5")

@tkelman
Copy link
Contributor Author

tkelman commented Sep 17, 2016

@jrevels recent nanosoldier webhooks aren't going through, "We had a problem connecting to the server. The most common problem with these types of errors is a misconfigured SSL certificate. Try verifying your certificate chain."

* Attempt to fix osx travis

We're now getting
> Error: You must `brew link autoconf` before staticfloat/julia/julia can be installed

Probably because Homebrew officially started supporting 10.12 Sierra, and that broke something.
Try to use the 10.10 Travis OSX image instead of the default 10.9 (https://docs.travis-ci.com/user/osx-ci-environment/)

* explicitly install cmake on osx travis

(cherry picked from commit 45bc17d)
@tkelman
Copy link
Contributor Author

tkelman commented Sep 18, 2016

@nanosoldier runbenchmarks(ALL, vs = ":release-0.5")

@nanosoldier
Copy link
Collaborator

Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. cc @jrevels

@tkelman
Copy link
Contributor Author

tkelman commented Sep 18, 2016

https://gist.github.com/525cd694db739bc38a7b17761fbaa2d8

Given @timholy's comment at #18401 (comment), I think I'm going to revert 15096f0 (cc @mbauman) and rerun pkgeval one hopefully-last time for good measure.

@timholy
Copy link
Sponsor Member

timholy commented Sep 18, 2016

Re #18401: out of curiosity, did you get any failures because of it? It is a tough call: on one hand it might conflict with a "better" way to solve the problem for 0.6, on the other hand it fixes a regression from 0.4. I tend to be conservative by not adding behavioral changes at the last minute, but I recognize this is a difficult decision.

@tkelman
Copy link
Contributor Author

tkelman commented Sep 19, 2016

From that most recent pkgeval run, it neither caused nor solved any new failures afaict. However note that PyPlot is broken right now so any regressions in packages using it might have been missed.

@tkelman
Copy link
Contributor Author

tkelman commented Sep 19, 2016

I'm about to merge and tag this. https://gist.github.com/636a2c609ac0ce380053f0973eb47746
Speak up soon if you think something's missing or I should reconsider any of what's here.

@ViralBShah
Copy link
Member

Is #17602 release blocking, or should it be tagged 0.5.x?

@tkelman
Copy link
Contributor Author

tkelman commented Sep 19, 2016

If there were a PR last week, or a repeatable demonstration of an issue there then maybe, but without a proposed PR I don't think we can hold this for anything else.

@tkelman tkelman merged commit db7eaf9 into release-0.5 Sep 19, 2016
@tkelman tkelman deleted the tk/backports-0.5.0 branch September 19, 2016 17:19
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.

None yet