New favorite tool 😍

  • zygo_histo_morpheus
    link
    fedilink
    arrow-up
    52
    ·
    7 months ago

    Looking at the example

    Why does the generated bash look like that? Is this more safe somehow than a more straighforward bash if or does it just generate needlessly complicated bash?

    • sxt@lemmy.world
      link
      fedilink
      arrow-up
      31
      arrow-down
      1
      ·
      edit-2
      7 months ago

      I doubt the goal is to produce easily understood bash, otherwise you’d just write bash to begin with. It’s probably more similar to a typescript transpiler that takes in a language with different goals and outputs something the interpreter can execute quickly (no comment on how optimized this thing is).

    • thingsiplay@beehaw.org
      link
      fedilink
      arrow-up
      16
      ·
      7 months ago

      Especially as Bash can do that anyway with if [ "${__0_age}" -lt 18 ] as an example, and could be straight forward. Also Bash supports wildcard comparison, Regex comparison and can change variables with variable substitution as well. So using these feature would help in writing better Bash. The less readable output is expected though, for any code to code trans-compiler, its just not optimal in this case.

      • BatmanAoD
        link
        fedilink
        arrow-up
        12
        ·
        7 months ago

        It’s probably just easier to do all arithmetic in bc so that there’s no need to analyze expressions for Bash support and have two separate arithmetic codegen paths.

        • thingsiplay@beehaw.org
          link
          fedilink
          arrow-up
          8
          ·
          7 months ago

          But its the other way, not analyzing Bash code. The code is already known in Amber to be an expression, so converting it to Bash expression shouldn’t be like this I assume. This just looks unnecessary to me.

          • BatmanAoD
            link
            fedilink
            arrow-up
            4
            ·
            edit-2
            7 months ago

            No, I mean, analyzing the Amber expression to determine if Bash has a native construct that supports it is unnecessary if all arithmetic is implemented using bc. bc is strictly more powerful than the arithmetic implemented in native Bash, so just rendering all arithmetic as bc invocations is simpler than rendering some with bc and some without.

            Note, too, that in order to support Macs, the generated Bash code needs to be compatible with Bash v3.

            • thingsiplay@beehaw.org
              link
              fedilink
              arrow-up
              1
              ·
              7 months ago

              I see, it’s a universal solution. But the produced code is not optimal in this case. I believe the Amber code SHOULD analyze it and decide if a more direct and simple code generation for Bash is possible. That is what I would expect from a compilers work. Otherwise the generated code becomes write only, not read only.

              • BatmanAoD
                link
                fedilink
                arrow-up
                2
                ·
                7 months ago

                Compiled code is already effectively write-only. But I can imagine there being some efficiency gains in not always shelling out for arithmetic, so possibly that’s a future improvement for the project.

                That said, my reaction to this project overall is to wonder whether there are really very many situations in which it’s more convenient to run a compiled Bash script than to run a compiled binary. I suppose the Bash has the advantage of being truly “compile once, run anywhere”.