noticed that you had some dupes in defines.h here is my working ones
// ===== Added for Freetronics LCD Shield =====
#define PIN_LCD_RS 8 // LCD rs pin – default = 19
#define PIN_LCD_EN 9 // LCD enable pin – default = 18
#define PIN_LCD_D4 2 // LCD d4 pin – default = 20 -DKP- or D2 for the LCD shield stacked on a net card that uses D4 for the SDcard
#define PIN_LCD_D5 5 // LCD d5 pin – default = 21
#define PIN_LCD_D6 6 // LCD d6 pin – default = 22
#define PIN_LCD_D7 7 // LCD d7 pin – default = 23
#define PIN_LCD_RS 8 // LCD rs pin – default = 19
//#define PIN_LCD_BACKLIGHT 10 // LCD backlight pin – default = 12 -DKP- net shield uses D10 as well. so I have backlight always on
#define PIN_LCD_CONTRAST 36 // LCD contrast pin – default = 13
#define PIN_ETHER_CS 37 // Ethernet controller chip select pin – default = 4
#define PIN_SD_CS 38 // SD card chip select pin – default = 0
#define PIN_RAINSENSOR 39 // rain sensor is connected to pin D3 – default = 11
#define BUTTON_ADC_PIN A0 // A0 is the button ADC input