Opened 14 years ago

Closed 14 years ago

#22097 closed update (fixed)

radlib-2.8.4 updated portfile

Reported by: mbrooksclark@… Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc:
Port: radlib

Description

This is an updated portfile for the most recent release of radlib (v2.8.4).

Attachments (1)

radlib_Portfile.diff (1.9 KB) - added by mbrooksclark@… 14 years ago.
Updated portfile for radlib-2.8.4

Download all attachments as: .zip

Change History (9)

comment:1 Changed 14 years ago by mbrooksclark@…

Cc: mbrooksclark@… added

Cc Me!

comment:2 Changed 14 years ago by mf2k (Frank Schima)

Cc: mbrooksclark@… removed
Keywords: haspatch added

Changed 14 years ago by mbrooksclark@…

Attachment: radlib_Portfile.diff added

Updated portfile for radlib-2.8.4

comment:3 in reply to:  description Changed 14 years ago by mbrooksclark@…

Replying to mbrooksclark@…:

This is an updated portfile for the most recent release of radlib (v2.8.4).

I just added an updated diff file for the radlib port. Please let me know if there's anything else I need to do to get the updates added to the distribution.

comment:4 Changed 14 years ago by mbrooksclark@…

Any chance we can get the updated radlib and wview portfiles pushed to the distribution?

comment:5 Changed 14 years ago by jmroot (Joshua Root)

Version: 1.8.1

Does the new version actually no longer work with postgres or mysql5-devel?

comment:6 in reply to:  5 Changed 14 years ago by mbrooksclark@…

Replying to jmr@…:

Does the new version actually no longer work with postgres or mysql5-devel?

I don't know that anyone is using it with postgres, and I personally haven't tested it. Unless somebody absolutely needs it, I just felt it would be better to keep the radlib and wview installations as simple as possible.

As far as I know, it's a relatively small community using radlib/wview to archive their weather station data. wview recently moved to sqlite3 for its primary archive format.

comment:7 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Owner: changed from macports-tickets@… to ryandesign@…
Status: newassigned
Summary: radlib-2.8.4 updated poetfileradlib-2.8.4 updated portfile

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

Resolution: fixed
Status: assignedclosed

I committed your patch in r60331 with the following changes:

  • I removed the lone tab character and used spaces to be consistent with the whitespace in the rest of the portfile
  • I made sqlite3 the default variant only if the user has not already requested the mysql5 variant
  • I did not make your requested changes to the mysql5 variant: I did not add a dependency on mysql5-server (I would expect radlib would work fine with a mysql5 server hosted on a different computer), and I did not change the mysql5 dependency to prevent the use of mysql5-devel (I would expect radlib to work fine with mysql5-devel). If you disagree, please explain.
Note: See TracTickets for help on using tickets.