mysql50-server not starting correctly on FreeBSD 6.1-PRERELEASE (ldconfig, rcorder)

[John Nielsen]

I installed mysql50-server from an up-to-date ports tree on a new server I'm 
setting up running FreeBSD 6.1.  The port adds the appropriate paths to 
ldconfig, but the startup script does not require ldconfig.  This was causing 
mysqld not to start up correctly since on my system the mysql-server.sh 
script was being run before /etc/rc.d/ldconfig.  The mysql script failed with 
errors about libraries being unavailable, etc.  Running the mysql-server.sh 
script again once the system is up works fine.

I'm not entirely conversant with the new rc script syntax, but I tried adding 
"ldconfig" to the REQUIRES line of mysql-server.sh and that seems to have 
fixed the problem.

I now have two questions:

1) Is this a correct fix?

2) If so, should I file a PR with a patch or can someone just get this 
committed?

Thanks,

JN