chaotic-aur / toolbox

Unified kit with all the scripts required for maintaining the repository 🧰
https://aur.chaotic.cx
GNU Affero General Public License v3.0
81 stars 11 forks source link

chore: don't silence telegram notifications output to gather logs #97

Closed dr460nf1r3 closed 5 months ago

dr460nf1r3 commented 9 months ago

To find out why the user part of the deploy logs isn't working anymore after updating telegram-send.

https://github.com/rahiel/telegram-send/issues/109#issuecomment-1784163104

xiota commented 9 months ago

Does line 113 also need to be removed / commented out?

  [[ "$CAUR_SILENT" == '1' ]] && return 0
PedroHLC commented 9 months ago

I don't think this should be merged. Can't you edit this files in its final place, in said builder where you want to debug this?

dr460nf1r3 commented 9 months ago

Does line 113 also need to be removed / commented out?

  [[ "$CAUR_SILENT" == '1' ]] && return 0

Shouldn't matter in our case, since we run CAUR_SILENT=0 by default (https://github.com/chaotic-aur/toolbox/blob/main/src/chaotic.sh#L51).

dr460nf1r3 commented 9 months ago

I don't think this should be merged. Can't you edit this files in its final place, in said builder where you want to debug this?

As you might know, we use NixOS to build the system configuration, so files are immutable 🤔

PedroHLC commented 9 months ago

As you might know, we use NixOS to build the system configuration, so files are immutable 🤔

Good time to learn patches + fetchpatch

dr460nf1r3 commented 9 months ago

I didn't think about this possibility! Giving it a try soon 😁