Re: [UPDATE] security/libssh (and new port devel/cmocka)

2014-01-29 Thread Remi Pointel

Le 22-01-2014 01:35, Brad Smith a écrit :

On 21/01/14 7:30 PM, Stuart Henderson wrote:

On 2014/01/21 18:57, Brad Smith wrote:

On 21/01/14 6:07 PM, Stuart Henderson wrote:

On 2014/01/08 14:07, Remi Pointel wrote:

Hi,

this is the diff to update libssh to latest release 0.6.0.

It needs the new port devel/cmocka also attached, for regress 
tests.


please clean up the whitespace at EOL of HOMEPAGE in 
cmocka/Makefile,
and zap PERMIT_PACKAGE_FTP/PERMIT_DISTFILES_FTP - also it looks like 
it

should be SHARED_ONLY (no static library to go with the shared one).


Don't know why you say it looks like there is no static library. 
There

definitely are such libraries in the PLIST.

lib/libssh.a
@lib lib/libssh.so.${LIBssh_VERSION}
lib/libssh_threads.a
@lib lib/libssh_threads.so.${LIBssh_threads_VERSION}


That's cmocka, not libssh


My bad, I thought you were referring to libssh. Anyway,
cmocka has a CMake option WITH_STATIC_LIB that is disabled
by default. That should be enabled.


I modified cmocka to have WITH_STATIC_LIB and UNIT_TESTING, so we have 
now regress tests and the static library.


Are you ok for importing cmocka and for updating libssh to 0.6.0 ?

Cheers,

Remi.

cmocka-0.3.2.tar.gz
Description: GNU Zip compressed data


Re: [UPDATE] security/libssh (and new port devel/cmocka)

2014-01-21 Thread Brad Smith

On 21/01/14 7:30 PM, Stuart Henderson wrote:

On 2014/01/21 18:57, Brad Smith wrote:

On 21/01/14 6:07 PM, Stuart Henderson wrote:

On 2014/01/08 14:07, Remi Pointel wrote:

Hi,

this is the diff to update libssh to latest release 0.6.0.

It needs the new port devel/cmocka also attached, for regress tests.


please clean up the whitespace at EOL of HOMEPAGE in cmocka/Makefile,
and zap PERMIT_PACKAGE_FTP/PERMIT_DISTFILES_FTP - also it looks like it
should be SHARED_ONLY (no static library to go with the shared one).


Don't know why you say it looks like there is no static library. There
definitely are such libraries in the PLIST.

lib/libssh.a
@lib lib/libssh.so.${LIBssh_VERSION}
lib/libssh_threads.a
@lib lib/libssh_threads.so.${LIBssh_threads_VERSION}


That's cmocka, not libssh


My bad, I thought you were referring to libssh. Anyway,
cmocka has a CMake option WITH_STATIC_LIB that is disabled
by default. That should be enabled.


--
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.



Re: [UPDATE] security/libssh (and new port devel/cmocka)

2014-01-21 Thread Stuart Henderson
On 2014/01/21 18:57, Brad Smith wrote:
> On 21/01/14 6:07 PM, Stuart Henderson wrote:
> >On 2014/01/08 14:07, Remi Pointel wrote:
> >>Hi,
> >>
> >>this is the diff to update libssh to latest release 0.6.0.
> >>
> >>It needs the new port devel/cmocka also attached, for regress tests.
> >
> >please clean up the whitespace at EOL of HOMEPAGE in cmocka/Makefile,
> >and zap PERMIT_PACKAGE_FTP/PERMIT_DISTFILES_FTP - also it looks like it
> >should be SHARED_ONLY (no static library to go with the shared one).
> 
> Don't know why you say it looks like there is no static library. There
> definitely are such libraries in the PLIST.
> 
> lib/libssh.a
> @lib lib/libssh.so.${LIBssh_VERSION}
> lib/libssh_threads.a
> @lib lib/libssh_threads.so.${LIBssh_threads_VERSION}

That's cmocka, not libssh



Re: [UPDATE] security/libssh (and new port devel/cmocka)

2014-01-21 Thread Brad Smith

On 21/01/14 6:07 PM, Stuart Henderson wrote:

On 2014/01/08 14:07, Remi Pointel wrote:

Hi,

this is the diff to update libssh to latest release 0.6.0.

It needs the new port devel/cmocka also attached, for regress tests.


please clean up the whitespace at EOL of HOMEPAGE in cmocka/Makefile,
and zap PERMIT_PACKAGE_FTP/PERMIT_DISTFILES_FTP - also it looks like it
should be SHARED_ONLY (no static library to go with the shared one).


Don't know why you say it looks like there is no static library. There
definitely are such libraries in the PLIST.

lib/libssh.a
@lib lib/libssh.so.${LIBssh_VERSION}
lib/libssh_threads.a
@lib lib/libssh_threads.so.${LIBssh_threads_VERSION}

--
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.



Re: [UPDATE] security/libssh (and new port devel/cmocka)

2014-01-21 Thread Stuart Henderson
On 2014/01/08 14:07, Remi Pointel wrote:
> Hi,
> 
> this is the diff to update libssh to latest release 0.6.0.
> 
> It needs the new port devel/cmocka also attached, for regress tests.

please clean up the whitespace at EOL of HOMEPAGE in cmocka/Makefile,
and zap PERMIT_PACKAGE_FTP/PERMIT_DISTFILES_FTP - also it looks like it
should be SHARED_ONLY (no static library to go with the shared one).



Re: [UPDATE] security/libssh (and new port devel/cmocka)

2014-01-18 Thread Remi Pointel
On Wed, 08 Jan 2014 14:07:46 +0100
Remi Pointel  wrote:
> Hi,
> 
> this is the diff to update libssh to latest release 0.6.0.
> 
> It needs the new port devel/cmocka also attached, for regress tests.
 
ping.