Opened 7 years ago

Closed 4 years ago

#53747 closed update (fixed)

lmdb : update to 0.9.19

Reported by: RJVB (René Bertin) Owned by:
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc: kurthindenburg (Kurt Hindenburg), l2dy (Zero King), chrstphrchvz (Christopher Chavez)
Port: lmdb

Description

Please find attached an update to lmdb 0.9.19 .

Attachments (1)

lmdb.diff (2.1 KB) - added by RJVB (René Bertin) 7 years ago.

Download all attachments as: .zip

Change History (9)

Changed 7 years ago by RJVB (René Bertin)

Attachment: lmdb.diff added

comment:1 Changed 7 years ago by kurthindenburg (Kurt Hindenburg)

Cc: kurthindenburg added; RJVB removed

If you're checking for darwin and linux, doesn't the platforms linux need linux in it?

platforms darwin

comment:2 Changed 7 years ago by RJVB (René Bertin)

It's not needed in any strict sens of the term. I even have the impression that quite a few Portfiles that do check for other OS types (I noticed FreeBSD the other day) don't mention this in the platforms statement.

I wouldn't mind adding the info, but it can give the impression that it's non trivial to use the port on another OS or even that there's official support. Neither of those impressions would be true. In addition, there are lots of ports that work "as is" on Linux. IOW, I never know what purpose the platforms statement really serves.

FWIW we wouldn't be having this discussion if my proposition to support an else clause for the platform syntax had been implemented already ;)

comment:3 Changed 7 years ago by l2dy (Zero King)

Cc: l2dy added

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

The platforms statement serves to inform the user what platforms the port is believed to work on. MacPorts doesn't check this value, merely shows it to the user.

comment:5 Changed 5 years ago by chrstphrchvz (Christopher Chavez)

lmdb has since been updated to 0.9.22. Do any of the other proposed Portfile changes still apply, or can this be closed?

comment:6 Changed 5 years ago by chrstphrchvz (Christopher Chavez)

Cc: chrstphrchvz added

comment:7 Changed 5 years ago by RJVB (René Bertin)

actually, lmdb is at 0.9.23 since a while already. The other changes are just to make the port platform-agnostic and thus not something I'm going to insist on.

comment:8 Changed 4 years ago by mf2k (Frank Schima)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.