[GAP_pkg_caratinterface] new package

[fingolfin]

No description provided.

[fingolfin]

This apparently results in the error

ERROR: LoadError: `Order` is already defined in Base
Stacktrace:
 [1] error(s::String)
   @ Base ./error.jl:33
 [2] ExecutableProduct(binnames::Vector{String}, varname::Symbol, dir_path::Nothing)
   @ BinaryBuilderBase /cache/julia-buildkite-plugin/depots/e2fd9734-29d8-45cd-b0eb-59f7104f3131/packages/BinaryBuilderBase/LieLT/src/Products.jl:356
 [3] ExecutableProduct(binnames::Vector{String}, varname::Symbol)
   @ BinaryBuilderBase /cache/julia-buildkite-plugin/depots/e2fd9734-29d8-45cd-b0eb-59f7104f3131/packages/BinaryBuilderBase/LieLT/src/Products.jl:355
 [4] ExecutableProduct(::String, ::Symbol)
   @ BinaryBuilderBase /cache/julia-buildkite-plugin/depots/e2fd9734-29d8-45cd-b0eb-59f7104f3131/packages/BinaryBuilderBase/LieLT/src/Products.jl:364
 [5] top-level scope
   @ /cache/build/yggy-amdci7-0/julialang/yggdrasil/G/GAP_pkg/GAP_pkg_caratinterface/build_tarballs.jl:50
in expression starting at /cache/build/yggy-amdci7-0/julialang/yggdrasil/G/GAP_pkg/GAP_pkg_caratinterface/build_tarballs.jl:50

and indeed we have this in BinaryBuilderBase:

    function ExecutableProduct(binnames::Vector{String}, varname::Symbol, dir_path::Union{AbstractString, Nothing}=nothing)
        if isdefined(Base, varname)
            error("`$(varname)` is already defined in Base")
        end

But why? And is this documented somewhere?

Any pointers, @giordano

(I'll work around it here by renaming, presumably also for Symbol below, it is no big issue; but I still am curious as to why this is an error in the first place)

[giordano]

Because you can't refer to a variable with the same name as a variable exported by Base (unless you qualify it with the module, but we also export the var), we had issues in the past. But perhaps the check can be refined to limit it to exported symbols.