Opened 14 years ago

Closed 14 years ago

#26071 closed update (fixed)

Update moodle to 1.9.9 and add postgresql variant

Reported by: ldeck Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc:
Port: moodle

Description

Attaching a diff to update the moodle port to the current stable version of modle, 1.9.9.

Also adding a variant for utilising postgresql.

Attachments (1)

Portfile.diff (1.6 KB) - added by ldeck 14 years ago.
update moodle to 1.9.9 adding postgresql variant

Download all attachments as: .zip

Change History (4)

Changed 14 years ago by ldeck

Attachment: Portfile.diff added

update moodle to 1.9.9 adding postgresql variant

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

Cc: ryandesign@… added; ryandesign removed
Keywords: haspatch added; moodle removed
Version: 1.9.1

Bare usernames don't work in the cc field, you have to use complete email addresses.

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

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

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

Resolution: fixed
Status: assignedclosed

Thanks for the suggestion. I updated moodle in r70599.

Instead of adding just a postgresql variant, I added postgresql and mysql variants, with mysql still being the default. This way it's clear to the user that they have a choice between PostgreSQL and MySQL, and not just a choice of PostgreSQL on or off. I also added variant descriptions. We would usually mark these variants as conflicting with one another, but in this case it didn't seem to cause any harm to let the user select both at once and then decide at runtime which they want.

Instead of adding "universal_variant no", I used "supported_archs noarch" since this port installs no architecture-specific files. This wasn't in fact 100% true: it did install some pre-compiled i386 binaries of mimetex for Mac OS X, Linux, FreeBSD and Windows. I modified the port to remove these files; if we later find we do want mimetex after all, making a separate mimetex port seems more appropriate.

Note: See TracTickets for help on using tickets.