search

Roger-luo / IonCLI.jl

The Ion CLI for Julia.
MIT License
46 stars 2 forks source link

should `ion update julia` be supported? #44

Open simeonschaub opened 3 years ago

simeonschaub commented 3 years ago

Both of these currently fail:

[simeon@majestix ~]$ ion update julia latest
ERROR: MethodError: no method matching (::IonBase.var"#8#9"{Tuple{String,String}})()
Closest candidates are:
  #8(::Any) at /home/runner/.julia/packages/IonBase/cqJsJ/src/command/pkg.jl:106
Stacktrace:
 [1] map(::IonBase.var"#8#9"{Tuple{String,String}}) at abstractarray.jl:2247
 [2] update(::String, ::Vararg{String,N} where N; level::Int64, glob::Bool) at /home/runner/.julia/packages/IonBase/cqJsJ/src/command/pkg.jl:106
 [3] update(::String, ::Vararg{String,N} where N) at /home/runner/.julia/packages/IonBase/cqJsJ/src/command/pkg.jl:103
 [4] command_main(::Array{String,1}) at /home/runner/.julia/packages/Comonicon/6OS1s/src/codegen/ast.jl:255
 [5] command_main() at /home/runner/.julia/packages/Comonicon/6OS1s/src/codegen/ast.jl:46
 [6] julia_main() at /home/runner/.julia/packages/Comonicon/6OS1s/src/parse/cast.jl:316
 [7] julia_main() at none:32
[simeon@majestix ~]$ ion update julia 
ERROR: MethodError: no method matching (::IonBase.var"#8#9"{Tuple{String}})()
Closest candidates are:
  #8(::Any) at /home/runner/.julia/packages/IonBase/cqJsJ/src/command/pkg.jl:106
Stacktrace:
 [1] map(::IonBase.var"#8#9"{Tuple{String}}) at abstractarray.jl:2247
 [2] update(::String; level::Int64, glob::Bool) at /home/runner/.julia/packages/IonBase/cqJsJ/src/command/pkg.jl:106
 [3] update(::String) at /home/runner/.julia/packages/IonBase/cqJsJ/src/command/pkg.jl:103
 [4] command_main(::Array{String,1}) at /home/runner/.julia/packages/Comonicon/6OS1s/src/codegen/ast.jl:255
 [5] command_main() at /home/runner/.julia/packages/Comonicon/6OS1s/src/codegen/ast.jl:46
 [6] julia_main() at /home/runner/.julia/packages/Comonicon/6OS1s/src/parse/cast.jl:316
 [7] julia_main() at none:32

Would this be possible to support?

Roger-luo commented 3 years ago

we currently forward some CLI commands to Pkg, so ion update is for updating packages in current environment. and julia is actually a registered package in General: https://github.com/JuliaRegistries/General/tree/master/J/julia

this is why it didn't say julia is not a package.

update an application can use ion upgrade in principle, but this is not implemented.