Opened 8 years ago

Closed 8 years ago

#40741 closed submission (fixed)

libPyQGLViewer @2.3.17 submission

Reported by: christophe.pradal@… Owned by: ryandesign (Ryan Schmidt)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: macports@…, raphael-st (Raphael Straub)
Port: libPyQGLViewer

Description

Tested on 10.7.5. Work nicely.

This is a fork of libQGLViewer. We have to modify the headers of this library to be able to build PyQGLViewer, a set of Python bindings for the libQGLViewer library.

Attachments (7)

Portfile (3.7 KB) - added by christophe.pradal@… 8 years ago.
graphics/libPyQGLViewer/Portfile
patch-contribs.pro.diff (250 bytes) - added by christophe.pradal@… 8 years ago.
graphics/libPyQGLViewer/files/
patch-designerPlugin.pro.diff (315 bytes) - added by christophe.pradal@… 8 years ago.
graphics/libPyQGLViewer/files/
patch-QGLViewer.pro.diff (375 bytes) - added by christophe.pradal@… 8 years ago.
graphics/libPyQGLViewer/files/
universal.sh (261 bytes) - added by christophe.pradal@… 8 years ago.
graphics/libPyQGLViewer/files/
Portfile.2 (3.6 KB) - added by christophe.pradal@… 8 years ago.
New Portfile including latest modifications
Portfile.3 (3.4 KB) - added by christophe.pradal@… 8 years ago.
Replace the initial Portfile, taking into account the comments

Download all attachments as: .zip

Change History (11)

Changed 8 years ago by christophe.pradal@…

Attachment: Portfile added

graphics/libPyQGLViewer/Portfile

Changed 8 years ago by christophe.pradal@…

Attachment: patch-contribs.pro.diff added

graphics/libPyQGLViewer/files/

Changed 8 years ago by christophe.pradal@…

graphics/libPyQGLViewer/files/

Changed 8 years ago by christophe.pradal@…

Attachment: patch-QGLViewer.pro.diff added

graphics/libPyQGLViewer/files/

Changed 8 years ago by christophe.pradal@…

Attachment: universal.sh added

graphics/libPyQGLViewer/files/

comment:1 Changed 8 years ago by mf2k (Frank Schima)

Keywords: graphics removed
Port: libPyQGLViewer added
Version: 2.2.0

A few quick comments:

  • Since you are not a Macports committer, your email address needs to be complete and obfuscated. In this case, inria.fr:christophe.pradal.
  • Depending on qt4-x11 for the x11 variant is a bad idea because I don't believe that port can even build? So the variant should just be removed.

Changed 8 years ago by christophe.pradal@…

Attachment: Portfile.2 added

New Portfile including latest modifications

Changed 8 years ago by christophe.pradal@…

Attachment: Portfile.3 added

Replace the initial Portfile, taking into account the comments

comment:2 in reply to:  1 Changed 8 years ago by christophe.pradal@…

Replying to macsforever2000@…:

A few quick comments:

  • Since you are not a Macports committer, your email address needs to be complete and obfuscated. In this case, inria.fr:christophe.pradal.
  • Depending on qt4-x11 for the x11 variant is a bad idea because I don't believe that port can even build? So the variant should just be removed.

In the last Portfile (Portfile.3), I add myself as a maintener using an obfuscated address and I remove the x11 variant. In fact, on my computer, I only use qt4-mac. I do not succeed to install qt4-X11. Thanks for your very fast comments on this port.

comment:3 Changed 8 years ago by ryandesign (Ryan Schmidt)

Cc: macports@… added
Owner: changed from macports-tickets@… to ryandesign@…
Status: newassigned

This port had previously been requested in #30839.

comment:4 Changed 8 years ago by ryandesign (Ryan Schmidt)

Cc: raphael@… added
Resolution: fixed
Status: assignedclosed

Added port in r112291 with some changes:

  • moved the PortGroup line to the top of the portfile where those typically go
  • removed trailing whitespace from version line
  • fixed typo in long description
  • removed unnecessary and undesirable change of distfile to an unversioned distfile (the server will serve the file with any name)
  • fixed capitalization of changeLog.txt file in destroot (this would have failed on case-sensitive systems like our buildbots)
Note: See TracTickets for help on using tickets.