Skip to content

Conversation

semarie
Copy link
Contributor

@semarie semarie commented Jan 28, 2021

the libc update has rust-lang/libc#2035, which permit to unbreak rustc nightly build on OpenBSD

@rust-highfive
Copy link
Contributor

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 28, 2021
@Mark-Simulacrum
Copy link
Member

Please also bump an explicit libc dependency somewhere so this is kept during Cargo.lock rebases and so forth.

@semarie
Copy link
Contributor Author

semarie commented Jan 28, 2021

@Mark-Simulacrum like that ?

@Mark-Simulacrum
Copy link
Member

Yes, r=me with commits squashed. Thanks!

@semarie
Copy link
Contributor Author

semarie commented Jan 28, 2021

ok, done.

@Mark-Simulacrum
Copy link
Member

@bors r+ rollup=never

@bors
Copy link
Collaborator

bors commented Jan 28, 2021

📌 Commit 24ef774 has been approved by Mark-Simulacrum

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 28, 2021
@bors
Copy link
Collaborator

bors commented Jan 29, 2021

⌛ Testing commit 24ef774 with merge fc7706dde5430d17af516a62dd3d646e6b18afba...

@rust-log-analyzer
Copy link
Collaborator

The job dist-x86_64-musl failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)
[RUSTC-TIMING] addr2line test:false 0.452
[RUSTC-TIMING] core test:false 41.004
[RUSTC-TIMING] gimli test:false 5.931
[RUSTC-TIMING] object test:false 10.861
error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
   |
   |
15 |     libc::timespec { tv_sec: <libc::time_t>::MAX, tv_nsec: 1_000_000_000 - 1 };
   |
   |
   = note: `-D deprecated` implied by `-D warnings`

error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
   |
   |
17 | fn saturating_cast_to_time_t(value: u64) -> libc::time_t {


error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
   |
   |
18 |     if value > <libc::time_t>::MAX as u64 { <libc::time_t>::MAX } else { value as libc::time_t }


error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
   |
   |
18 |     if value > <libc::time_t>::MAX as u64 { <libc::time_t>::MAX } else { value as libc::time_t }


error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
   |
   |
18 |     if value > <libc::time_t>::MAX as u64 { <libc::time_t>::MAX } else { value as libc::time_t }


error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
   |
   |
91 |             .checked_add((nsec / 1_000_000_000) as libc::time_t)


error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
    |
    |
303 |             tv_sec: self.stat.st_mtime as libc::time_t,


error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
    |
    |
319 |             tv_sec: self.stat.st_atime as libc::time_t,


error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
   --> library/std/src/sys/unix/net.rs:333:47
    |
333 |                 let secs = if dur.as_secs() > libc::time_t::MAX as u64 {


error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
   --> library/std/src/sys/unix/net.rs:334:21
334 |                     libc::time_t::MAX
    |                     ^^^^^^^^^^^^^^^^^


error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
   --> library/std/src/sys/unix/net.rs:336:38
    |
336 |                     dur.as_secs() as libc::time_t


error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
   --> library/std/src/sys/unix/thread.rs:159:38
    |
159 |                     tv_sec: cmp::min(libc::time_t::MAX as u64, secs) as libc::time_t,


error: use of deprecated type alias `libc::time_t`: This type is changed to 64-bit in musl 1.2.0, we'll follow that change in the future release. See #1848 for more info.
   --> library/std/src/sys/unix/thread.rs:159:73
    |
159 |                     tv_sec: cmp::min(libc::time_t::MAX as u64, secs) as libc::time_t,

error: aborting due to 13 previous errors

[RUSTC-TIMING] std test:false 2.544
[RUSTC-TIMING] std test:false 2.544
error: could not compile `std`

To learn more, run the command again with --verbose.
command did not execute successfully: "/checkout/obj/build/x86_64-unknown-linux-musl/stage0/bin/cargo" "build" "--target" "x86_64-unknown-linux-musl" "-Zbinary-dep-depinfo" "-j" "16" "--release" "--locked" "--color" "always" "--features" "panic-unwind backtrace profiler compiler-builtins-c" "--manifest-path" "/checkout/library/test/Cargo.toml" "--message-format" "json-render-diagnostics"
failed to run: /checkout/obj/build/bootstrap/debug/bootstrap dist --build x86_64-unknown-linux-musl
Build completed unsuccessfully in 0:02:46

@bors
Copy link
Collaborator

bors commented Jan 29, 2021

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jan 29, 2021
@semarie
Copy link
Contributor Author

semarie commented Jan 29, 2021

cc @JohnTitor @nagisa

per rust-lang/libc#1848 (comment), it is know since october 2020 that libstd couldn't be build with libc >= 0.2.80 .

would be acceptable to add some annotations in libstd to allow the depreciation ? like #[cfg_attr(target_env = "musl", allow(deprecated))] where time_t is in use ?

@nagisa
Copy link
Member

nagisa commented Jan 29, 2021

Sorry, I have no idea what the right way forward here should be.

@JohnTitor
Copy link
Member

would be acceptable to add some annotations in libstd to allow the depreciation ? like #[cfg_attr(target_env = "musl", allow(deprecated))] where time_t is in use ?

If it's acceptable here, I think it's the simplest solution. I still haven't found the solution to support both versions of musl.

@jonas-schievink
Copy link
Contributor

@bors r-

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 30, 2021
@JohnTitor
Copy link
Member

@semarie rust-lang/libc#2052 will unblock this issue.

@bors
Copy link
Collaborator

bors commented Feb 3, 2021

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

@semarie
Copy link
Contributor Author

semarie commented Feb 3, 2021

#81678 is a rollup, but it is including libc bump to 0.2.85. So libc fix for OpenBSD which is in 0.2.83 is included. Closing.

@semarie semarie closed this Feb 3, 2021
@semarie semarie deleted the openbsd-libc branch February 3, 2021 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants