Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

mc fails to build against musl libc #3267

Closed
mc-butler opened this issue Sep 8, 2014 · 5 comments
Closed

mc fails to build against musl libc #3267

mc-butler opened this issue Sep 8, 2014 · 5 comments
Assignees
@aborodin
Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress
Milestone
4.8.14

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/3267
Reporter paulbarker (paul@….me.uk)

mc fails to build when using musl as the libc provider. This is due to the CTRL() macro not being defined in <termios.h>. We could include <sys/ttydefaults.h> explicitly but it's easier just to ensure CTRL is defined.

I've attached a patch taken from the Sabotage Linux distro which fixes this. This patch has also been tested and works with the OpenEmbedded build system. The patch is in unified diff format but I can provide it in another format if needed.

Note

Original attachments:

  • mc-CTRL.patch (raw) by paulbarker (paul@….me.uk) on Sep 8, 2014 at 10:24 UTC
@mc-butler
Copy link
Author

Changed by paulbarker (paul@….me.uk) on Sep 8, 2014 at 10:24 UTC

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Sep 27, 2014 at 17:28 UTC (comment 1)

  • Branch state changed from no branch to on review
  • Milestone changed from Future Releases to 4.8.14
  • Owner set to andrew_b
  • Status changed from new to accepted
  • Votes set to andrew_b

Branch: 3267_musl_libc
[9b126e229a142874014582309575ac0dcf70ad5f]

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Oct 1, 2014 at 10:19 UTC (comment 2)

  • Branch state changed from on review to approved
  • Votes changed from andrew_b to andrew_b slavazanko

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Oct 1, 2014 at 11:21 UTC (comment 3)

  • Status changed from accepted to testing
  • Resolution set to fixed
  • Branch state changed from approved to merged
  • Votes changed from andrew_b slavazanko to committed-master

Merged to master: [e78b636].

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Oct 1, 2014 at 11:24 UTC (comment 4)

  • Status changed from testing to closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aborodin aborodin

Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress
Milestone
4.8.14
Development

No branches or pull requests
2 participants
@aborodin @mc-butler