Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 30 Oct 2003 12:03:58 +0100
From:      Oliver Eikemeier <eikemeier@fillmore-labs.com>
To:        Trevor Johnson <trevor@jpj.net>
Cc:        Clement Laforet <sheepkiller@cultdeadsheep.org>
Subject:   Re: cvs commit: ports/www/apache2
Message-ID:  <3FA0F01E.5040107@fillmore-labs.com>
In-Reply-To: <20031029233231.V7442@blues.jpj.net>
References:  <200310292246.h9TMkjSI062232@repoman.freebsd.org> <3FA05A25.2020408@fillmore-labs.com> <20031030013046.26bb944c.sheepkiller@cultdeadsheep.org> <20031029233231.V7442@blues.jpj.net>

next in thread | previous in thread | raw e-mail | index | archive | help
Trevor Johnson wrote:

> Clement Laforet wrote:
> 
>>Oliver Eikemeier <eikemeier@fillmore-labs.com> wrote:
>>
>>>WITH_HTTP_PORT, WITHOUT_CGI, WITHOUT_ICONS wouldn't be that bad...
> 
> [...]
> 
>>You're right... I'll correct it with PORTREVISION 1, with future changes
>>in Makefile.modules.
> 
> I had thought of the same thing, but I was in a hurry.  How's this:
> <URL:http://people.freebsd.org/~trevor/ports/apache2-cosmetic.diff>?

I really don't want to be portlint, but hey, you asked for it:

> +CONFLICTS=	apache+ipv6-1.* \
> +		apache+modssl-1.* \
> +		apache+ssl-1.* \
> +		apache-1.* \
> +		apache_fp-1.* \
> +		caudium-devel-1.* \
> +		caudium10-1.* \
> +		caudium12-* \
> +		ru-apache+mod_ssl-1.* \
> +		ru-apache-1.* \
> +		thttpd-2.*
> +

I can't find any conflicts with the caudium* ports, and www/mini_httpd
is missing. How about:

+CONFLICTS=	apache+ipv6-1.* \
+		apache+modssl-1.* \
+		apache+ssl-1.* \
+		apache-1.* \
+		apache_fp-1.* \
+		mini_httpd-1.* \
+		ru-apache+mod_ssl-1.* \
+		ru-apache-1.* \
+		thttpd-2.*


> +#	WITH_PORT                default: 80

I'm fine with that, even though WITH_HTTP_PORT seems more appealing to me.


> +#	WITHOUT_ERROR:            Do not install www/error

I would make this the default ;-) Honestly: there must be a better name
for it. Do we need that many options?


> .if defined(WITH_LDAP)
> USE_LDAP=	YES

That doesn't seem to work, it's USE_OPENLDAP as far as I can remember.


> +	@${ECHO}
> +	@${ECHO_MSG} "*** To see all available knobs, type:  make show-options"
> +	@${ECHO}
Mixing ECHO and ECHO_MSG is strange, how about using ECHO here?


> show-options:
> 	@${EGREP} '^##' ${.CURDIR}/Makefile | ${SED} 's/##//'

Not that I particularly like this, but how about using:
	@${SED} -ne 's/^##//p' ${.CURDIR}/Makefile




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3FA0F01E.5040107>