Spent the last hour manually making each change in a diff between 0.6.4 (which worked for me) and 0.6.7 (which doesn't), and noticed something important. The -h code uses a char * as a boolean variable to see if it is there or not, but should be using the first character in the string. Changed that, and got it working on MediaOne (my cable modem provider). Diff as follows: diff pump-0.6.7/pump.c pump-mine/pump.c 1171c1171 < cmd.u.start.reqHostname ? --- > cmd.u.start.reqHostname[0] ? Before what would happen is pump would sit there for about 15 seconds before finally stating "Operation failed".
(and just to beat breed.com to the punch...) The patch at http://www.cse.ucsc.edu/~breed/pump.patch does NOT fix this :P.
Ugh. I just committed this to CVS -- would you like to test the new version for me before we stick it on the errata? ------- Additional Comments From 06/22/99 15:51 ------- Sure, give me an address of some sort to get access to it and I will test it (don't know where the pump development goes on so don't know where to get the CVS version, or do you mean the redhat CVS? well, regardless...).
Sure, give me an address of some sort to get access to it and I will test it (don't know where the pump development goes on so don't know where to get the CVS version, or do you mean the redhat CVS? well, regardless...).