Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Git Attributes #7

Open
tajmone opened this issue Nov 6, 2018 · 1 comment
Open

Git Attributes #7

tajmone opened this issue Nov 6, 2018 · 1 comment

Comments

@tajmone
Copy link
Contributor

tajmone commented Nov 6, 2018

Ciao @Silvan87,

ho notato che hai cambiato i settaggi base del .gitattributes da:

* text=auto

a

* text=auto eol=lf

Nella mia ultima PR (#6) ho dovuto cambiarlo e tornare al settaggio orginale perché su Windows è un vero casino lavorare con le EOL native di Linux (ho Git impostato su core.safecrlf=true, per cui non mi lascia neanche fare lo staging dei file con EOL nativa).

A meno che tu non abbia motivi specifici per desiderare che tutti i file di testo vengano trattati con EOL di Linux, dovresti lasciare quel settaggio di base su nativo, e poi al massimo impone la LF su estensioni specifiche.

Ho visto che hai anche messo:

"esempi/Cromwell/Cromwell.html"  binary
"esempi/La casa/La casa.html"    binary

... ma la seconda direttiva non prende per via dello spazio — il file Cromwell.html mi risulta come binario in Git, che non mi fa vedere le modifiche apportate, mentre La casa.html mi risulta come file di testo).

Quello dei nomi con spazi nei file di configurazione Git è un problema noto, le soluzioni possibili sono due (entrambe non perfette):

"esempi/La casa/La?casa.html"    binary
"esempi/La casa/La[[:space:]]casa.html"    binary

Io credo che dichiarare binari i file HTML ha come effetto sgradito di nascondere le modifiche dei contenuti dei file HTML quando Git rileva che sono stati modificati. È vero che essendo la maggior parte dei loro contenuti cifrati in Vigeneré non è che si possa capire molto dal diffing, però non è neanche che appesantiscano il lavoro.

@Silvan87
Copy link
Owner

Silvan87 commented Nov 7, 2018

Ok, ho rimesso impostazioni per EOL meno problematiche per Windows. Invece, il problema degli spazi l'ho risolto considerando tutti gli html negli esempi dei file binari, tanto lì sono tutti autogenerati o copiati dalla cartella sorgente. Non è previsto, né raccomandabile modificarli manualmente.

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

No branches or pull requests

2 participants