CVS commit: pkgsrc/geography/mapserver

["Greg Troxel" <gdt%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   gdt
Date:           Wed Aug 16 14:43:51 UTC 2023

Modified Files:
        pkgsrc/geography/mapserver: options.mk

Log Message:
geography/mapserver: Question postgis bl3

Postgis operates within pgsql, so you use it via sql, not by linking.
It can make sense to DEPEND on it, but bl3 seems irregular.

Note that mapserver was not revbumped on postgis update because I
don't believe it really depends on it this way and thus any ABI change
is not really a change to the ABI used by mapserver.


To generate a diff of this commit:
cvs rdiff -u -r1.5 -r1.6 pkgsrc/geography/mapserver/options.mk

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/geography/mapserver/options.mk
diff -u pkgsrc/geography/mapserver/options.mk:1.5 pkgsrc/geography/mapserver/options.mk:1.6
--- pkgsrc/geography/mapserver/options.mk:1.5   Thu Jul  7 08:57:06 2016
+++ pkgsrc/geography/mapserver/options.mk       Wed Aug 16 14:43:50 2023
@@ -1,4 +1,4 @@
-# $NetBSD: options.mk,v 1.5 2016/07/07 08:57:06 wiz Exp $
+# $NetBSD: options.mk,v 1.6 2023/08/16 14:43:50 gdt Exp $
 
 PKG_OPTIONS_VAR=       PKG_OPTIONS.mapserver
 PKG_SUPPORTED_OPTIONS= fastcgi pgsql mysql x11
@@ -18,6 +18,8 @@ CONFIGURE_ARGS+=      --with-fastcgi
 # PostgreSQL/PostGIS support
 #
 .if !empty(PKG_OPTIONS:Mpgsql)
+# \todo Explain why this is bl3 rather than DEPENDS.  It doesn't make
+# sense, given how postgis works.
 .include "../../databases/postgresql-postgis2/buildlink3.mk"
 CONFIGURE_ARGS+=       --with-postgis
 .endif