Feature/coding style

[cpantel]

No description provided.

[jcecconi]

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]

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]

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]

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]

can we merge this request?

[cpantel]

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.