pkgsrc-Bugs archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

pkg/40100: devel/doxygen/Makefile mishandles Darwin case sensitivity



>Number:         40100
>Category:       pkg
>Synopsis:       devel/doxygen/Makefile mishandles Darwin case sensitivity
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu Dec 04 00:15:01 +0000 2008
>Originator:     Tom Yu
>Release:        pkgsrc trunk 20081201
>Organization:
>Environment:
Darwin SPARK-GAP.MIT.EDU 9.5.0 Darwin Kernel Version 9.5.0: Wed Sep  3 11:29:43 
PDT 2008; root:xnu-1228.7.58~1/RELEASE_I386 i386


>Description:
The doxygen package acts as if it had an inconsistent PLIST.  I am using a UFS 
partition to build packages from pkgsrc.  While the UFS partition is 
case-sensitive, doxygen defaults to treating any Mac OS as case-insensitive, so 
the test for whether the lowercase file "./descr" exists incorrectly determines 
that the PLIST should have filenames with uppercase letters.
>How-To-Repeat:
Attempt to build a package for devel/doxygen on Mac OS X 10.5 using a UFS 
partition for pkgsrc.
>Fix:
Index: devel/doxygen/Makefile
===================================================================
RCS file: /cvsroot/pkgsrc/devel/doxygen/Makefile,v
retrieving revision 1.64
diff -u -r1.64 Makefile
--- devel/doxygen/Makefile      11 Nov 2008 00:44:12 -0000      1.64
+++ devel/doxygen/Makefile      4 Dec 2008 00:06:58 -0000
@@ -29,12 +29,6 @@
 
 PLIST_VARS+=           ci cs
 
-.if exists(./descr)
-PLIST.ci=              yes     # case-insensitive
-.else
-PLIST.cs=              yes     # case-sensitive
-.endif
-
 SUBST_CLASSES+=                pybin
 SUBST_STAGE.pybin=     pre-configure
 SUBST_FILES.pybin=     doc/Makefile.in
@@ -54,6 +48,12 @@
 CONFIGURE_ARGS+=       -platform freebsd-g++
 .endif
 
+.if ${OPSYS} == "Darwin"
+PLIST.ci=              yes     # case-insensitive
+.else
+PLIST.cs=              yes     # case-sensitive
+.endif
+
 post-patch:
        touch ${WRKSRC}/src/doxytag.l
 




Home | Main Index | Thread Index | Old Index