Pantek Library
Hosting Provided By
CybrHost
High Speed Hosting

Re: 2 new ports: LinPopUp and CDK

From: Jolan Luff <jolan(at)cryptonomicon.org>
Date: Fri Mar 14 2003 - 17:01:29 EST

On Fri, Mar 14, 2003 at 08:55:06AM +0000, Jorge Cortes wrote:
> http://www.openbsd.org.mx/~jcortes/ports/LinPopUp-1.2.0.tgz

I highly recommended adding:

PKGNAME= ${DISTNAME:L} I think HOMEPAGE should be http://www.littleigloo.org/software_002.php3.

Put # GPL above the PERMIT_* lines.

It doesn't respect CFLAGS or CC.

This definitely needs to be fixed, as pvalchev@ just went through the tree fixing them all:

execute.c: In function `execute_program':
execute.c:46: warning: couldn't find null pointer sentinel value starting at 2
execute.c:46: warning: (integer 0 is not a null pointer in varargs context)

This should probably be fixed too:

Do you need help?X

send.o: In function `send_send':
send.o(.text+0x690): warning: tmpnam() possibly used unsafely; consider using mkstemp()

I noticed that the files are dated 3 years ago.. is this no longer actively maintained? :(

> http://www.openbsd.org.mx/~jcortes/ports/cdk-4.9.10.tgz

DISTNAME= cdk

This is bad. When they release a new version, the checksum will fail. You should try to get them to version the distfile, or mirror a versioned copy of the distfile somewhere.

No comment above PERMIT_* lines. It *looks* like a BSD-style license.

Doesn't respect CC.

This also looks like it isn't actively maintained :( Maybe DISTNAME won't be a problem then... :P

Do you need more help?X

I'm not sure if installing all the example program source in share/examples is needed (or desired).

  • jolan
Received on Fri Mar 14 17:02:16 2003

This archive was generated by hypermail 2.1.8 : Wed Aug 23 2006 - 13:43:41 EDT


Contact Us  Legal Notices  Order Services Online 
Pantek Home  Privacy Policy  IT news  Site Map  Pantek Library