public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
* [ITP] geomview 1.9.5
@ 2017-06-12 17:05 Ken Brown
  2017-06-12 17:51 ` Ken Brown
  0 siblings, 1 reply; 5+ messages in thread
From: Ken Brown @ 2017-06-12 17:05 UTC (permalink / raw)
  To: cygwin-apps

[-- Attachment #1: Type: text/plain, Size: 1752 bytes --]

geomview is in Debian:

  https://packages.debian.org/jessie/geomview

The cygport file and patches are attached.

As I said in https://sourceware.org/ml/cygwin-apps/2017-06/msg00019.html, I'm not sure about the naming convention for the versioned library subpackage.  I made a choice, but I can easily change it if I got it wrong.

Package files:

http://sanibeltranquility.com/cygwin/x86/release/geomview/geomview-1.9.5-1-src.tar.xz
http://sanibeltranquility.com/cygwin/x86/release/geomview/geomview-1.9.5-1.hint
http://sanibeltranquility.com/cygwin/x86/release/geomview/geomview-1.9.5-1.tar.xz
http://sanibeltranquility.com/cygwin/x86/release/geomview/libgeomview-devel/libgeomview-devel-1.9.5-1.hint
http://sanibeltranquility.com/cygwin/x86/release/geomview/libgeomview-devel/libgeomview-devel-1.9.5-1.tar.xz
http://sanibeltranquility.com/cygwin/x86/release/geomview/libgeomview_1_9_5/libgeomview_1_9_5-1.9.5-1.hint
http://sanibeltranquility.com/cygwin/x86/release/geomview/libgeomview_1_9_5/libgeomview_1_9_5-1.9.5-1.tar.xz
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/geomview-1.9.5-1-src.tar.xz
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/geomview-1.9.5-1.hint
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/geomview-1.9.5-1.tar.xz
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/libgeomview-devel/libgeomview-devel-1.9.5-1.hint
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/libgeomview-devel/libgeomview-devel-1.9.5-1.tar.xz
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/libgeomview_1_9_5/libgeomview_1_9_5-1.9.5-1.hint
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/libgeomview_1_9_5/libgeomview_1_9_5-1.9.5-1.tar.xz

License: LGPL

Ken



[-- Attachment #2: no_undefined.patch --]
[-- Type: text/plain, Size: 470 bytes --]

--- origsrc/geomview-1.9.5/src/lib/Makefile.am	2014-03-12 12:51:13.000000000 -0400
+++ src/geomview-1.9.5/src/lib/Makefile.am	2017-06-08 17:07:53.078591900 -0400
@@ -11,4 +11,4 @@ libgeomview_la_SOURCES =
 libgeomview_la_LIBADD = \
  $(OOGLLIBS) $(OPENGLLIBS) $(SOCKETLIBS) $(XLIBS) $(ZLIB_LIB)
 libgeomview_la_DEPENDENCIES = $(OOGLLIBS) Makefile.am
-libgeomview_la_LDFLAGS = -release @PACKAGE_VERSION@
+libgeomview_la_LDFLAGS = -no-undefined -release @PACKAGE_VERSION@

[-- Attachment #3: weak_symbols.patch --]
[-- Type: text/plain, Size: 776 bytes --]

--- origsrc/geomview-1.9.5/src/lib/mg/opengl/mgopenglshade.c	2014-03-12 12:51:13.000000000 -0400
+++ src/geomview-1.9.5/src/lib/mg/opengl/mgopenglshade.c	2017-06-09 10:25:41.647550600 -0400
@@ -413,10 +413,15 @@ mgopengl_lightmodeldef(int lightmodel, L
  *	  supports it;
  *  - or, if we're on a system that claims to support it at compile time,
  *	  we'll just hope that that system also supports it at run time.
+ * But Cygwin does not support weak symbols, and the use of "#pragma
+ * weak" causes "undefined symbol" errors when linking on x86_64
+ * Cygwin.
  */
 
+#ifndef __CYGWIN__
 # pragma weak glBindTextureEXT
 # pragma weak glDeleteTexturesEXT
+#endif
 
 # ifndef GL_EXT_texture_object
 /* If <GL/gl.h> doesn't know about glBindTextureEXT etc., declare here. */

[-- Attachment #4: geomview.cygport --]
[-- Type: text/plain, Size: 1223 bytes --]

NAME="geomview"
VERSION=1.9.5
RELEASE=1

SRC_URI="https://sourceforge.net/projects/geomview/files/geomview/${PV}/${P}.tar.xz"

PATCH_URI="no_undefined.patch"
PATCH_URI+=" weak_symbols.patch"

HOMEPAGE="http://www.geomview.org/"
SUMMARY="An interactive 3D viewing program"
DESCRIPTION="Geomview lets you view and manipulate three-dimensional objects: you
use the mouse to rotate, translate, zoom in and out, and so on.
Geomview can be used as a standalone viewer for static objects or as a
display engine for other programs which produce dynamically changing
geometry.  Geomview can display objects described in a variety of file
formats.  Geomview comes with a wide selection of example objects, and
you can create your own objects too."

PKG_NAMES="${PN}
 lib${PN}_${PV//./_}
 lib${PN}-devel
"

geomview_CATEGORY="Math"
libgeomview_1_9_5_CATEGORY="Math Libs"
libgeomview_devel_CATEGORY="Math Devel"


geomview_CONTENTS="
  usr/bin/*.exe
  usr/bin/geomview
  usr/bin/hvectext
  usr/bin/remotegv
  usr/libexec
  usr/share
"

libgeomview_1_9_5_CONTENTS="usr/bin/cyg*-1-9-5.dll"

libgeomview_devel_CONTENTS="
  usr/include
  usr/lib
"

CYGCONF_ARGS="--with-htmlbrowser=cygstart"

DEPEND="libXm-devel libGL-devel libGLU-devel"

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [ITP] geomview 1.9.5
  2017-06-12 17:05 [ITP] geomview 1.9.5 Ken Brown
@ 2017-06-12 17:51 ` Ken Brown
  2017-06-12 19:18   ` Jon Turney
  0 siblings, 1 reply; 5+ messages in thread
From: Ken Brown @ 2017-06-12 17:51 UTC (permalink / raw)
  To: cygwin-apps

[-- Attachment #1: Type: text/plain, Size: 1384 bytes --]

On 6/12/2017 1:05 PM, Ken Brown wrote:
> geomview is in Debian:
> 
>   https://packages.debian.org/jessie/geomview
> 
> The cygport file and patches are attached.
> 
> As I said in 
> https://sourceware.org/ml/cygwin-apps/2017-06/msg00019.html, I'm not 
> sure about the naming convention for the versioned library subpackage.  
> I made a choice, but I can easily change it if I got it wrong.

Revised package files and cygport file, based on Yaakov's response:

http://sanibeltranquility.com/cygwin/x86/release/geomview/geomview-1.9.5-1-src.tar.xz
http://sanibeltranquility.com/cygwin/x86/release/geomview/geomview-1.9.5-1.hint
http://sanibeltranquility.com/cygwin/x86/release/geomview/geomview-1.9.5-1.tar.xz
http://sanibeltranquility.com/cygwin/x86/release/geomview/geomview-devel/geomview-devel-1.9.5-1.hint
http://sanibeltranquility.com/cygwin/x86/release/geomview/geomview-devel/geomview-devel-1.9.5-1.tar.xz
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/geomview-1.9.5-1-src.tar.xz
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/geomview-1.9.5-1.hint
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/geomview-1.9.5-1.tar.xz
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/geomview-devel/geomview-devel-1.9.5-1.hint
http://sanibeltranquility.com/cygwin/x86_64/release/geomview/geomview-devel/geomview-devel-1.9.5-1.tar.xz


[-- Attachment #2: geomview.cygport --]
[-- Type: text/plain, Size: 1033 bytes --]

NAME="geomview"
VERSION=1.9.5
RELEASE=1

SRC_URI="https://sourceforge.net/projects/geomview/files/${PN}/${PV}/${P}.tar.xz"

PATCH_URI="no_undefined.patch"
PATCH_URI+=" weak_symbols.patch"

HOMEPAGE="http://www.geomview.org/"
SUMMARY="An interactive 3D viewing program"
DESCRIPTION="Geomview lets you view and manipulate three-dimensional objects: you
use the mouse to rotate, translate, zoom in and out, and so on.
Geomview can be used as a standalone viewer for static objects or as a
display engine for other programs which produce dynamically changing
geometry.  Geomview can display objects described in a variety of file
formats.  Geomview comes with a wide selection of example objects, and
you can create your own objects too."

PKG_NAMES="${PN} ${PN}-devel"

geomview_CATEGORY="Math"
geomview_devel_CATEGORY="Math Devel"


geomview_CONTENTS="
  usr/bin
  usr/libexec
  usr/share
"

geomview_devel_CONTENTS="
  usr/include
  usr/lib
"

CYGCONF_ARGS="--with-htmlbrowser=cygstart"

DEPEND="libXm-devel libGL-devel libGLU-devel"

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [ITP] geomview 1.9.5
  2017-06-12 17:51 ` Ken Brown
@ 2017-06-12 19:18   ` Jon Turney
  2017-06-12 19:26     ` Ken Brown
  0 siblings, 1 reply; 5+ messages in thread
From: Jon Turney @ 2017-06-12 19:18 UTC (permalink / raw)
  To: cygwin-apps

On 12/06/2017 18:51, Ken Brown wrote:
> On 6/12/2017 1:05 PM, Ken Brown wrote:
>> geomview is in Debian:
>>
>>   https://packages.debian.org/jessie/geomview
>>
>> The cygport file and patches are attached.

I think DEPEND could also include zlib-devel.

I'm not sure the discussion of the issue with weak symbols is 
technically correct, but then I'm not quite sure exactly why it's not 
working :)

Apart from that, looks good.  I added this to your uploads.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [ITP] geomview 1.9.5
  2017-06-12 19:18   ` Jon Turney
@ 2017-06-12 19:26     ` Ken Brown
  2017-06-12 20:03       ` Ken Brown
  0 siblings, 1 reply; 5+ messages in thread
From: Ken Brown @ 2017-06-12 19:26 UTC (permalink / raw)
  To: cygwin-apps

On 6/12/2017 3:17 PM, Jon Turney wrote:
> On 12/06/2017 18:51, Ken Brown wrote:
>> On 6/12/2017 1:05 PM, Ken Brown wrote:
>>> geomview is in Debian:
>>>
>>>   https://packages.debian.org/jessie/geomview
>>>
>>> The cygport file and patches are attached.
> 
> I think DEPEND could also include zlib-devel.
> 
> I'm not sure the discussion of the issue with weak symbols is 
> technically correct, but then I'm not quite sure exactly why it's not 
> working :)
> 
> Apart from that, looks good.  I added this to your uploads.

Thanks.  Maybe I'll raise the issue with weak symbols separately on the 
Cygwin list, since I would like to better understand what's going on.

Ken

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [ITP] geomview 1.9.5
  2017-06-12 19:26     ` Ken Brown
@ 2017-06-12 20:03       ` Ken Brown
  0 siblings, 0 replies; 5+ messages in thread
From: Ken Brown @ 2017-06-12 20:03 UTC (permalink / raw)
  To: cygwin-apps

On 6/12/2017 3:26 PM, Ken Brown wrote:
> On 6/12/2017 3:17 PM, Jon Turney wrote:
>> On 12/06/2017 18:51, Ken Brown wrote:
>>> On 6/12/2017 1:05 PM, Ken Brown wrote:
>>>> geomview is in Debian:
>>>>
>>>>   https://packages.debian.org/jessie/geomview
>>>>
>>>> The cygport file and patches are attached.
>>
>> I think DEPEND could also include zlib-devel.
>>
>> I'm not sure the discussion of the issue with weak symbols is 
>> technically correct, but then I'm not quite sure exactly why it's not 
>> working :)
>>
>> Apart from that, looks good.  I added this to your uploads.
> 
> Thanks.  Maybe I'll raise the issue with weak symbols separately on the 
> Cygwin list, since I would like to better understand what's going on.

I just tried to extract a self-contained STC with "#pragma weak" from 
the geomview sources, and I couldn't get it to fail.  So I too am not 
sure why it's not working.  I don't think this is worth worrying about. 
I'll just modify the comment.

Ken

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-06-12 20:03 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-12 17:05 [ITP] geomview 1.9.5 Ken Brown
2017-06-12 17:51 ` Ken Brown
2017-06-12 19:18   ` Jon Turney
2017-06-12 19:26     ` Ken Brown
2017-06-12 20:03       ` Ken Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).