Opened 7 years ago
Closed 6 years ago
#54662 closed defect (fixed)
mc @4.8.19 +ncurses: build fails with ncurses 6
| Reported by: | philippetev (Philip Petev) | Owned by: | kurthindenburg (Kurt Hindenburg) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 2.4.1 |
| Keywords: | haspatch | Cc: | |
| Port: | mc |
Description
The latest version 4.8.19 (the ncurses variant) can't be built against ncurses 6.x. The problem is solved upstream, but it seems the maintainer of that port hasn't seen it yet.
The solution: lib/tty/tty-ncurses.c should be patched against the latest upstream version of that file in order the variant to be built with ncurses 6.0 and later.
Obviously, that problem won't be present in the future versions.
For more info:
https://github.com/MidnightCommander/mc/commits/master/lib/tty/tty-ncurses
Ticket 3697
Attachments (1)
Change History (12)
comment:1 Changed 7 years ago by Schamschula (Marius Schamschula)
| Priority: | High → Normal |
|---|
comment:2 follow-up: 3 Changed 7 years ago by philippetev (Philip Petev)
If failed build is not High priority, then what it is?
I've already prepared the patch here:
https://mega.nz/#!xBdHlC5Q!1grv9tZexRCFQnLfYxvPMI8YGw9AQD4phs4lORrq2ZI
All it take is to be added to the files subdir and the Portfile.
comment:3 Changed 7 years ago by mf2k (Frank Schima)
Replying to philippetev:
If failed build is not High priority, then what it is?
We have thousands of open tickets, many of which are build failures. Why is this build failure higher priority than any of those? The Macports team will decide what is High priority. Usually it is based on the number of users and ports affected.
comment:4 Changed 7 years ago by raimue (Rainer Müller)
Can you please attach the patch to this ticket? External hosters are known not to keep files reliable for long time archiving and are not accessible for everyone (think of corporate firewalls).
Changed 7 years ago by philippetev (Philip Petev)
| Attachment: | patch_ncurses6x.diff added |
|---|
Patch for ncurses 6.x+
comment:6 Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
| Keywords: | haspatch added |
|---|---|
| Owner: | set to mkae |
| Status: | new → assigned |
| Summary: | Midnight Commander (ncurses) can't be built → mc @4.8.19 +ncurses: build fails with ncurses 6 |
comment:7 Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
Replying to philippetev:
For more info: https://github.com/MidnightCommander/mc/commits/master/lib/tty/tty-ncurses
Ticket 3697
Looks like this is the commit: https://github.com/MidnightCommander/mc/commit/4d46a108629beb66a293672db7b44f863b6598ba
And this is the ticket: http://midnight-commander.org/ticket/3697
comment:9 Changed 7 years ago by mf2k (Frank Schima)
| Owner: | mkae deleted |
|---|
comment:10 Changed 6 years ago by lyskov (Sergey Lyskov)
Upstream ticket now marked as fixed, could port be update to include the fix? Thanks,
comment:11 Changed 6 years ago by kurthindenburg (Kurt Hindenburg)
| Owner: | set to kurthindenburg |
|---|---|
| Resolution: | → fixed |
| Status: | assigned → closed |
Please do not set priority!