Skip to content

Conversation

@rust-highfive
Copy link

r? @gnzlbg

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

Copy link
Contributor

@gnzlbg gnzlbg left a comment

Choose a reason for hiding this comment

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

The hard part of the upgrade is probably updating libc-test/build.rs to remove all ignored APIs that are available in 18.04 but that are not available in 17.10. I'd guess that one can do that later.

@mati865
Copy link
Member Author

mati865 commented Mar 3, 2019

Probably these 2 could go away once all the images are updated:

  • libc/libc-test/build.rs

    Lines 539 to 551 in 8d94e00

    // These are not in a glibc release yet, only in kernel headers.
    "AF_XDP"
    | "PF_XDP"
    | "SOL_XDP"
    | "IPV6_FLOWINFO"
    | "IPV6_FLOWLABEL_MGR"
    | "IPV6_FLOWINFO_SEND"
    | "IPV6_FLOWINFO_FLOWLABEL"
    | "IPV6_FLOWINFO_PRIORITY"
    if linux =>
    {
    true
    }
  • libc/libc-test/build.rs

    Lines 595 to 609 in 8d94e00

    // There seems to be a small error in EGLIBC's eventfd.h header. The
    // [underlying system call][1] always takes its first `count`
    // argument as an `unsigned int`, but [EGLIBC's <sys/eventfd.h>
    // header][2] declares it to take an `int`. [GLIBC's header][3]
    // matches the kernel.
    //
    // EGLIBC is no longer actively developed, and Debian, the largest
    // distribution that had been using it, switched back to GLIBC in
    // April 2015. So effectively all Linux <sys/eventfd.h> headers will
    // be using `unsigned int` soon.
    //
    // [1]: https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/tree/fs/eventfd.c?id=refs/tags/v3.12.51#n397
    // [2]: http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/trusty/eglibc/trusty/view/head:/sysdeps/unix/sysv/linux/sys/eventfd.h
    // [3]: https://sourceware.org/git/?p=glibc.git;a=blob;f=sysdeps/unix/sysv/linux/sys/eventfd.h;h=6295f32e937e779e74318eb9d3bdbe76aef8a8f3;hb=4e42b5b8f89f0e288e68be7ad70f9525aebc2cff#l34
    "eventfd" if linux => true,

In fact this PR modifies API only for powerpc* because other files that I changed use pinned musl and I found nothing about powerpc in libc-test/build.rs.

@gnzlbg
Copy link
Contributor

gnzlbg commented Mar 3, 2019

@bors: r+

@bors
Copy link
Contributor

bors commented Mar 3, 2019

📌 Commit 96eb1b2 has been approved by gnzlbg

@gnzlbg
Copy link
Contributor

gnzlbg commented Mar 3, 2019

Thank you!

@bors
Copy link
Contributor

bors commented Mar 3, 2019

⌛ Testing commit 96eb1b2 with merge 750a7a6...

@bors
Copy link
Contributor

bors commented Mar 3, 2019

☀️ Test successful - checks-cirrus, checks-travis, status-appveyor
Approved by: gnzlbg
Pushing 750a7a6 to master...

@bors bors merged commit 96eb1b2 into rust-lang:master Mar 3, 2019
@mati865 mati865 deleted the docker-images-working branch March 4, 2019 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants