cvs commit: ports/news/inn Makefile

Kris Kennaway kris at obsecurity.org
Fri Aug 25 20:19:19 UTC 2006


On Wed, Aug 23, 2006 at 08:57:28PM +0000, Shaun Amott wrote:
> shaun       2006-08-23 20:57:28 UTC
> 
>   FreeBSD ports repository
> 
>   Modified files:
>     news/inn             Makefile 
>   Log:
>   Amend hostname check to succeed if /bin/hostname fails.
>   Another work-around for those pesky amd64 machines.

This isn't going to work as you intend since the IGNORE check is run
on pointyhat to decide whether to build the ports, which naturally
succeeds.

The real issue here is not 'amd64 machines' but ports built in a jail
without access to a DNS server (this will become the default in the
near future).

Why does this port need to look up its hostname during build?  If it
is hard-coding the result in the package, then it is clearly wrong.
On the other hand if it not hard-coding the result then it begs the
question of why do it at all.

Kris

> |  PORTNAME=	inn
> | @@ -48,12 +48,13 @@ CONFIGURE_ARGS+=	--mandir=${PREFIX}/man 
> |  # Attempt to avoid failures later on, when
> |  # makedbz is executed: we need a valid hostname.
> |  
> | -.if exists(/bin/echo) && exists(/bin/hostname)
> | -HOSTNAME!=	`/bin/echo /bin/hostname`
> | +.if exists(/bin/hostname)
> | +HOSTNAME!=	/bin/hostname
> | +.  if ${HOSTNAME} != ""
> |  HOSTNAME:=	${HOSTNAME:C/[^\.]//g}
> | -
> | -.  if ${HOSTNAME} == ""
> | +.    if ${HOSTNAME} == ""
> |  IGNORE=		requires a fully-qualified hostname in order to build
> | +.    endif
> |  .  endif
> |  .endif
> |  
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-ports/attachments/20060825/e345cb77/attachment.pgp


More information about the cvs-ports mailing list