Opened 13 years ago

Closed 13 years ago

#27350 closed update (fixed)

digikam: updated portfile to 1.6.0

Reported by: jgosmann (Jan Gosmann) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch maintainer Cc: julien@…
Port: digikam

Description

Finally I was able to update the outdated digikam port. I will attach a patch for the portfile.

Attachments (1)

Portfile-digikam.diff (1.4 KB) - added by jgosmann (Jan Gosmann) 13 years ago.
Patch to apply to the portfile

Download all attachments as: .zip

Change History (11)

comment:1 Changed 13 years ago by jgosmann (Jan Gosmann)

See also the related tickets #27348 and #27349.

comment:2 Changed 13 years ago by jmroot (Joshua Root)

Keywords: haspatch maintainer added
Version: 1.9.2

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

Your patch hardcodes /opt/local, but shouldn't.

You also added a dependency on mysql5-server. Does digikam really require a MySQL server running on the same computer, or would a MySQL server on another computer be sufficient? If so, then the dependency should be on mysql5, not on mysql5-server, and should also be written so that mysql5-devel could satisfy it (see any existing port that depends on mysql5 for an example of how this is done).

comment:4 Changed 13 years ago by jgosmann (Jan Gosmann)

I fixed the hardcoded /opt/local path and removed the mysql5-server dependency. Nevertheless, digikam depends on kdegraphics4 which depends on other ports depending on mysql5-server.

comment:5 Changed 13 years ago by julien@…

Cc: julien@… added

Cc Me!

comment:6 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

You removed the mysql5-server dependency, but did not replace it with a mysql5 dependency? Does this software use the MySQL libraries?

The only port in MacPorts today that depends on mysql5-server is akonadi, and this is an error. I already brought this up in #25547 but it was not addressed. I have re-filed this request as #27404.

Changed 13 years ago by jgosmann (Jan Gosmann)

Attachment: Portfile-digikam.diff added

Patch to apply to the portfile

comment:7 Changed 13 years ago by jgosmann (Jan Gosmann)

Digikam 1.6.0 has just been released today. I've attached an updated portfile.

About the mysql dependeny I am a bit unsure. I added it for the 1.5.0 version, because for some reason these configure options concerning MySQL were needed. With 1.6.0 this is not any longer the case and MySQL is not listed as dependency in the Readme file.

I suppose that MySQL is not really a dependency of Digikam and did not include it again in the new Portfile. But I contacted the main developer and depending on his answer I will add that dependency.

comment:8 Changed 13 years ago by julien@…

I think mysql is an optional dependency as digikam can be used with sqlite as well.

Julien

comment:9 Changed 13 years ago by jgosmann (Jan Gosmann)

I verified that MySQL is just an optional runtime dependency which can be used instead of sqlite.

Can someone commit the patch to the port tree (together with the patches of Ticket #27349 and #27348)? (I have no permissions to so. Maybe I should apply to become MacPorts committer.)

comment:10 Changed 13 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: newclosed
Summary: digikam @1.5.0 updated portfiledigikam: updated portfile to 1.6.0
Note: See TracTickets for help on using tickets.