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

WIP: test gcc dll replacement on appveyor #196

Closed
wants to merge 3 commits into from
Closed

Conversation

tkelman
Copy link
Contributor

@tkelman tkelman commented Aug 5, 2017

not sure if this will work, but this may be a way to deal with the win32 issue

@coveralls
Copy link

coveralls commented Aug 5, 2017

Coverage Status

Coverage remained the same at 91.48% when pulling 7301dd4 on tk/appveyortest into 0fa5949 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.7%) to 90.807% when pulling 4ddc67e on tk/appveyortest into 0fa5949 on master.

@coveralls
Copy link

coveralls commented Aug 5, 2017

Coverage Status

Coverage remained the same at 91.48% when pulling 4ddc67e on tk/appveyortest into 0fa5949 on master.

1 similar comment
@coveralls
Copy link

coveralls commented Aug 5, 2017

Coverage Status

Coverage remained the same at 91.48% when pulling 4ddc67e on tk/appveyortest into 0fa5949 on master.

@lobingera
Copy link
Contributor

@tkelman Do you think this can be included in mainstream WinRPM soon?

@tkelman
Copy link
Contributor Author

tkelman commented Aug 6, 2017

it's a bit risky since the opensuse repo can change at any time, if anything breaks it may result in julia not starting correctly. maybe we could do it only on 32 bit?

@lobingera
Copy link
Contributor

@tkelman

maybe we could do it only on 32 bit?

yes.

Can you explain what error/failure you're working around? In my own quest to see why the 32bit fails it looked like the library is installed, but never recognized by julia

@tkelman
Copy link
Contributor Author

tkelman commented Aug 8, 2017

The short version is the opensuse build service upgraded to gcc 7 a few months ago, but the Windows Julia binaries are still built (cross-compiled from cygwin on the buildbots) using gcc 5. Evidently there are a few new functions in libgcc in gcc 7 that get used by cairo, but the libgcc that Julia includes (from gcc 5) does not have.

@lobingera
Copy link
Contributor

@tkelman Would you recommend to merge this, now?

@tkelman
Copy link
Contributor Author

tkelman commented Sep 29, 2017

Not really, not in this exact form. Could maybe start doing this over in WinRPM's build script but only on 32 bit (since it introduces a risk of upstream changes from opensuse breaking Julia installs unpredictably).

I'm no longer actively doing Julia development, so addressing this issue should probably be undertaken by someone else who regularly uses and tests things on Windows. I don't think there's anyone who fits that description at the moment, at least in terms of active people with commit access to base Julia. Maybe BinDeps2 will have a fully working Windows cross-compilation story eventually.

@lobingera
Copy link
Contributor

lobingera commented Sep 29, 2017

BinDeps2 development already started?

@ararslan
Copy link
Contributor

BinDeps2 development already started?

https://github.com/JuliaPackaging/BinDeps2.jl

@giordano
Copy link
Contributor

giordano commented Sep 7, 2019

This can probably be closed now, I guess

@ararslan ararslan closed this Sep 7, 2019
@ararslan ararslan deleted the tk/appveyortest branch September 7, 2019 17:51
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.

5 participants