Opened 10 years ago

Closed 10 years ago

#45584 closed update (fixed)

py-matplotlib-basemap Portfile updated to basemap version 1.0.7

Reported by: pgf Owned by: larryv (Lawrence Velázquez)
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc: petrrr, Schamschula (Marius Schamschula)
Port: py-matplotlib-basemap

Description

Please find in attachment the Portfile for py-matplotlib-basemap updated to basemap version 1.0.7. Tested on my system (Mac OS X Mavericks, XCode 6.1, MacPorts 2.3.3), everything seems OK.

Attachments (3)

Portfile (1.6 KB) - added by pgf 10 years ago.
Portfile-py-matplotlib-basemap.diff (786 bytes) - added by pgf 10 years ago.
Portfile_py-matplotlib-basemap.diff (801 bytes) - added by pgf 10 years ago.

Download all attachments as: .zip

Change History (21)

Changed 10 years ago by pgf

Attachment: Portfile added

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

Keywords: basemap removed
Version: 2.3.2

Thanks. Per the guidelines, please instead attach a unified diff of the Portfile so we can easily see what changes you are proposing.

Changed 10 years ago by pgf

comment:2 Changed 10 years ago by pgf

unified diff added.

comment:3 Changed 10 years ago by mf2k (Frank Schima)

Keywords: haspatch added

comment:4 Changed 10 years ago by larryv (Lawrence Velázquez)

Owner: changed from macports-tickets@… to larryv@…
Status: newassigned

Thanks.

comment:5 Changed 10 years ago by larryv (Lawrence Velázquez)

Resolution: fixed
Status: assignedclosed

comment:6 Changed 10 years ago by pgf

As far as I know basemap is ready for python 3.x . I'd really like to have it in pyhton 3.x, but unfortunately I'm not enough expert in port file development to try it myself. Is anyone interested?

comment:7 in reply to:  6 ; Changed 10 years ago by larryv (Lawrence Velázquez)

It doesn’t look like it would be too hard to add a Python 3.4 subport, but at least two of its dependencies needs to have it added first. You should open a new ticket requesting Python 3 support.

comment:8 in reply to:  7 Changed 10 years ago by pgf

Replying to larryv@…:

It doesn’t look like it would be too hard to add a Python 3.4 subport, but at least two of its dependencies needs to have it added first. You should open a new ticket requesting Python 3 support.

I digged a little more into this subject and actually it turned out that the dependency on py-dap, which in turn requires py-httplib2, is no more necessary.
According to the Basemap ChangeLog it was removed in version 1.0.2 .
On the other hand some functionalities require extra packages which are not currently listed as dependencies:

  • scipy : required by the 'interp' interpolation routine, provided by port py-scipy
  • Python Imaging Library : required by 'warpimage', provided by port py-Pillow (or alternatively the older py-pil)
  • OWSLib : required by 'wmsimage', actually not a MacPorts package, but I managed to create a port #45691 .

What about adding these as variants?

variant PIL description {Install Python Imaging Library} {
  depends_lib-append port:py${python.version}-Pillow
}
variant scipy description {Install scipy} {
  depends_lib-append port:py${python.version}-scipy
}

Last edited 10 years ago by pgf (previous) (diff)

Changed 10 years ago by pgf

comment:9 Changed 10 years ago by cdeil (Christoph Deil)

I've filed a new issue to discuss Python 3 support for basemap: #46665

comment:10 Changed 10 years ago by petrrr

Resolution: fixed
Status: closedreopened

comment:11 Changed 10 years ago by petrrr

In future, please avoid discussing new issues, adding patches, etc. to closed tickets. Such information will usually get lost. Either reopen if resolution is not complete or open a new ticket.

Reopening this to address the dependency issues, which was raised.

comment:12 Changed 10 years ago by petrrr

I modified the dependency accordingly in r132168.

comment:13 Changed 10 years ago by petrrr

Cc: petr@… added

Cc Me!

comment:14 Changed 10 years ago by petrrr

I am just reconsidering r132168. These are optional dependencies for specific functionality, which most users may not need, so maybe they should be installed by default?

On the other hand, adding the proposed variants here, IMHO is not very helpful. Having the dependencies in the registry just makes the system more rigid, while without you can just get or remove the functionality at any time by installing uninstalling the respective package.

So what you think, should I just remove the dependencies again?

comment:15 in reply to:  14 ; Changed 10 years ago by Schamschula (Marius Schamschula)

Replying to petr@…:

I am just reconsidering r132168. These are optional dependencies for specific functionality, which most users may not need, so maybe they should be installed by default?

On the other hand, adding the proposed variants here, IMHO is not very helpful. Having the dependencies in the registry just makes the system more rigid, while without you can just get or remove the functionality at any time by installing uninstalling the respective package.

So what you think, should I just remove the dependencies again?

Great. This update conflicts with my install of gpredict, which is built against py27-pil…

comment:16 Changed 10 years ago by Schamschula (Marius Schamschula)

Cc: mschamschula@… added

Cc Me!

comment:17 in reply to:  15 Changed 10 years ago by petrrr

Replying to mschamschula@…:

Replying to petr@…:

I am just reconsidering r132168. These are optional dependencies for specific functionality, which most users may not need, so maybe they should be installed by default?

On the other hand, adding the proposed variants here, IMHO is not very helpful. Having the dependencies in the registry just makes the system more rigid, while without you can just get or remove the functionality at any time by installing uninstalling the respective package.

So what you think, should I just remove the dependencies again?

Great. This update conflicts with my install of gpredict, which is built against py27-pil…

Right! This is another issue. I'll look at this later ...

However, this could be resolved with a path dependency as most pil/Pillow dependent port now do. And maybe we should replace py-pil by py-Pillow at some point.

comment:18 Changed 10 years ago by petrrr

Resolution: fixed
Status: reopenedclosed

In r133806 I removed the optional dependencies again. This partly reverts r132168.

Note: See TracTickets for help on using tickets.