Building error in graphics/p5-PerlMagick

["D'Arcy J.M. Cain" <darcy%NetBSD.org@localhost>]

Something looked odd to me so I tried a test and now it is odder.  I
removed fontconfig from my system.  This involved removing many other
dependent packages but I was surprised that ImageMagick was not one of
them.  So then I tried to build the subject package without fontconfig
installed.  Since ImageMagick was already installed it did not build it
and thus not fontconfig.  I had to make this change to
graphics/p5-PerlMagick first.

Index: Makefile
===================================================================
RCS file: /cvsroot/pkgsrc/graphics/p5-PerlMagick/Makefile,v
retrieving revision 1.104
diff -u -r1.104 Makefile
--- Makefile    22 Aug 2014 11:24:36 -0000      1.104
+++ Makefile    17 Sep 2014 10:07:51 -0000
@@ -31,5 +31,6 @@
 
 BUILDLINK_API_DEPENDS.ImageMagick+=    ImageMagick>=${DISTVERSION}
 .include "../../graphics/ImageMagick/buildlink3.mk"
+.include "../../fonts/fontconfig/buildlink3.mk"
 .include "../../lang/perl5/module.mk"
 .include "../../mk/bsd.pkg.mk"

First question, any harm in committing that change?

Second question, when I did pkg_delete on fontconfig, shouldn't it have
failed until ImageMagick was deleted first?  Doesn't including the
fontconfig buildlink file generate a dependency on fontconfig?

-- 
D'Arcy J.M. Cain <darcy%NetBSD.org@localhost>
http://www.NetBSD.org/ IM:darcy%Vex.Net@localhost