Opened 15 years ago
Closed 15 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)
Change History (9)
comment:1 Changed 15 years ago by mbrooksclark@…
Cc: | mbrooksclark@… added |
---|
comment:2 Changed 15 years ago by mf2k (Frank Schima)
Cc: | mbrooksclark@… removed |
---|---|
Keywords: | haspatch added |
Changed 15 years ago by mbrooksclark@…
Attachment: | radlib_Portfile.diff added |
---|
Updated portfile for radlib-2.8.4
comment:3 Changed 15 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 15 years ago by mbrooksclark@…
Any chance we can get the updated radlib and wview portfiles pushed to the distribution?
comment:5 follow-up: 6 Changed 15 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 Changed 15 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 15 years ago by ryandesign (Ryan Carsten Schmidt)
Owner: | changed from macports-tickets@… to ryandesign@… |
---|---|
Status: | new → assigned |
Summary: | radlib-2.8.4 updated poetfile → radlib-2.8.4 updated portfile |
comment:8 Changed 15 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
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.
Cc Me!