andresgongora / synth-shell-greeter

A system status report and your personal ASCII-art logo for your terminal sessions
GNU General Public License v3.0
14 stars 15 forks source link

Error: Unsupported argument after '+' #36

Closed PinyaColada closed 1 year ago

PinyaColada commented 1 year ago

Describe the bug When I open the terminal it prints Error: Unsupported argument after '+' and then it prints my configuration with shell-greeter completely normal

To Reproduce Steps to reproduce the behavior: I just installed and it started like this.

olmari commented 1 year ago

What OS? And(or what shell?

pendenaor commented 1 year ago

Having the same issue on WSL2 + Distrod + Archlinux system with GNU bash, version 5.1.16(1)-release (x86_64-pc-linux-gnu)

Error: Unsupported argument after '+'
                     .                      OS              Arch Linux
                     #                      Kernel          5.10.60.1-microsoft-standard-WSL2
andresgongora commented 1 year ago

Hmmm I'm unable to replicate it. Do you 'bc' installed?

On Wed 26 Oct 2022, 15:46 Sté, @.***> wrote:

Having the same issue on WSL2 + Distrod + Archlinux system

Error: Unsupported argument after '+' . OS Arch Linux

Kernel 5.10.60.1-microsoft-standard-WSL2

— Reply to this email directly, view it on GitHub https://github.com/andresgongora/synth-shell-greeter/issues/36, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC36INRYT4XD7TROTH6NSD3WFEY3RANCNFSM6AAAAAARMWXQ3Q . You are receiving this because you are subscribed to this thread.Message ID: @.***>

pendenaor commented 1 year ago

like that?

> bc --version
bc 1.07.1
PinyaColada commented 1 year ago

Hi, I have arch linux with bash and with kernel 6.0.2

pendenaor commented 1 year ago

find the culprit :smile: in synth-shell-greeter.sh, function getExternalIPv4 (line 1269)

$ drill +time=1 +tries=1 TXT -4 +short o-o.myaddr.l.google.com @ns1.google.com
Error: Unsupported argument after '+'
andresgongora commented 1 year ago

Awesome! I think it's fixed now as of 845e47a. Another solution, while this makes it upstream, is to install dig from the dnsutils package