[so] [Tema1][General] Eroare checker coding style

Mihai Barbulescu b12mihai at gmail.com
Fri Mar 10 13:03:13 EET 2017


2017-03-10 9:33 GMT+02:00 Mihai Barbulescu <b12mihai at gmail.com>:
> 2017-03-10 8:42 GMT+02:00 Andrei-Costin SIMION (66952) via so
> <so at cursuri.cs.pub.ro>:
>>
>>
>> Am incercat ambele metode.
>>
>>
>>
>> Grep nu returneaza nimic, deci se pare ca e ok. La fel si cu notepad++, nu
>> modifica efectiv ceva.
>>
>>
>>
>> Cu toate acestea, checkerul tot nu trece pentru headerul hash_table.h...
>
> Tu ai tema actualizata pe vmchecker? E vorba de andrei.simion0405,
> care e urcata doar pe windows?
>
> Da-ne acces (asistentilor, evident) la un repo de gitlab privat sau
> urca cea mai recenta versiune a codului tau pe Linux si pe Windows in
> vmchecker ca sa vedem si noi minunea asta: n-ai trailing spaces si
> totusi checkpatch le raporteaza.
>

A fost descoperita minunea de catre Ioana: tu ti-ai editat fisierele
pe Windows si s-au pastrat DOS_LINE_ENDINGS.
Trebuie fie sa folosesti git sa iti commiti fisierele cu unix line
endings (see this [1]) fie in Notepad++ (sau plugiun de visual studio
daca stii tu, aici e teritoriu necunoscut pt mine :) ) EDIT -> EOL
conversion -> alegi UNIX (LF)

Cel mai sigur este sa rulati pe fisierele voastre si scriptul acesta:
https://raw.githubusercontent.com/torvalds/linux/master/scripts/cleanfile
 -- in cazul in care mai sunt surprize ca la Andrei.

Vom investiga pe partea noastra si daca e bug in checkpatch.pl si veti
fi anuntati daca au avut loc actualizari iar la cei care in continuare
reclama probleme vom resubmite temele.

[1] http://stackoverflow.com/questions/2517190/how-do-i-force-git-to-use-lf-instead-of-crlf-under-windows

-- 
Cu stimă,
Mihai Bărbulescu


More information about the so mailing list