doesn't seem to work with 1.0 beta4 about pubsubclient

[davidmonro]

OK, its a simple fix. Looks like they made Client a pure virtual class and wrapped EthernetClient around it. Hope this is a sensible way to feed in a patch - I have no idea what I am doing with github yet :)

diff -ur ./PubSubClient/PubSubClient.cpp /home/davidm/sketchbook/libraries/PubSubClient/PubSubClient.cpp
--- ./PubSubClient/PubSubClient.cpp 2011-08-05 00:08:43.000000000 +0930
+++ /home/davidm/sketchbook/libraries/PubSubClient/PubSubClient.cpp 2011-09-12 17:34:07.000000000 +0930
@@ -5,7 +5,7 @@
 */

 #include "PubSubClient.h"
-#include "Client.h"
+#include "EthernetClient.h"
 #include "string.h"

 PubSubClient::PubSubClient() : _client() {
diff -ur ./PubSubClient/PubSubClient.h /home/davidm/sketchbook/libraries/PubSubClient/PubSubClient.h
--- ./PubSubClient/PubSubClient.h   2011-08-05 00:08:43.000000000 +0930
+++ /home/davidm/sketchbook/libraries/PubSubClient/PubSubClient.h   2011-09-12 17:34:26.000000000 +0930
@@ -8,7 +8,7 @@
 #define PubSubClient_h

 #include "Ethernet.h"
-#include "Client.h"
+#include "EthernetClient.h"

 #define MAX_PACKET_SIZE 128
 #define KEEPALIVE 15000 // max value = 255000
@@ -33,7 +33,7 @@

 class PubSubClient {
 private:
-   Client _client;
+   EthernetClient _client;
    uint8_t buffer[MAX_PACKET_SIZE];
    uint8_t nextMsgId;
    long lastOutActivity;

[knolleary]

Hi DM,

thanks for that; I hadn't updated to beta4 yet.

Have made those changes, merged them into the master branch and gotten rid of the arduino-1.0 branch. I figure arduino-1.0 is due to be released soon enough that there is no need to keep a separate branch.

(The real reason being I accidentally put these changes into master and decided I couldn't be bothered to revert and put them in the a-1 branch - ho hum).