4 ms·

The real WTF is that APT spawns an external process to check disk usage, instead of invoking statvfs() on its own. Some cases are less clear-cut, however. For

• orf 9y ago
  It could be called from a bash script rather than from within apt itself.
• phaemon 9y ago
  It isn't `apt`. See how they mentioned that `mysql` packages were installing? Well, in the `preinst` shell script, in the .deb, there is the line: `if LC_ALL=C BLOCKSIZE= df --portability $DATADIR/. | tail -n 1 | awk '{ exit ($4>1000) }'; then` There's `df` being called without the full path and that's what would be spawning Dwarf Fortress.
  • glandium 9y ago
    That doesn't make it much better though. Parsing the output of df (with a useless tail -1) is kind of awful when you can use something like stat -f -c %a.
    • MichaelBurge 9y ago
      That has the same problem, just with a different name. So your solution solves a style complaint, not an actual functional problem.
      • sslayer 9y ago
        I agree with your comment about it being functional. But wouldn't removing tail from the pipe remove complexity and speed up the processing(Arguably, most likely only by a small fraction)
    • linsomniac 9y ago
      How is the "tail -1" useless? The output of that df command is two lines, one contains the column headers and the next contains the data.
      • glandium 9y ago
        because `... | tail -n -1 | awk '{...}'` is the same as `... | awk 'END {...}'`.
• lorenzhs 9y ago
  On the node thing: that decision has recently been reversed by the technical committee, effective in the next release (it is in unstable now). See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=862051 https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=862051 - nodejs will now simply be 'node'.