multimedia/mplayer missing dependencies

[Frédéric Fauberteau <triaxx%NetBSD.org@localhost>]

I rebuilt multimedia/mplayer and got this kind of errors:

===> Creating toolchain wrappers for mplayer-1.3.0
ERROR: libXtst>=1.0.1 is not installed; can't buildlink files.

I supposed missing dependencies and resolved it with:

Index: Makefile
===================================================================
RCS file: /cvsroot/pkgsrc/multimedia/mplayer/Makefile,v
retrieving revision 1.100
diff -r1.100 Makefile
27a28,29
> .include "../../devel/libgetopt/buildlink3.mk"
29a32
> .include "../../x11/libXtst/buildlink3.mk"
30a34
> .include "../../x11/libXrender/buildlink3.mk"
36a41
> .include "../../graphics/glu/buildlink3.mk"

Is it ok or misunderstanding?

If ok, where are the right place for these dependencies? 
mplayer/Makefile or mplayer-share/Makefile.depends?