Jeremie Courreges-Anglas <j...@wxcvbn.org> writes:
> SASANO Takayoshi <u...@mx5.nisiq.net> writes:
>
>> Hello,
>>
>> here is the port of xorriso, iso9660 file manipulator.
>>
>> For my mistake, I already imported this to CVS tree.
>> Sorry and if there is serious problem please remove it on CVS.
>
> No need to remove it IMO.
>
> The port looks good. Some tweaks:
> - no need to set REVISION for a new port. Since xorriso is currently
> unhooked, we can safely kill it.
> - patch-configure can go away if we tweak CPPFLAGS and LDFLAGS through
> the environment.
> - NO_TEST should only be used if tests fail because there are no tests.
> This is not the case here.
> - I suggest using upstream's description for DESCR
>
> With the diff below committed, ok jca@ to hook up xorriso in
> sysutils/Makefile.
Here's a refreshed diff.
Index: Makefile
===================================================================
RCS file: /d/cvs/ports/sysutils/xorriso/Makefile,v
retrieving revision 1.2
diff -u -p -r1.2 Makefile
--- Makefile 1 Jul 2017 15:13:10 -0000 1.2
+++ Makefile 1 Jul 2017 17:08:02 -0000
@@ -3,7 +3,6 @@
COMMENT = ISO 9660 Rock Ridge Filesystem Manipulator
DISTNAME = xorriso-1.4.6
-REVISION = 0
CATEGORIES = sysutils
@@ -20,8 +19,8 @@ LIB_DEPENDS = archivers/bzip2 \
converters/libiconv
CONFIGURE_STYLE = gnu
-
-NO_TEST = Yes
+CONFIGURE_ENV = CPPFLAGS="-I${LOCALBASE}/include" \
+ LDFLAGS="-L${LOCALBASE}/lib ${LDFLAGS}"
pre-configure:
${SUBST_CMD} ${WRKSRC}/configure
Index: patches/patch-configure
===================================================================
RCS file: patches/patch-configure
diff -N patches/patch-configure
--- patches/patch-configure 30 Jun 2017 13:26:43 -0000 1.1.1.1
+++ /dev/null 1 Jan 1970 00:00:00 -0000
@@ -1,14 +0,0 @@
-$OpenBSD: patch-configure,v 1.1.1.1 2017/06/30 13:26:43 uaa Exp $
---- configure.orig Fri Sep 16 22:51:36 2016
-+++ configure Fri Jun 30 00:29:56 2017
-@@ -2529,6 +2529,10 @@ test -n "$target_alias" &&
-
-
- case $target_os in
-+openbsd*)
-+ LDFLAGS="$LDFLAGS -L/usr/local/lib"
-+ CPPFLAGS="$CPPFLAGS -I/usr/local/include"
-+ ;;
- freebsd*)
- LDFLAGS="$LDFLAGS -L/usr/local/lib"
- CPPFLAGS="$CPPFLAGS -I/usr/local/include"
Index: pkg/DESCR
===================================================================
RCS file: /d/cvs/ports/sysutils/xorriso/pkg/DESCR,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 DESCR
--- pkg/DESCR 30 Jun 2017 13:26:43 -0000 1.1.1.1
+++ pkg/DESCR 30 Jun 2017 15:34:04 -0000
@@ -1,2 +1,5 @@
-xorriso copies files info Rock Ridge enhanced ISO 9660 filesystems,
-and vice versa.
+xorriso copies file objects from POSIX compliant filesystems into Rock
+Ridge enhanced ISO 9660 filesystems and allows session-wise manipulation
+of such filesystems. It can load the management information of existing
+ISO images and it writes the session results to optical media or to
+filesystem objects.
--
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE
