ports/151637: Update port: security/maia Much needed updates for Perl and PHP.
ohauer at FreeBSD.org
ohauer at FreeBSD.org
Sun Dec 5 19:32:59 UTC 2010
Synopsis: Update port: security/maia Much needed updates for Perl and PHP.
State-Changed-From-To: open->feedback
State-Changed-By: ohauer
State-Changed-When: Sun Dec 5 19:31:55 UTC 2010
State-Changed-Why:
I'll take it.
Hi Janky Jay,
I've taken a look over your second patch, looks good so far but I have
a view questions and suggestions.
Are those php modules really needed?
For example if I do not use mysql but postgres insted do I need the
module mysql and mysqli?
If this is not the case then this the preferred solution
.if defined(WITH_MYSQL)
USE_MYSQL= yes
USE_PHP+= mysql mysqli
RUN_DEPENDS+= ${SITE_PERL}/${PERL_ARCH}/DBD/mysql.pm:${PORTSDIR}/databases/p5-DBD-mysql
.endif
Same for postgres and others.
As you see in the Makefile snipped before I removed mysql-server
with USE_MYSQL, same for postgres and apache.
Maybe the sql servers are running on a different machine/cluster?
In your first patch you had additional php-filter, is this module not
needed or was it forgotten in the second patch?
It seems you rolled the tar file from the maia 1.0 branch, rev 1499.
Here I have the following suggestion.
Do not use the same name for the sources, use DISTNAME instead.
For example:
svn co https://www.maiamailguard.com/svn/branches/1.0 -r1500 maia-1.0.2a-r1500
Now you can use in Makefile
DISTNAME= ${PORTNAME}-${PORTVERSION}-${SVNREVISION}
and
SVNREVISION= r1500
This way there is no distinfo clash and it helps paranoid users like
me to compair the sources ;)
I've implemented some of the suggested changes, within an update to
svn revision 1500. You can find them here.
Diff to current port
http://people.freebsd.org/~ohauer/diffs/PR_151637-maia_2010-12-05.diff
New resulting port (svn updated to rev 1500)
http://people.freebsd.org/~ohauer/diffs/PR_151637-maia_2010-12-05.shar
Please look into the php modules and if possible build the source tar
as suggested.
--
Regards,
olli
Responsible-Changed-From-To: freebsd-ports-bugs->ohauer
Responsible-Changed-By: ohauer
Responsible-Changed-When: Sun Dec 5 19:31:55 UTC 2010
Responsible-Changed-Why:
http://www.freebsd.org/cgi/query-pr.cgi?pr=151637
More information about the freebsd-ports-bugs
mailing list