Skip to content

fix <FONT SIZE="1">#26

Open
PsykeDady wants to merge 8 commits into
LegolasTheElf:masterfrom
PsykeDady:master
Open

fix <FONT SIZE="1">#26
PsykeDady wants to merge 8 commits into
LegolasTheElf:masterfrom
PsykeDady:master

Conversation

@PsykeDady

@PsykeDady PsykeDady commented Jul 22, 2021

Copy link
Copy Markdown

prima del fix mi usciva per ogni santo:
Mannaggia <FONT SIZE="-1"><b>NOMESANTO

@elmazzun

Copy link
Copy Markdown

A me esce ancora ma non per ogni santo:

$ ./mannaggia.sh
Mannaggia
Mannaggia
Mannaggia  <FONT SIZE="-1"><b>Francesco Saverio
Mannaggia
Mannaggia
Mannaggia
Mannaggia  <FONT SIZE="-1"><b>6 Redentori Mercedari
Mannaggia

@PsykeDady

Copy link
Copy Markdown
Author

A me esce ancora ma non per ogni santo:

$ ./mannaggia.sh
Mannaggia
Mannaggia
Mannaggia  <FONT SIZE="-1"><b>Francesco Saverio
Mannaggia
Mannaggia
Mannaggia
Mannaggia  <FONT SIZE="-1"><b>6 Redentori Mercedari
Mannaggia

Ma noto anche che hai dei Mannaggia vuoti ? Mmmm stasera investigo su

Hai l ultima versione che ho pushato vero ?

@elmazzun

Copy link
Copy Markdown

Mannaggia a Sant'Onofrio, ho eseguito lo script senza neanche vedere se ci fossero altri branch oltre al main.
Colpa mia, scusate.

@PsykeDady

PsykeDady commented Jul 28, 2021

Copy link
Copy Markdown
Author

Non credo di aver capito perché io sono sul branch master ahahha ma ok

@elmazzun

Copy link
Copy Markdown

Sono sul branch master, commit 31ee66d0, tramite Git for Windows su Windows 10: un'altra esecuzione dello script stampa:

Mannaggia  <FONT SIZE="-1"><b>Panfilo di Roma
Mannaggia
Mannaggia
Mannaggia
Mannaggia
Mannaggia  <FONT SIZE="-1"><b>Una (Huna)
Mannaggia  <FONT SIZE="-1"><b>Edisto

Nel codice vedo che le stringhe da cancellare sono:

DELSTRING1="</FONT>"
DELSTRING2="</b>"

Forse il master non e' aggiornato come gli altri branch?

@PsykeDady

Copy link
Copy Markdown
Author

Fai una cosa, clona direttamente dal mio Fork perché non hanno approvato la PR

@PsykeDady

PsykeDady commented Aug 3, 2021

Copy link
Copy Markdown
Author

Riguardo il merge con la PR #27 io per farla funzionare devo rimuovere il flag -ao alsa (funzione say riga 40) dal comando di mplayer per farlo funzionare

Non so se è un mio problema quindi non l'ho pushato, in caso quando e se verrà approvata la PR ne aprirò un altra per correggere questa cosa

EDIT: rimossa con il commit 9f9c5d2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants