search

ciaa / firmware_v1

Firmware de la CIAA
http://www.proyecto-ciaa.com.ar
126 stars 120 forks source link

Feature/coding style #439

Open cpantel opened 8 years ago

jcecconi commented 8 years ago

Carlos, lo primero que hago para saber como probarlo sin mirar el makefile es "make help"....y ups...nada aparece al respecto con las reglas que se agregaron. Cuando puedas agregale la ayuda, yo mientras voy probando el PR.

jcecconi commented 8 years ago

En el repo hay un monton de archivos que tienen CRLF y sin embargo al ejecutar make code_sanity en cygwin no me los menciona

$ make code_sanity

Checking for carriage return EOL OK

Checking for trailing spaces

Pero si abro con un editor hexa por ejemplo blinking.c, este archivo SI tiene. Según encontré http://unix.stackexchange.com/questions/214770/grep-doesnt-match-carriage-return-characters , habría que agregar la opción --binary al llamar a egrep. Lo hice a modo de prueba y ahora me arroja lo siguiente

$ make code_sanity

Checking for carriage return FAIL, there are files with DOS EOL (CRLF) examples/adc_dac/src/adc_dac.c examples/baremetal/src/baremetal.c examples/blinking/src/blinking.c examples/blinking_echo/src/blinking_echo.c examples/blinking_lwip/inc/echo.h examples/blinking_lwip/inc/lpc_18xx43xx_emac_config.h examples/blinking_lwip/inc/lwipopts.h examples/blinking_lwip/src/blinking_lwip.c examples/blinking_lwip/src/echo.c examples/blinking_modbus_master/src/blinking_modbus_master.c examples/blinking_modbus_slave/src/blinking_modbus_slave.c examples/blinking_multicore/src/cortexM0/blinking_multicore.c examples/blinking_multicore/src/cortexM4/blinking_multicore.c examples/rtcs_example/src/rtcs_example.c examples/rtos_example/src/rtos_example.c examples/sleeping_example/src/sleeping_example.c modules/posix/inc/ciaaPOSIX_stdio.h modules/posix/inc/ciaaPOSIX_unistd.h modules/posix/src/ciaaPOSIX_stdio.c modules/posix/src/ciaaPOSIX_unistd.c modules/posix/test/mtest/test_block_devices/src/test_block_devices.c modules/posix/test/mtest/test_stdio_stack/src/test_stdio_stack.c modules/systests/blinking_modbus_adc_dac/src/blinking_modbus_adc_dac.c modules/systests/ioboard/src/ioboard.c modules/systests/testmaster/src/modbusComm.c modules/systests/testmaster/src/testmaster.c modules/systests/testslave/src/testslave.c modules/systests/uart_receive/src/uart_receive.c modules/systests/uart_transmit/src/uart_transmit.c make: *\ [Makefile:442: code_sanity] Error 1

jcecconi commented 8 years ago

En el sh que hacen el fix, también debería agregarse el --binary para que encuentre el CRLF. Como frutilla del postre, no está instalado en cygwin para el CIAA IDE SUITE actual el paquete de dos2unix (en un futuro cercano se hace fácilmente).

cpantel commented 7 years ago

Juan,

si tira

Checking for carriage return FAIL, there are files with DOS EOL (CRLF)

es porque ya ha pasado el test anterior, está ok, está hallando terminación de linea de DOS

mcerdeiro commented 7 years ago

can we merge this request?

cpantel commented 7 years ago

Mariano: I would rather prefer to modify it a little bit as told in "[CIAA-Firmware] 5 Pull Requests" but it will be easier and quicker to merge and then fix, go ahead, please.

Carlos Pantelides @dev4sec https://github.com/cpantel http://seguridad-agile.blogspot.com/

  From: Mariano Cerdeiro <notifications@github.com>

To: ciaa/Firmware Firmware@noreply.github.com Cc: Carlos Pantelides carlos_pantelides@yahoo.com; Author author@noreply.github.com Sent: Thursday, October 6, 2016 4:42 PM Subject: Re: [ciaa/Firmware] Feature/coding style (#439)

can we merge this request?— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.