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.

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

[PATCH] don't set TOSTOP attribute on slave terminal #1637

Closed
mc-butler opened this issue Sep 24, 2009 · 8 comments
Closed

[PATCH] don't set TOSTOP attribute on slave terminal #1637

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

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/1637
Reporter slavazanko (@slavaz)
Mentions alk@….by

From: Aliaksey Kandratsenka

TOSTOP doesn't work well with some software. Examples known to me are:

  • debugging multithreaded program with gdb (gdb job stops)
  • running 'check' tests in fork mode (default) that do some output

(tests mysteriously fail due to timeout)

IMO TOSTOP adds too little value compared to compatibility problems it
creates.

Note

Original attachments:

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Sep 24, 2009 at 11:33 UTC

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Sep 24, 2009 at 11:33 UTC (comment 1)

  • Description edited

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 23, 2009 at 13:26 UTC (comment 2)

  • Status changed from new to accepted
  • Owner set to slavazanko

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 23, 2009 at 13:28 UTC (comment 3)

  • Milestone changed from 4.7 to 4.7.0
  • Severity changed from no branch to on review

Created branch 1637_no_TOSTOP

Initial [233f943]

Review, please.

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 23, 2009 at 13:28 UTC (comment 4)

  • Votes set to slavazanko

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Dec 23, 2009 at 17:36 UTC (comment 5)

  • Severity changed from on review to approved
  • Votes changed from slavazanko to slavazanko andrew_b

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 23, 2009 at 18:55 UTC (comment 6)

  • Status changed from accepted to testing
  • Resolution set to fixed
  • Severity changed from approved to merged
  • Votes changed from slavazanko andrew_b to commited-master

merge [0ad310c]

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Dec 23, 2009 at 18:56 UTC (comment 7)

  • Status changed from testing to closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress
Development

No branches or pull requests

2 participants