Opened 12 years ago

Closed 12 years ago

#34009 closed update (fixed)

autojump: update to v20

Reported by: johnny.kuan@… Owned by: drkp (Dan Ports)
Priority: Normal Milestone:
Component: ports Version: 2.0.4
Keywords: haspatch maintainer Cc:
Port: autojump

Description

Please let me know if hardcoding ${prefix}/bin/python2.7 is a bad idea.

Attachments (1)

autojump-20.patch (1.1 KB) - added by johnny.kuan@… 12 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Keywords: maintainer added

comment:2 in reply to:  description ; Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to johnny.kuan@…:

Please let me know if hardcoding ${prefix}/bin/python2.7 is a bad idea.

You could offer python variants, but I'm fine with just using version 2.7.

The update doesn't work for me because the checksums don't match. Can you check if upstream has perhaps updated the distfile since you submitted this patch, and if so, check what the differences are between the file they have now and the one you got before?

comment:3 in reply to:  2 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to ryandesign@…:

Replying to johnny.kuan@…:

Please let me know if hardcoding ${prefix}/bin/python2.7 is a bad idea.

You could offer python variants, but I'm fine with just using version 2.7.

It looks like the only versions that will work are 2.7 or 3.2 and up. So I wouldn't bother with variants.

Changed 12 years ago by johnny.kuan@…

Attachment: autojump-20.patch added

comment:4 Changed 12 years ago by johnny.kuan@…

Patch with updated checksums is attached. They basically added notes that python2.7+ is now required.

comment:5 Changed 12 years ago by drkp (Dan Ports)

Owner: changed from macports-tickets@… to dports@…
Status: newassigned

comment:6 Changed 12 years ago by drkp (Dan Ports)

Resolution: fixed
Status: assignedclosed

Thanks, committed in r91992.

Note: See TracTickets for help on using tickets.