JuliaParallel / PETSc.jl

Julia wrappers for the PETSc library
Other
121 stars 38 forks source link

LoadError: ccall method definition: argument 1 type dosen't correspond to a C type #187

Closed Blumenkranz closed 3 weeks ago

Blumenkranz commented 8 months ago

I tried to install PETSc with ]add PETSc and I got

ERROR: LoadError: ccall method definition: argument 1 type doesn't correspond to a C type
Stacktrace:
 [1] top-level scope
   @ ~/.julia/packages/PETSc/LSdRz/src/ksp.jl:199
 [2] eval(m::Module, e::Any)
   @ Core ./boot.jl:370
 [3] top-level scope
   @ ~/.julia/packages/PETSc/LSdRz/src/lib.jl:120
 [4] include(mod::Module, _path::String)
   @ Base ./Base.jl:457
 [5] include(x::String)
   @ PETSc ~/.julia/packages/PETSc/LSdRz/src/PETSc.jl:1
 [6] top-level scope
   @ ~/.julia/packages/PETSc/LSdRz/src/PETSc.jl:34
 [7] include
   @ ./Base.jl:457 [inlined]
 [8] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt128}}, source::Nothing)
   @ Base ./loading.jl:2049
 [9] top-level scope
   @ stdin:3

The Julia version is 1.9 and I'm using an M1 mac.

boriskaus commented 8 months ago

yes, PETSc.jl is not working and not maintained at the moment. I'm trying to halfway maintain the binarybuilder version of PETSc (PETSc_jll) but lack time to do this for PETSc.jl

Blumenkranz commented 8 months ago

I see. Thanks for your reply

mSamiolo commented 8 months ago

Noted, just out completition, it does not work also with Julia 1.10.1 with x86 instruction set. Neither if I use an existing PETSc installation.

ViralBShah commented 3 weeks ago

This should be fixed now on the main branch with #194. Can you dev the package and try it out?

ViralBShah commented 3 weeks ago

Keeping it open until a new release is tagged.

Blumenkranz commented 3 weeks ago

This should be fixed now on the main branch with #194. Can you dev the package and try it out?

Yes, the main branch can be successfully loaded now