• OwOarchist@pawb.social
      link
      fedilink
      English
      arrow-up
      54
      ·
      1 month ago

      rm -rf “$STEAMROOT/”* could be evaluated as rm -rf “/”* if $STEAMROOT is empty

      Ah, good times.

      • Victor@lemmy.world
        link
        fedilink
        arrow-up
        13
        ·
        1 month ago

        Probably a good idea to enable the nounset shell option when shipping commercial-grade install scripts. 😅

        • fruitcantfly@programming.dev
          link
          fedilink
          arrow-up
          2
          ·
          edit-2
          1 month ago

          That really ought to have been the default behavior. Opting out only takes one or two extra characters. Well, four if you don’t habitually bracket your variable names, which you should probably be doing anyways

        • OwOarchist@pawb.social
          link
          fedilink
          English
          arrow-up
          2
          ·
          1 month ago

          Or just include the trailing slash in $STEAMROOT, so that if the variable is empty, it will evaluate to rm -rf "" and do nothing.

          • Victor@lemmy.world
            link
            fedilink
            arrow-up
            3
            arrow-down
            1
            ·
            1 month ago

            That seems like a more error-prone solution than the shell option. That would make it evaluate to rm -rf * which would probably delete your home directory, I’m guessing. And also put the burden of knowing the slash needs to be included on something completely disconnected from the expression. Nah, don’t do it like that.

  • bitjunkie@lemmy.world
    link
    fedilink
    arrow-up
    18
    ·
    edit-2
    1 month ago

    This entire thread is giving me flashbacks to my junior dev days and forgetting a where clause when updating/deleting database records

    • Buckshot@programming.dev
      link
      fedilink
      arrow-up
      4
      ·
      1 month ago

      We’ve all done that at some point.

      That sinking feeling when your quick update doesn’t return immediately.

      Good to get into the habit of wrapping your queries in a begin transaction/rollback until you verify the result 😁

      • bitjunkie@lemmy.world
        link
        fedilink
        arrow-up
        2
        ·
        edit-2
        1 month ago

        I was never able to fully form the habit with transactions. The guardrails I use are writing the where clause first and keeping my query tool in read-only mode for prod connections and only disabling it once I’m sure I need to make a change.

  • zitrone 🍋@europe.pub
    link
    fedilink
    arrow-up
    17
    ·
    1 month ago

    that is why you don’t run random install script as root

    i hate that so many tools nowadays want you to curl | sh for installing

    • w33n1s@lemmy.world
      link
      fedilink
      arrow-up
      5
      ·
      1 month ago

      Glad I’m not the only one. It’s always the shell apps that use annoying emojis too

    • SpatchyIsOnline@lemmy.world
      link
      fedilink
      arrow-up
      3
      ·
      1 month ago

      i hate that so many tools nowadays want you to curl | sh for installing

      Using NixOS has the advantage of making me not want to touch these tools. If I can’t install it reproducibly with Nix, I’ll just look for an alternative :)

  • axx@slrpnk.net
    link
    fedilink
    arrow-up
    3
    ·
    1 month ago

    This is from 15 years ago. How many more times will we have to see it?

      • nil@piefed.ca
        link
        fedilink
        English
        arrow-up
        1
        arrow-down
        4
        ·
        1 month ago

        Ubuntu tries to become (or is already) the de facto standard. That is what assholes do.

          • nil@piefed.ca
            link
            fedilink
            English
            arrow-up
            2
            ·
            1 month ago

            Honestly, I meant it for a joke, but people started taking it seriously, I don’t know where to end this.

            • TrickDacy@lemmy.world
              link
              fedilink
              arrow-up
              3
              ·
              1 month ago

              Hmm alright. I too dislike some things about Ubuntu but on the other hand you cannot blame people (myself included) who either have tried other distros and had support issues and came back to Ubuntu based, or those who just want the easier route to begin with. Ubuntu-based seems to be a good compromise for many. I’ve also used arch and fedora and had more headaches and breakages on those. Also I literally have a laptop that gets supported on Ubuntu so it makes life easier all around. Fuck some of the things canonical does but I just do not want to spend time with things like fixing my wifi connection after running pacman -S install or whatever that command is.

      • sbeak@sopuli.xyz
        link
        fedilink
        English
        arrow-up
        8
        ·
        1 month ago

        Ubuntu is owned by Canonical, and they have been making some decisions that a lot of people in the community don’t like, most notably, the enforcement of snaps (in a nutshell, the repos are proprietary and centralised), but they have been involved in various other corpo shenanigans as well

        But I don’t think “X distro is bad, don’t use it” is very useful unless it is dangerous or harmful to the user.

        Ubuntu (or the various other buntus) work for some people, and without them, far fewer people would use Linux. The benefit of Linux is the amount of choice you get and being able to easily switch between distros (you can even dual boot if you want!)

        So if you like Mint, stick with that. If somebody prefers the buntus, it’s great as long as they are respectful to others!

        Distro wars are stupid

      • MrKoyun@lemmy.world
        link
        fedilink
        arrow-up
        5
        ·
        1 month ago

        It’s not really bad. It’s just kinda distasteful due to stuff like Snap packages and Canonical’s position of privacy as a company. But it isn’t really a bad distro if you ignore that stuff. If it works it works.

      • nil@piefed.ca
        link
        fedilink
        English
        arrow-up
        3
        ·
        1 month ago

        Ubuntu kinda forces you to use snap (which sucks), and is owned by Canonical.

        Neither of them apply to Linux Mint, so nice choice.