Opened 10 years ago
Closed 10 years ago
#44805 closed enhancement (fixed)
Added livecheck to mksh
Reported by: | Schamschula (Marius Schamschula) | Owned by: | macports-tickets@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.3.1 |
Keywords: | haspatch maintainer | Cc: | |
Port: | mksh |
Description
I've added livecheck to mksh.
Attachments (1)
Change History (4)
comment:1 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
comment:2 Changed 10 years ago by Schamschula (Marius Schamschula)
Ryan,
Good catch (I'm not the greatest regex expert).
Updated the regex as per your suggestion.
Changed 10 years ago by Schamschula (Marius Schamschula)
Attachment: | Portfile-mksh.diff added |
---|
comment:3 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
I think your second backslash migrated a bit:
Note how when you run livecheck, the "\a" disappears from the regex (because an escaped "a" has no meaning in regex syntax):
The backslash belongs before the closing square bracket:
Also, note that you're defining a character class here. You're saying you want to match a sequence of 1 or more characters from the set "0, 1, 2, 3, 4, 5, 6, 7, 8, 9, a, b, c, d, e, f, g, R, period", in any order. Looking at the download directory, I think we can be more precise than that. All the existing versions begin with "R", then have at least one digit, and then optionally a letter suffix. So we could write: