Fix patchwork naming issue on OSX

[shashi]

related to JuliaLang/julia#9079

WTF!!!!!

[jiahao]

This still doesn't fix the problem for me: (problem with precompilation)

julia> using Compose #this branch, julia release-0.3 head

julia> Compose.Patchable
ERROR: Patchable not defined

julia> Pkg.installed("Patchwork")
v"0.1.6"

julia> Compose.isinstalled("Patchwork", v"0.1.2")
true

[shashi]

This is not merged yet. You need to checkout the shashi-osx-fix branch.

On Tue, Jul 14, 2015, 2:31 PM Jiahao Chen [email protected] wrote:

This still doesn't fix the problem for me:

julia> using Compose #master, julia release-0.3 head

julia> Compose.Patchable
ERROR: Patchable not defined

julia> Pkg.installed("Patchwork")
v"0.1.6"

julia> Compose.isinstalled("Patchwork", v"0.1.2")
true

—
Reply to this email directly or view it on GitHub
#130 (comment).

[jiahao]

This is not merged yet. You need to checkout the shashi-osx-fix branch.

Yes, I applied this patch and it still doesn't work for me.

[jiahao]

Ok never mind I forgot I had Compose precompiled on my release branch.

It does work for me now.

[shashi]

I checked again, I did rename it properly with git. Can you check if
src/patchwork.jl isn't lurking? And it worked for Alan...

On Tue, Jul 14, 2015, 2:53 PM Jiahao Chen [email protected] wrote:

This is not merged yet. You need to checkout the shashi-osx-fix branch.

Yes, I applied this patch and i still doesn't work for me.

—
Reply to this email directly or view it on GitHub
#130 (comment).

[shashi]

Oh.

On Tue, Jul 14, 2015, 3:10 PM Shashi Gowda [email protected] wrote:

I checked again, I did rename it properly with git. Can you check if
src/patchwork.jl isn't lurking? And it worked for Alan...

On Tue, Jul 14, 2015, 2:53 PM Jiahao Chen [email protected]
wrote:

This is not merged yet. You need to checkout the shashi-osx-fix branch.

Yes, I applied this patch and i still doesn't work for me.

—
Reply to this email directly or view it on GitHub
#130 (comment).

[jiahao]

For the record, the problem is fixed so long as Compose is not precompiled. Even with this patch, the Patchable not defined error is thrown if Compose is precompiled.

[shashi]

Oh wow.

On Tue, Jul 14, 2015, 3:26 PM Jiahao Chen [email protected] wrote:

For the record, the problem is fixed so long as Compose is not
precompiled. Even with this patch, the Patchable not defined error is
thrown if Compose is precompiled.

—
Reply to this email directly or view it on GitHub
#130 (comment).