From nobody@FreeBSD.org  Sat Mar 17 19:42:41 2012
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id 642861065677
	for <freebsd-gnats-submit@FreeBSD.org>; Sat, 17 Mar 2012 19:42:41 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from red.freebsd.org (red.freebsd.org [IPv6:2001:4f8:fff6::22])
	by mx1.freebsd.org (Postfix) with ESMTP id 451FF8FC19
	for <freebsd-gnats-submit@FreeBSD.org>; Sat, 17 Mar 2012 19:42:41 +0000 (UTC)
Received: from red.freebsd.org (localhost [127.0.0.1])
	by red.freebsd.org (8.14.4/8.14.4) with ESMTP id q2HJgfMu075826
	for <freebsd-gnats-submit@FreeBSD.org>; Sat, 17 Mar 2012 19:42:41 GMT
	(envelope-from nobody@red.freebsd.org)
Received: (from nobody@localhost)
	by red.freebsd.org (8.14.4/8.14.4/Submit) id q2HJgeg6075817;
	Sat, 17 Mar 2012 19:42:40 GMT
	(envelope-from nobody)
Message-Id: <201203171942.q2HJgeg6075817@red.freebsd.org>
Date: Sat, 17 Mar 2012 19:42:40 GMT
From: Fernando <fernando.apesteguia@gmail.com>
To: freebsd-gnats-submit@FreeBSD.org
Subject: [maintainer update] graphics/opennurbs
X-Send-Pr-Version: www-3.1
X-GNATS-Notify:

>Number:         166210
>Category:       ports
>Synopsis:       [maintainer update] graphics/opennurbs
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    scheidell
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          maintainer-update
>Submitter-Id:   current-users
>Arrival-Date:   Sat Mar 17 19:50:12 UTC 2012
>Closed-Date:    Thu Mar 22 18:03:33 UTC 2012
>Last-Modified:  Thu Mar 22 18:10:03 UTC 2012
>Originator:     Fernando
>Release:        9.0-RELEASE
>Organization:
>Environment:
FreeBSD beastie 9.0-RELEASE FreeBSD 9.0-RELEASE #0: Tue Jan  3 07:46:30 UTC 2012     root@farrell.cse.buffalo.edu:/usr/obj/usr/src/sys/GENERIC  amd64
>Description:
This update installs the library headers needed to compile against this library.
>How-To-Repeat:

>Fix:
Apply the patch attached

Patch attached with submission follows:

diff -ruN opennurbs.orig/Makefile opennurbs/Makefile
--- opennurbs.orig/Makefile	2012-03-17 20:35:05.000000000 +0100
+++ opennurbs/Makefile	2012-03-17 14:20:44.000000000 +0100
@@ -32,5 +32,9 @@
 .for i in brep dump read roundtrip userdata write
 	@${INSTALL_PROGRAM} ${WRKSRC}/example_${i}/example_${i} ${EXAMPLESDIR}
 .endfor
+	@${MKDIR} ${PREFIX}/include/openNURBS
+	@${MKDIR} ${PREFIX}/include/openNURBS/zlib
+	@${MV} ${WRKSRC}/*.h ${PREFIX}/include/openNURBS/
+	@${MV} ${WRKSRC}/zlib/*.h ${PREFIX}/include/openNURBS/zlib
 
 .include <bsd.port.mk>
diff -ruN opennurbs.orig/pkg-plist opennurbs/pkg-plist
--- opennurbs.orig/pkg-plist	2012-03-17 20:35:05.000000000 +0100
+++ opennurbs/pkg-plist	2012-03-17 20:31:26.000000000 +0100
@@ -1,3 +1,120 @@
+include/openNURBS/examples_linking_pragmas.h
+include/openNURBS/opennurbs.h
+include/openNURBS/opennurbs_3dm.h
+include/openNURBS/opennurbs_3dm_attributes.h
+include/openNURBS/opennurbs_3dm_properties.h
+include/openNURBS/opennurbs_3dm_settings.h
+include/openNURBS/opennurbs_annotation.h
+include/openNURBS/opennurbs_annotation2.h
+include/openNURBS/opennurbs_arc.h
+include/openNURBS/opennurbs_arccurve.h
+include/openNURBS/opennurbs_archive.h
+include/openNURBS/opennurbs_array.h
+include/openNURBS/opennurbs_array_defs.h
+include/openNURBS/opennurbs_base32.h
+include/openNURBS/opennurbs_base64.h
+include/openNURBS/opennurbs_beam.h
+include/openNURBS/opennurbs_bezier.h
+include/openNURBS/opennurbs_bitmap.h
+include/openNURBS/opennurbs_bounding_box.h
+include/openNURBS/opennurbs_box.h
+include/openNURBS/opennurbs_brep.h
+include/openNURBS/opennurbs_circle.h
+include/openNURBS/opennurbs_color.h
+include/openNURBS/opennurbs_compress.h
+include/openNURBS/opennurbs_cone.h
+include/openNURBS/opennurbs_crc.h
+include/openNURBS/opennurbs_curve.h
+include/openNURBS/opennurbs_curveonsurface.h
+include/openNURBS/opennurbs_curveproxy.h
+include/openNURBS/opennurbs_cylinder.h
+include/openNURBS/opennurbs_defines.h
+include/openNURBS/opennurbs_detail.h
+include/openNURBS/opennurbs_dimstyle.h
+include/openNURBS/opennurbs_dll_resource.h
+include/openNURBS/opennurbs_ellipse.h
+include/openNURBS/opennurbs_error.h
+include/openNURBS/opennurbs_evaluate_nurbs.h
+include/openNURBS/opennurbs_extensions.h
+include/openNURBS/opennurbs_font.h
+include/openNURBS/opennurbs_fpoint.h
+include/openNURBS/opennurbs_fsp.h
+include/openNURBS/opennurbs_fsp_defs.h
+include/openNURBS/opennurbs_geometry.h
+include/openNURBS/opennurbs_gl.h
+include/openNURBS/opennurbs_group.h
+include/openNURBS/opennurbs_hatch.h
+include/openNURBS/opennurbs_hsort_template.h
+include/openNURBS/opennurbs_instance.h
+include/openNURBS/opennurbs_intersect.h
+include/openNURBS/opennurbs_knot.h
+include/openNURBS/opennurbs_layer.h
+include/openNURBS/opennurbs_light.h
+include/openNURBS/opennurbs_line.h
+include/openNURBS/opennurbs_linecurve.h
+include/openNURBS/opennurbs_linestyle.h
+include/openNURBS/opennurbs_linetype.h
+include/openNURBS/opennurbs_lookup.h
+include/openNURBS/opennurbs_mapchan.h
+include/openNURBS/opennurbs_massprop.h
+include/openNURBS/opennurbs_material.h
+include/openNURBS/opennurbs_math.h
+include/openNURBS/opennurbs_matrix.h
+include/openNURBS/opennurbs_memory.h
+include/openNURBS/opennurbs_mesh.h
+include/openNURBS/opennurbs_nurbscurve.h
+include/openNURBS/opennurbs_nurbssurface.h
+include/openNURBS/opennurbs_object.h
+include/openNURBS/opennurbs_object_history.h
+include/openNURBS/opennurbs_objref.h
+include/openNURBS/opennurbs_offsetsurface.h
+include/openNURBS/opennurbs_optimize.h
+include/openNURBS/opennurbs_plane.h
+include/openNURBS/opennurbs_planesurface.h
+include/openNURBS/opennurbs_pluginlist.h
+include/openNURBS/opennurbs_point.h
+include/openNURBS/opennurbs_pointcloud.h
+include/openNURBS/opennurbs_pointgeometry.h
+include/openNURBS/opennurbs_pointgrid.h
+include/openNURBS/opennurbs_polycurve.h
+include/openNURBS/opennurbs_polyedgecurve.h
+include/openNURBS/opennurbs_polyline.h
+include/openNURBS/opennurbs_polylinecurve.h
+include/openNURBS/opennurbs_qsort_template.h
+include/openNURBS/opennurbs_quaternion.h
+include/openNURBS/opennurbs_rand.h
+include/openNURBS/opennurbs_rendering.h
+include/openNURBS/opennurbs_revsurface.h
+include/openNURBS/opennurbs_rtree.h
+include/openNURBS/opennurbs_sphere.h
+include/openNURBS/opennurbs_string.h
+include/openNURBS/opennurbs_sumsurface.h
+include/openNURBS/opennurbs_surface.h
+include/openNURBS/opennurbs_surfaceproxy.h
+include/openNURBS/opennurbs_system.h
+include/openNURBS/opennurbs_textlog.h
+include/openNURBS/opennurbs_texture.h
+include/openNURBS/opennurbs_texture_mapping.h
+include/openNURBS/opennurbs_torus.h
+include/openNURBS/opennurbs_unicode.h
+include/openNURBS/opennurbs_userdata.h
+include/openNURBS/opennurbs_uuid.h
+include/openNURBS/opennurbs_version.h
+include/openNURBS/opennurbs_viewport.h
+include/openNURBS/opennurbs_workspace.h
+include/openNURBS/opennurbs_x.h
+include/openNURBS/opennurbs_xform.h
+include/openNURBS/opennurbs_zlib.h
+include/openNURBS/zlib/crc32.h
+include/openNURBS/zlib/deflate.h
+include/openNURBS/zlib/inffast.h
+include/openNURBS/zlib/inffixed.h
+include/openNURBS/zlib/inflate.h
+include/openNURBS/zlib/inftrees.h
+include/openNURBS/zlib/trees.h
+include/openNURBS/zlib/zconf.h
+include/openNURBS/zlib/zlib.h
+include/openNURBS/zlib/zutil.h
 lib/libopenNURBS.a
 libexec/openNURBS/example_brep
 libexec/openNURBS/example_dump
@@ -6,3 +123,5 @@
 libexec/openNURBS/example_userdata
 libexec/openNURBS/example_write
 @dirrm libexec/openNURBS
+@dirrm include/openNURBS/zlib
+@dirrm include/openNURBS


>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->scheidell 
Responsible-Changed-By: scheidell 
Responsible-Changed-When: Mon Mar 19 10:04:25 UTC 2012 
Responsible-Changed-Why:  
I'll take it. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=166210 
State-Changed-From-To: open->closed 
State-Changed-By: scheidell 
State-Changed-When: Mon Mar 19 14:04:03 UTC 2012 
State-Changed-Why:  
Committed. Thanks! 

http://www.freebsd.org/cgi/query-pr.cgi?pr=166210 
State-Changed-From-To: closed->open 
State-Changed-By: scheidell 
State-Changed-When: Wed Mar 21 22:43:46 UTC 2012 
State-Changed-Why:  
MV's need to be changed to INSTALL_DATA 

http://www.freebsd.org/cgi/query-pr.cgi?pr=166210 

From: Michael Scheidell <scheidell@FreeBSD.org>
To: fernando.apesteguia@gmail.com, bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/166210: [maintainer update] graphics/opennurbs
Date: Thu, 22 Mar 2012 07:02:32 -0400 (EDT)

 --ELM1332414152-74878-0_
 Content-Transfer-Encoding: 8bit
 Content-Type: text/plain; charset="UNKNOWN-8BIT"
 
 On 3/22/12 3:01 AM, Fernando Apestegu??a wrote:
 > On Wed, Mar 21, 2012 at 11:43 PM,  <scheidell@freebsd.org> wrote:
 >> Synopsis: [maintainer update] graphics/opennurbs
 >>
 >> State-Changed-From-To: closed->open
 >> State-Changed-By: scheidell
 >> State-Changed-When: Wed Mar 21 22:43:46 UTC 2012
 >> State-Changed-Why:
 >> MV's need to be changed to INSTALL_DATA
 >
 > MV's?
 >
 > Sorry but I don't know what you mean :S
 >
 
 The ports audit committee has requested that a change be made to the Makefile.
 Instread of using 'mv' (move) to move the files to the target directory, they want you to use 'INSTALL_DATA'
 (which by default is 'install -m 444').
 
 Without this patch, this operation will fail:
 make clean install
 make deinstall
 make install
 (since there are no headers in the source directory anymore)
 
 Could you test this patch? and if there is some reason you require move instead of install data, please get back to me ASAP?
 
 
 -- 
 Michael Scheidell, CTO
 SECNAP Network Security Corporation
 http://people.freebsd.org/~scheidell
 
 ______________________________________________________________________
 This email has been scanned and certified safe by SpammerTrap(r). 
 For Information please see http://www.spammertrap.com/
 ______________________________________________________________________  
   
 --ELM1332414152-74878-0_
 Content-Transfer-Encoding: 7bit
 Content-Type: text/x-patch
 Content-Disposition: attachment; filename="pr166210.patch"
 
 Index: Makefile
 ===================================================================
 RCS file: /home/pcvs/ports/graphics/opennurbs/Makefile,v
 retrieving revision 1.2
 diff -u -r1.2 Makefile
 --- Makefile	19 Mar 2012 14:03:54 -0000	1.2
 +++ Makefile	22 Mar 2012 10:58:15 -0000
 @@ -34,7 +34,7 @@
  .endfor
  	@${MKDIR} ${PREFIX}/include/openNURBS
  	@${MKDIR} ${PREFIX}/include/openNURBS/zlib
 -	@${MV} ${WRKSRC}/*.h ${PREFIX}/include/openNURBS/
 -	@${MV} ${WRKSRC}/zlib/*.h ${PREFIX}/include/openNURBS/zlib
 +	@${INSTALL_DATA} ${WRKSRC}/*.h ${PREFIX}/include/openNURBS/
 +	@${INSTALL_DATA} ${WRKSRC}/zlib/*.h ${PREFIX}/include/openNURBS/zlib
  
  .include <bsd.port.mk>
 
 --ELM1332414152-74878-0_--

From: =?ISO-8859-1?Q?Fernando_Apestegu=EDa?= <fernando.apesteguia@gmail.com>
To: Michael Scheidell <scheidell@freebsd.org>
Cc: bug-followup@freebsd.org
Subject: Re: ports/166210: [maintainer update] graphics/opennurbs
Date: Thu, 22 Mar 2012 15:44:14 +0100

 On Thu, Mar 22, 2012 at 12:02 PM, Michael Scheidell
 <scheidell@freebsd.org> wrote:
 > On 3/22/12 3:01 AM, Fernando Apestegu=EDa wrote:
 >> On Wed, Mar 21, 2012 at 11:43 PM, =A0<scheidell@freebsd.org> wrote:
 >>> Synopsis: [maintainer update] graphics/opennurbs
 >>>
 >>> State-Changed-From-To: closed->open
 >>> State-Changed-By: scheidell
 >>> State-Changed-When: Wed Mar 21 22:43:46 UTC 2012
 >>> State-Changed-Why:
 >>> MV's need to be changed to INSTALL_DATA
 >>
 >> MV's?
 >>
 >> Sorry but I don't know what you mean :S
 >>
 >
 > The ports audit committee has requested that a change be made to the Make=
 file.
 > Instread of using 'mv' (move) to move the files to the target directory, =
 they want you to use 'INSTALL_DATA'
 > (which by default is 'install -m 444').
 >
 > Without this patch, this operation will fail:
 > make clean install
 > make deinstall
 > make install
 > (since there are no headers in the source directory anymore)
 >
 > Could you test this patch? and if there is some reason you require move i=
 nstead of install data, please get back to me ASAP?
 
 Today (or tomorrow the latest) I can test your patch and send you feedback.
 
 >
 >
 > --
 > Michael Scheidell, CTO
 > SECNAP Network Security Corporation
 > http://people.freebsd.org/~scheidell
 >
 > ______________________________________________________________________
 > This email has been scanned and certified safe by SpammerTrap(r).
 > For Information please see http://www.spammertrap.com/
 > ______________________________________________________________________
 >
State-Changed-From-To: open->closed 
State-Changed-By: scheidell 
State-Changed-When: Thu Mar 22 17:36:32 UTC 2012 
State-Changed-Why:  
Committed. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=166210 

From: =?ISO-8859-1?Q?Fernando_Apestegu=EDa?= <fernando.apesteguia@gmail.com>
To: Michael Scheidell <scheidell@freebsd.org>
Cc: bug-followup@freebsd.org
Subject: Re: ports/166210: [maintainer update] graphics/opennurbs
Date: Thu, 22 Mar 2012 18:30:38 +0100

 On Thu, Mar 22, 2012 at 3:44 PM, Fernando Apestegu=EDa
 <fernando.apesteguia@gmail.com> wrote:
 > On Thu, Mar 22, 2012 at 12:02 PM, Michael Scheidell
 > <scheidell@freebsd.org> wrote:
 >> On 3/22/12 3:01 AM, Fernando Apestegu=EDa wrote:
 >>> On Wed, Mar 21, 2012 at 11:43 PM, =A0<scheidell@freebsd.org> wrote:
 >>>> Synopsis: [maintainer update] graphics/opennurbs
 >>>>
 >>>> State-Changed-From-To: closed->open
 >>>> State-Changed-By: scheidell
 >>>> State-Changed-When: Wed Mar 21 22:43:46 UTC 2012
 >>>> State-Changed-Why:
 >>>> MV's need to be changed to INSTALL_DATA
 >>>
 >>> MV's?
 >>>
 >>> Sorry but I don't know what you mean :S
 >>>
 >>
 >> The ports audit committee has requested that a change be made to the Mak=
 efile.
 >> Instread of using 'mv' (move) to move the files to the target directory,=
  they want you to use 'INSTALL_DATA'
 >> (which by default is 'install -m 444').
 >>
 >> Without this patch, this operation will fail:
 >> make clean install
 >> make deinstall
 >> make install
 >> (since there are no headers in the source directory anymore)
 >>
 >> Could you test this patch? and if there is some reason you require move =
 instead of install data, please get back to me ASAP?
 >
 > Today (or tomorrow the latest) I can test your patch and send you feedbac=
 k.
 
 I updated my ports today and applied the patch. It seems to work. All
 the headers seem to be properly installed under
 /usr/local/include/openNURBS/. The sequence:
 
 make clean install
 make deinstall
 make install
 
 doesn't fail anymore.
 
 Thanks for the heads up and sorry for the inconvenience ;)
 
 >
 >>
 >>
 >> --
 >> Michael Scheidell, CTO
 >> SECNAP Network Security Corporation
 >> http://people.freebsd.org/~scheidell
 >>
 >> ______________________________________________________________________
 >> This email has been scanned and certified safe by SpammerTrap(r).
 >> For Information please see http://www.spammertrap.com/
 >> ______________________________________________________________________
 >>

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/166210: commit references a PR
Date: Thu, 22 Mar 2012 17:36:30 +0000 (UTC)

 scheidell    2012-03-22 17:36:16 UTC
 
   FreeBSD ports repository
 
   Modified files:
     graphics/opennurbs   Makefile 
   Log:
   - Change install from mv to install_data
   - No PORTREVISION bump (only affected reinstall without clean)
   
   PR:             ports/166210
   Reported by:    wxs@
   Approved by:    Fernando Apesteguía <fernando.apesteguia@gmail.com> (maintainer)
   Feature safe:   yes
   
   Revision  Changes    Path
   1.3       +2 -2      ports/graphics/opennurbs/Makefile
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
>Unformatted:
