Skip to content

Add TIOCSBRK and TIOCCBRK#1587

Merged
JohnTitor merged 2 commits into
rust-lang:masterfrom
Susurrus:termios_breaks_consts
May 7, 2020
Merged

Add TIOCSBRK and TIOCCBRK#1587
JohnTitor merged 2 commits into
rust-lang:masterfrom
Susurrus:termios_breaks_consts

Conversation

@Susurrus

@Susurrus Susurrus commented Nov 7, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@rust-highfive

rust-highfive commented Nov 7, 2019

Copy link
Copy Markdown

r? @gnzlbg

(rust_highfive has picked a reviewer for you, use r? to override)

@Susurrus Susurrus force-pushed the termios_breaks_consts branch from d3c18c1 to e25a83e Compare November 8, 2019 13:24
@Susurrus

Susurrus commented Nov 8, 2019

Copy link
Copy Markdown
Contributor Author

Got a few errors on some platforms that just say "bash error XXX", so force-repushing to trigger a CI rerun. We'll see what happens.

@Susurrus Susurrus force-pushed the termios_breaks_consts branch from e25a83e to 9c0772d Compare November 8, 2019 13:36
@Susurrus

Susurrus commented Nov 8, 2019

Copy link
Copy Markdown
Contributor Author

Looks like fetching some packages is failing on some builds, not really certain why or if it's a temporary thing (I would assume so). I'll let it sit a couple of days and rerun again.

@gnzlbg

gnzlbg commented Nov 18, 2019

Copy link
Copy Markdown
Contributor

@bors: r+

@bors

bors commented Nov 18, 2019

Copy link
Copy Markdown
Contributor

📌 Commit 9c0772d has been approved by gnzlbg

@bors

bors commented Nov 19, 2019

Copy link
Copy Markdown
Contributor

⌛ Testing commit 9c0772d with merge 506d2e7...

bors added a commit that referenced this pull request Nov 19, 2019
@bors
Auto merge of #1587 - Susurrus:termios_breaks_consts, r=gnzlbg
506d2e7 
Add TIOCSBRK and TIOCCBRK
@bors

bors commented Nov 19, 2019

Copy link
Copy Markdown
Contributor

💔 Test failed - status-azure

gnzlbg
gnzlbg suggested changes Nov 19, 2019
View reviewed changes

@gnzlbg gnzlbg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The sparc64-unknown-linux-gnu failure is legit.

@gnzlbg

gnzlbg commented Nov 28, 2019

Copy link
Copy Markdown
Contributor

@Susurrus can you take a look at the failure?

@gnzlbg gnzlbg closed this Nov 28, 2019
@gnzlbg gnzlbg reopened this Nov 28, 2019
@Susurrus Susurrus force-pushed the termios_breaks_consts branch 3 times, most recently from 6915e9b to 0a4e073 Compare December 7, 2019 21:55
@Susurrus

Susurrus commented Dec 7, 2019

Copy link
Copy Markdown
Contributor Author

Don't know why that last one failed; couldn't tell what the error was. Force repushed to trigger a new CI run.

@Susurrus

Susurrus commented Dec 8, 2019

Copy link
Copy Markdown
Contributor Author

So it's failing due to timing out, but the platforms that didn't get tested that last time were passing earlier so I think this is ready to merge, just needs some playing with CI flakiness.

@Susurrus Susurrus force-pushed the termios_breaks_consts branch from 0a4e073 to a459d0f Compare December 8, 2019 13:48
@Susurrus

Susurrus commented Dec 8, 2019

Copy link
Copy Markdown
Contributor Author

So, I've force-pushed twice and I'm still getting the same CI time-out failures. Don't know what to do to push this forward.

@Susurrus

Susurrus commented Dec 13, 2019

Copy link
Copy Markdown
Contributor Author

Anything I can do to keep this moving forward?

1 similar comment
@Susurrus

Susurrus commented Dec 13, 2019

Copy link
Copy Markdown
Contributor Author

Anything I can do to keep this moving forward?

@gnzlbg

gnzlbg commented Dec 23, 2019

Copy link
Copy Markdown
Contributor

@bors: r+

@bors

bors commented Dec 23, 2019

Copy link
Copy Markdown
Contributor

📌 Commit a459d0f has been approved by gnzlbg

@bors

bors commented Dec 23, 2019

Copy link
Copy Markdown
Contributor

⌛ Testing commit a459d0f with merge 6717fba...

bors added a commit that referenced this pull request Dec 23, 2019
@bors
Auto merge of #1587 - Susurrus:termios_breaks_consts, r=gnzlbg
6717fba 
Add TIOCSBRK and TIOCCBRK
@bors

bors commented Dec 23, 2019

Copy link
Copy Markdown
Contributor

💔 Test failed - status-azure

@Susurrus

Susurrus commented Dec 29, 2019

Copy link
Copy Markdown
Contributor Author

Looks like it's still failing, but looks like a package is failing to install on one of the builds. However, it's been rerun several times now, and I think it's all been the same failure. Is this affecting other PRs or what can we do to resolve this?

@Susurrus Susurrus force-pushed the termios_breaks_consts branch from a459d0f to a0952c2 Compare January 11, 2020 17:39
@Susurrus

Susurrus commented Jan 12, 2020

Copy link
Copy Markdown
Contributor Author

So now that i686-apple-darwin is not a Tier 1 target, that target fails in CI, but it's because the Rust installation fails, not due to this PR.

And BuildChannelsOSX nightly fails due to the armv7-apple-ios Rust target not being installed correctly. Is anyone actively working on resolving these failures? If not, is an appropriate fix to remove these targets, if so I can do that in this PR.

@bors

bors commented Apr 13, 2020

Copy link
Copy Markdown
Contributor

☔ The latest upstream changes (presumably #1719) made this pull request unmergeable. Please resolve the merge conflicts.

@Susurrus Susurrus force-pushed the termios_breaks_consts branch from a0952c2 to be7b2c0 Compare May 7, 2020 02:36
@Susurrus

Susurrus commented May 7, 2020

Copy link
Copy Markdown
Contributor Author

Only failing on arm-linux-androideabi with what looks to be an unrelated failure.

@JohnTitor

JohnTitor commented May 7, 2020

Copy link
Copy Markdown
Member

The failure should be fixed by #1754, could you rebase?

Susurrus added 2 commits May 7, 2020 10:56
@Susurrus
Reorder consts for sparc64 to match header
d44f9b7
@Susurrus
Add TIOCSBRK/TIOCCBRK to more supported platforms
524ec8f 
Reorganized some constants as well so they're grouped a bit better.
@Susurrus Susurrus force-pushed the termios_breaks_consts branch from be7b2c0 to 524ec8f Compare May 7, 2020 17:57
@Susurrus

Susurrus commented May 7, 2020

Copy link
Copy Markdown
Contributor Author

CI finally passed!

@Susurrus Susurrus requested a review from gnzlbg May 7, 2020 21:01
JohnTitor
JohnTitor approved these changes May 7, 2020
View reviewed changes

@JohnTitor JohnTitor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@JohnTitor JohnTitor merged commit cf63ea1 into rust-lang:master May 7, 2020
@bors bors mentioned this pull request May 7, 2020
Closed
@Susurrus Susurrus deleted the termios_breaks_consts branch May 8, 2020 15:14
@Susurrus Susurrus mentioned this pull request May 14, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JohnTitor JohnTitor JohnTitor approved these changes

@gnzlbg gnzlbg Awaiting requested review from gnzlbg

Assignees

@gnzlbg gnzlbg

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Susurrus @rust-highfive @gnzlbg @bors @JohnTitor