[GRASS-git] [OSGeo/grass] cc82d8: utils(dep_tree2sql): Update to valid syntax, addre...

Edouard Choinière noreply at github.com
Tue Dec 3 02:44:38 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/OSGeo/grass
  Commit: cc82d8e102304d98a3119f493c46321649b2cf69
      https://github.com/OSGeo/grass/commit/cc82d8e102304d98a3119f493c46321649b2cf69
  Author: Edouard Choinière <27212526+echoix at users.noreply.github.com>
  Date:   2024-12-03 (Tue, 03 Dec 2024)

  Changed paths:
    M utils/dep_tree2sql.sh

  Log Message:
  -----------
  utils(dep_tree2sql): Update to valid syntax, address shellcheck warnings (#3547)

* utils(dep_tree2sql): Exit with exit code 1 when changing directory fails

* utils(dep_tree2sql): Use $(...) notation instead of legacy backticked command substitution

* utils(dep_tree2sql): Double quoting some variables

* utils(dep_tree2sql): Drop database only if exists, supported on postgres 9.2 and up (released september 2012)

* utils(dep_tree2sql): Replace fgrep with grep -F

* utils(dep_tree2sql): Double quoting some variables

* utils(dep_tree2sql): Replace egrep with grep -E

* utils(dep_tree2sql): Use read -r to prevent backslash interpretation

* utils(dep_tree2sql): Use 'find .. -print0 | xargs -0 ..' to allow non-alphanumeric filenames

* utils(dep_tree2sql): Replace `find *` with `find ./*`

This has a change in the paths returned, as they start with ./ instead

* utils(dep_tree2sql): Replace find -perm +111 with find -perm /a+x

-perm +mode was deprecated in favor of -perm /mode. It checks if any of the permissions bits in mode is set. It is thus replaced by the equivalent check of "any file that has the executable bit set for anybody"

* utils(dep_tree2sql): Use mapfile to assign libs variable with an array, that can be properly quoted in the call below

* utils(dep_tree2sql): Uniformize shell script formatting

Using `shfmt -w -s -i 4 -ci -bn -sr`

* Convert mixed tab/spaces indentation to spaces only

* utils(dep_tree2sql): Remove temporary directory at end of script



To unsubscribe from these emails, change your notification settings at https://github.com/OSGeo/grass/settings/notifications


More information about the grass-commit mailing list