• Tb0n3@sh.itjust.works
    link
    fedilink
    English
    arrow-up
    9
    arrow-down
    45
    ·
    edit-2
    11 months ago

    These rewrites in rust are merely just training exercises for those doing it. It wasn’t needed and in most cases isnt used.

    • Deebster@programming.dev
      link
      fedilink
      English
      arrow-up
      31
      arrow-down
      1
      ·
      11 months ago

      Large parts of the rewrite came from contributors who had never worked on fish before.

      That’s pretty useful alone.

      And there’s this:

      Thread Safety

      Allowing background functions and concurrent functions has been a goal for many years. I have been nursing a long-lived branch which allows full threaded execution. But though the changes are small, I have been reluctant to propose them, because they will make reasoning about the shell internals too complex: it is difficult in C++ to check and enforce what crosses thread boundaries.

      This is Rust’s bread and butter: we will encode thread requirements into our types, making it explicit and compiler-checked, via Send and Sync. Rust will allow turning on concurrent mode in a safe way, with a manageable increase in complexity, finally enabling this feature.

    • atzanteol@sh.itjust.works
      link
      fedilink
      arrow-up
      25
      arrow-down
      1
      ·
      11 months ago

      They did it “for the vibes”

      Vibes are just as important to free/open source software as proprietary software and although there were solid technical reasons for the port, the PR outcomes are added benefits.

    • ParetoOptimalDev
      link
      fedilink
      arrow-up
      5
      arrow-down
      1
      ·
      11 months ago

      It was needed to safely further support for concurrent features? If they follow through on adding that support, there will likely be adoption.

      The problem is in most cases the implementers stop at “same thing but in rust” without taking advantage of that.

      I can’t fully blame them since just duplicating an existing thing is a huge undertaking.

    • anteaters@feddit.de
      link
      fedilink
      arrow-up
      9
      arrow-down
      41
      ·
      11 months ago

      Angry downvotes because people don’t like to hear that a meme language is a meme language.

      • acwern@sh.itjust.works
        link
        fedilink
        arrow-up
        27
        ·
        11 months ago

        Probably not “angry” downvotes. OP provided a link where it’s explained exactly why the switch was made. Even if you don’t care for Rust it’s pretty clear that this was done with more purpose than just “Ooo let’s make it in Rust for fun”

        • Ephera@lemmy.ml
          link
          fedilink
          arrow-up
          6
          ·
          11 months ago

          And even if it was for fun, that would still be valid. The project is run by volunteers. If they don’t have fun, they stop doing it.