• roadrunner_ex
    link
    fedilink
    English
    arrow-up
    14
    ·
    4 months ago

    Putting aside the “should/shouldn’t do” argument, I was also wondering if the code is even viable. I imagine that ‘ls’ and ‘sudo’ are probably pretty ubiquitous, but I bet there exist some Linux installs out there with a different shell than ‘bash’, and some might not have ‘grep’ too. That would lead to some pretty cryptic bugs for the end user, eh?

    • SteveTech@programming.dev
      link
      fedilink
      English
      arrow-up
      16
      ·
      4 months ago

      I might be wrong, but I believe Debian ships without sudo, only su by default (or at least if you configure a root password in the setup).

      • N0x0n@lemmy.ml
        link
        fedilink
        arrow-up
        11
        ·
        edit-2
        4 months ago

        You’re right ! And I really should get used to how debian works with su.

        But with docker it’s very convenient to add an user in the docker groupe with sudo :/

        • SteveTech@programming.dev
          link
          fedilink
          English
          arrow-up
          6
          ·
          4 months ago

          And I really should get used to how debian works with su.

          I only know because installing sudo is usually the very first thing I do whenever I have to install it haha.

    • const_void@lemmy.ml
      link
      fedilink
      arrow-up
      12
      ·
      4 months ago

      I bet there exist some Linux installs out there with a different shell than ‘bash’,

      100%. Alpine is one such distro.

    • zea@lemmy.blahaj.zone
      link
      fedilink
      arrow-up
      4
      ·
      4 months ago

      I believe POSIX mandates grep and a shell that should be able to handle everything this code uses, but sudo is a problem

    • MangoPenguin@lemmy.blahaj.zone
      link
      fedilink
      English
      arrow-up
      1
      ·
      4 months ago

      Debian doesn’t come with sudo and it’s always annoying running into stuff that has it hardcoded (or just added by default to a command)