[libssh2] [PROPOSAL] add some more version defines

2007-07-14 Thread Guenter Knauf
Hi all, I want to suggest this (shamlessly stolen from libcurl, hehe): --- libssh2.h.orig Tue Jul 10 00:47:32 2007 +++ libssh2.h Sat Jul 14 21:44:32 2007 @@ -79,7 +79,31 @@ typedef long long libssh2_int64_t; #endif --- libssh2.h.orig Sat Jul 14 21:53:48 2007 +++ libssh2.h Sat Jul

Re: [libssh2] [libssh2-cvs] jehousley: libssh2/include libssh2.h, 1.76, 1.77

2007-07-14 Thread Daniel Stenberg
On Sat, 14 Jul 2007, CVS commit messages for libssh2 wrote: > For now make LIBSSH2_APINO a long, some compilers make constants an int it > seems. [...] > -#define LIBSSH2_APINO 200706151200 > +#define LIBSSH2_APINO 200706151200L On cur

Re: [libssh2] [libssh2-cvs] bagder: libssh2/example/simple scp_write.c, 1.1, 1.2 scp_write_nonblock.c, 1.3, 1.4 sftp_write.c, 1.4, 1.5 ssh2.c, 1.3, 1.4

2007-07-14 Thread Daniel Stenberg
On Sat, 14 Jul 2007, CVS commit messages for libssh2 wrote: > -hostaddr = inet_addr(argv[1]); > + hostaddr = inet_addr(argv[1]); Oops, my (emacs) macro made this into a TAB instead of space since that was what we once used. Did we agree on using spaces instead or where did we end up

Re: [libssh2] [libssh2-cvs] bagder: libssh2/example/simple scp_write.c, 1.1, 1.2 scp_write_nonblock.c, 1.3, 1.4 sftp_write.c, 1.4, 1.5 ssh2.c, 1.3, 1.4

2007-07-14 Thread James Housley
On Jul 14, 2007, at 5:01 PM, Daniel Stenberg wrote: > On Sat, 14 Jul 2007, CVS commit messages for libssh2 wrote: > >> -hostaddr = inet_addr(argv[1]); >> +hostaddr = inet_addr(argv[1]); > > Oops, my (emacs) macro made this into a TAB instead of space since > that was > what we once u

Re: [libssh2] [PROPOSAL] add some more version defines

2007-07-14 Thread James Housley
On Jul 14, 2007, at 3:51 PM, Guenter Knauf wrote: > Hi all, > I want to suggest this (shamlessly stolen from libcurl, hehe): > > --- libssh2.h.origTue Jul 10 00:47:32 2007 > +++ libssh2.h Sat Jul 14 21:44:32 2007 > @@ -79,7 +79,31 @@ > typedef long long libssh2_int64_t; > #endif > > --- libs

Re: [libssh2] [libssh2-cvs] jehousley: libssh2/include libssh2.h, 1.76, 1.77

2007-07-14 Thread James Housley
On Jul 14, 2007, at 4:04 PM, Daniel Stenberg wrote: > On Sat, 14 Jul 2007, CVS commit messages for libssh2 wrote: > >> For now make LIBSSH2_APINO a long, some compilers make constants >> an int it >> seems. > > [...] > >> -#define LIBSSH2_APINO 200706151200 >> +#def

Re: [libssh2] [PROPOSAL] add some more version defines

2007-07-14 Thread Daniel Stenberg
On Sat, 14 Jul 2007, Guenter Knauf wrote: > I want to suggest this (shamlessly stolen from libcurl, hehe): Perhaps not too suprisingly, I'm in favour! ;-) - This SF.net email is sponsored by DB2 Express Download DB2 Express

Re: [libssh2] [libssh2-cvs] bagder: libssh2 HACKING, NONE, 1.1 Makefile.am, 1.11, 1.12

2007-07-14 Thread James Housley
On Jul 14, 2007, at 5:28 PM, CVS commit messages for libssh2 wrote: > Update of /cvsroot/libssh2/libssh2 > In directory sc8-pr-cvs6.sourceforge.net:/tmp/cvs-serv5605 > > Modified Files: > Makefile.am > Added Files: > HACKING > Log Message: > Metion the current code style in the new fil

Re: [libssh2] [libssh2-cvs] bagder: libssh2 HACKING, NONE, 1.1 Makefile.am, 1.11, 1.12

2007-07-14 Thread Daniel Stenberg
On Sat, 14 Jul 2007, James Housley wrote: >> - keep source lines shorter than 80 columns > All of the work I did all the code is less then 132 columns. > > I tried to keep it as close to 80 as was reasonable without breaking the > lines in too many places so that the code became unreadable in,

Re: [libssh2] [libssh2-cvs] bagder: libssh2/example/simplescp_write.c, 1.1, 1.2 scp_write_nonblock.c, 1.3, 1.4 sftp_write.c, 1.4, 1.5 ssh2.c, 1.3, 1.4

2007-07-14 Thread Guenter Knauf
> On Sat, 14 Jul 2007, CVS commit messages for libssh2 wrote: >> -hostaddr = inet_addr(argv[1]); >> +hostaddr = inet_addr(argv[1]); > Oops, my (emacs) macro made this into a TAB instead of space since that > was > what we once used. Did we agree on using spaces instead or where did we

Re: [libssh2] [libssh2-cvs] jehousley: libssh2/include libssh2.h, 1.76, 1.77

2007-07-14 Thread Guenter Knauf
>> On current 32 bit systems, ints and longs are the same size: 32 bit. >> 200706151200 is a value that doesn't fit in 32 bits, be it int or >> long. >> >> 2^32 is 4294967296, and this APINO define is almost 47 times larger... > You are right, it would need to be a LONG LONG, which is not fully >

Re: [libssh2] bagder: libssh2 HACKING, NONE, 1.1 Makefile.am, 1.11, 1.12

2007-07-14 Thread James Housley
On Jul 14, 2007, at 5:38 PM, Daniel Stenberg wrote: > On Sat, 14 Jul 2007, James Housley wrote: > >>> - keep source lines shorter than 80 columns > >> All of the work I did all the code is less then 132 columns. >> >> I tried to keep it as close to 80 as was reasonable without >> breaking the

Re: [libssh2] [libssh2-cvs] jehousley: libssh2/include libssh2.h, 1.76, 1.77

2007-07-14 Thread James Housley
On Jul 14, 2007, at 5:48 PM, Guenter Knauf wrote: >>> On current 32 bit systems, ints and longs are the same size: 32 bit. >>> 200706151200 is a value that doesn't fit in 32 bits, be it int or >>> long. >>> >>> 2^32 is 4294967296, and this APINO define is almost 47 times >>> larger... > >> You a

[libssh2] 'make dist' is broken

2007-07-14 Thread Daniel Stenberg
Hey I just noticed that 'make dist' broke sometime after July 9 as both mine and Simon's automated snapshots are no longer made due to that... (and I'm off to bed) - This SF.net email is sponsored by DB2 Express Download DB

Re: [libssh2] [libssh2-cvs] bagder: libssh2 HACKING, NONE, 1.1 Makefile.am, 1.11, 1.12

2007-07-14 Thread Dan Fandrich
On Sat, Jul 14, 2007 at 05:34:40PM -0400, James Housley wrote: > On Jul 14, 2007, at 5:28 PM, CVS commit messages for libssh2 wrote: > > Metion the current code style in the new file called HACKING. In > > this, we > > could also add descriptions on how to send us patches and bug > > reports et