From 8f5fdeee9477a5a774b4ff95c1a424e09d602567 Mon Sep 17 00:00:00 2001 From: Leya Wehner Date: Fri, 6 Dec 2024 13:47:35 +0100 Subject: [PATCH] chore: Remove the AlwaysShowNews option (#288) The AlwaysShowNews was only introduced to keep compatibility with the historic & initial Arch-Update's behavior of always showing news on upgrade (regardless of whether there's a new one since the last run or not). I think it's fair to say that this was a poor design that just created undesired annoyances to people. As such, I'm not sure the AlwaysShowNews option is really desired / used by anyone and therefore could be deprecated & removed altogether. Closes https://github.com/Antiz96/arch-update/issues/284 --------- Co-authored-by: Robin Candau --- README-fr.md | 1 - README.md | 1 - doc/man/arch-update.conf.5 | 4 ---- doc/man/fr/arch-update.conf.5 | 4 ---- res/config/arch-update.conf.example | 1 - src/lib/config.sh | 4 ---- 6 files changed, 15 deletions(-) diff --git a/README-fr.md b/README-fr.md index aaabf8d..350753a 100644 --- a/README-fr.md +++ b/README-fr.md @@ -237,7 +237,6 @@ Les options prises en charge sont : - NoColor # Ne pas coloriser la sortie. - NoVersion # Ne pas afficher les modifications de versions des paquets lors du listing des mises à jour en attente (y compris lors de l'utilisation de l'option `-l / --list`). -- AlwaysShowNews # Toujours afficher les Arch news avant de mettre à jour, peu importe s'il y en a une nouvelle depuis la dernière exécution ou non. - NewsNum=[Num] # Nombre de Arch news à affcher avant la mise à jour et avec l'option `-n / --news` (voir la page de manuel arch-update(1) pour plus de details). La valeur par défaut est 5. - AURHelper=[AUR Helper] # AUR helper à utiliser pour la prise en charge des paquets AUR. Les valeurs valides sont `paru`, `yay` ou `pikaur`. Si cette option n'est pas spécifiée, Arch-Update utilisera le premier AUR helper disponible dans l'ordre suivant : `paru` puis `yay` puis `pikaur` (si aucun d'eux n'est installé, Arch-Update ne prendra pas en compte les paquets AUR). - PrivilegeElevationCommand=[Cmd] # Commande à utiliser pour l'élévation de privilège. Les valeurs valides sont `sudo`, `doas` ou `run0`. Si cette option n'est pas spécifiée, Arch-Update utilisera la première commande disponible dans l'odre suivant : `sudo`, `doas` puis `run0`. diff --git a/README.md b/README.md index 88cdbe5..7ec572f 100644 --- a/README.md +++ b/README.md @@ -237,7 +237,6 @@ The supported options are: - NoColor # Do not colorize output. - NoVersion # Do not show versions changes for packages when listing pending updates (including when using the `-l / --list` option). -- AlwaysShowNews # Always display Arch news before updating, regardless of whether there's a new one since the last run or not. - NewsNum=[Num] # Number of Arch news to display before updating and with the `-n / --news` option (see the arch-update(1) man page for more details). Defaults to 5. - AURHelper=[AUR Helper] # AUR helper to be used for AUR packages support. Valid values are `paru`, `yay` or `pikaur`. If this option is not set, Arch-Update will use the first available AUR helper in the following order: `paru` then `yay` then `pikaur` (in case none of them is installed, Arch-Update will not take AUR packages into account). - PrivilegeElevationCommand=[Cmd] # Command to be used for privilege elevation. Valid values are `sudo`, `doas` or `run0`. If this option is not set, Arch-Update will use the first available command in the following order: `sudo`, `doas` then `run0`. diff --git a/doc/man/arch-update.conf.5 b/doc/man/arch-update.conf.5 index a5eb76e..d3ed62b 100644 --- a/doc/man/arch-update.conf.5 +++ b/doc/man/arch-update.conf.5 @@ -27,10 +27,6 @@ Do not colorize output. .B NoVersion .RB "Do not show versions changes for packages when listing pending updates (including when using the " "-l / --list " "option, see the " "arch-update(1) " "man page for more details)." -.TP -.B AlwaysShowNews -Always display Arch news before updating, regardless of whether there's a new one since the last run or not. - .TP .B NewsNum=[Num] .RB "Number of Arch news to display before updating and with the " "-n / --news " "option (see the " "arch-update(1) " "man page for more details). Defaults to 5." diff --git a/doc/man/fr/arch-update.conf.5 b/doc/man/fr/arch-update.conf.5 index b493984..59fda7c 100644 --- a/doc/man/fr/arch-update.conf.5 +++ b/doc/man/fr/arch-update.conf.5 @@ -27,10 +27,6 @@ Ne pas coloriser la sortie. .B NoVersion .RB "Ne pas afficher les modifications de versions des paquets lors du listing des mises à jour en attente (y compris lors de l'utilisation de l'option " "-l / --list" ", voir la page de man " "arch-update(1) " "pour plus de détails)." -.TP -.B AlwaysShowNews -Toujours afficher les Arch news avant de mettre à jour, peu importe s'il y en a une nouvelle depuis la dernière exécution ou non. - .TP .B NewsNum=[Num] .RB "Nombre de Arch news à afficher avant la mise à jour et avec l'option " "-n / --news " "(voir la page de man " "arch-update(1) " "pour plus de détails). La valeur par défaut est 5." diff --git a/res/config/arch-update.conf.example b/res/config/arch-update.conf.example index 9af6ba8..6b0f530 100644 --- a/res/config/arch-update.conf.example +++ b/res/config/arch-update.conf.example @@ -4,7 +4,6 @@ #NoColor #NoVersion -#AlwaysShowNews #NewsNum=5 #AURHelper=paru #PrivilegeElevationCommand=sudo diff --git a/src/lib/config.sh b/src/lib/config.sh index 62eb624..54004e5 100755 --- a/src/lib/config.sh +++ b/src/lib/config.sh @@ -18,10 +18,6 @@ if [ -f "${config_file}" ]; then # shellcheck disable=SC2034 no_version=$(grep -Eq '^[[:space:]]*NoVersion[[:space:]]*$' "${config_file}" 2> /dev/null && echo "true") - # Check the "AlwaysShowNews" option in arch-update.conf - # shellcheck disable=SC2034 - show_news=$(grep -Eq '^[[:space:]]*AlwaysShowNews[[:space:]]*$' "${config_file}" 2> /dev/null && echo "true") - # Check the "NewsNum" option in arch-update.conf # shellcheck disable=SC2034 news_num=$(grep -E '^[[:space:]]*NewsNum[[:space:]]*=[[:space:]]*[1-9][0-9]*[[:space:]]*$' "${config_file}" 2> /dev/null | awk -F '=' '{print $2}' | tr -d '[:space:]')