#48765 closed update (fixed)
update mosh to 1.2.5
Reported by: | bket | Owned by: | quentinmit (Quentin Smith) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.3.3 |
Keywords: | haspatch | Cc: | |
Port: | mosh |
Description
Included a diff for updating mosh to the latest version (1.2.5).
Attachments (1)
Change History (12)
Changed 9 years ago by bket
comment:1 Changed 9 years ago by petrrr
Cc: | quentin@… removed |
---|---|
Owner: | changed from macports-tickets@… to quentin@… |
comment:2 follow-up: 3 Changed 9 years ago by quentinmit (Quentin Smith)
comment:3 Changed 9 years ago by bket
Replying to quentin@…:
LGTM. Unfortunately I can't add haspatch to this ticket; perhaps you can add that? (in the Keywords field)
Nope...I'm unable to edit details in the ticket. Only thing I can do is add a new comment. I guess this keyword is used to draw attention from someone who has commit rights?
comment:4 Changed 9 years ago by neverpanic (Clemens Lang)
Keywords: | haspatch added |
---|
comment:5 Changed 9 years ago by neverpanic (Clemens Lang)
I can commit this. Note that we're also moving our default perl version to 5.22 (see #48365). Do you want me to fix this along the way as well?
comment:6 Changed 9 years ago by neverpanic (Clemens Lang)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Committed in r140016 without the perl change -- we can do that in a follow-up.
comment:7 Changed 9 years ago by quentinmit (Quentin Smith)
Whoops. I was just about to say "I just tested with perl 5.22 and that works fine". I guess I should file a new ticket with the patch for the perl change?
comment:9 Changed 9 years ago by neverpanic (Clemens Lang)
-
Portfile
5 5 6 6 name mosh 7 7 version 1.2.5 8 revision 1 8 9 categories net 9 10 license {GPL-3+ OpenSSLException} 10 11 platforms darwin … … 25 26 port:protobuf-cpp \ 26 27 port:zlib \ 27 28 path:lib/libssl.dylib:openssl \ 28 port:p5.16-getopt-long 29 port:p5.22-getopt-long \ 30 port:p5.22-io-socket-ip 29 31 30 32 post-patch { 31 reinplace "s|#!/usr/bin/env perl|#!${prefix}/bin/perl5. 16|" ${worksrcpath}/scripts/mosh.pl33 reinplace "s|#!/usr/bin/env perl|#!${prefix}/bin/perl5.22|" ${worksrcpath}/scripts/mosh.pl 32 34 } 33 35 34 36 post-destroot {
OK to commit?
comment:10 Changed 9 years ago by quentinmit (Quentin Smith)
Yep, looks good. I just tested that and it seems to work correctly.
comment:11 Changed 9 years ago by neverpanic (Clemens Lang)
Committed in r140028, thank you for your time.
LGTM. Unfortunately I can't add haspatch to this ticket; perhaps you can add that? (in the Keywords field)