Ticket #34009 (closed update: fixed)
autojump: update to v20
| Reported by: | johnny.kuan@… | Owned by: | dports@… |
|---|---|---|---|
| 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
Change History
comment:2 in reply to: ↑ description ; follow-up: ↓ 3 Changed 13 months ago by 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.
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 13 months ago by ryandesign@…
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.
comment:4 Changed 13 months ago by johnny.kuan@…
Patch with updated checksums is attached. They basically added notes that python2.7+ is now required.