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

Update dependencies #1446

Merged
merged 2 commits into from
Jul 7, 2018
Merged

Update dependencies #1446

merged 2 commits into from
Jul 7, 2018

Conversation

mati865
Copy link
Contributor

@mati865 mati865 commented Jun 25, 2018

This PR fixes all the warnings, tested on Linux.

@Diggsey
Copy link
Contributor

Diggsey commented Jul 1, 2018

@bors r+

@bors
Copy link
Contributor

bors commented Jul 1, 2018

📌 Commit 2178cdb has been approved by Diggsey

@bors
Copy link
Contributor

bors commented Jul 1, 2018

⌛ Testing commit 2178cdb with merge da2a21c...

bors added a commit that referenced this pull request Jul 1, 2018
Update dependencies

This PR fixes all the warnings, tested on Linux.
@bors
Copy link
Contributor

bors commented Jul 1, 2018

💔 Test failed - status-travis

@mati865
Copy link
Contributor Author

mati865 commented Jul 1, 2018

Issues revealed by CI:

  • 32bit *-linux-androideabi failure: Solaris ENOATTR build failure Stebalien/xattr#18

  • 64bit *-linux-android: fixed by deprecrated/net2-rs@d958a03, new release was made recently with fix

  • powerpc64le-unknown-linux-gnu: LLVM Error on PowerPC64LE: Cannot select ... rust#50499

    LLVM error (click here to expand)
    LLVM ERROR: Cannot select: 0x7f006fa7d0d0: ch = PPCISD::STBRX<ST6[%675](align=1)>   0x7f006e017840, 0x7f006e1939c0, 0x7f006e0167b8, ValueType:ch:i48
    0x7f006e1939c0: i32 = truncate 0x7f006e193068
      0x7f006e193068: i64 = srl 0x7f006e016680, Constant:i32<16>
        0x7f006e016680: i64 = or 0x7f006e224f70, 0x7f006fa45b60
          0x7f006e224f70: i64,ch = CopyFromReg 0x7f006e017840, Register:i64 %249
            0x7f006e016000: i64 = Register %249
          0x7f006fa45b60: i64 = shl nuw nsw 0x7f006e193410, Constant:i32<16>
            0x7f006e193410: i64 = AssertZext 0x7f006e016af8, ValueType:ch:i8
              0x7f006e016af8: i64,ch = CopyFromReg 0x7f006e017840, Register:i64 %252
                0x7f006e1936e8: i64 = Register %252
            0x7f006e016bc8: i32 = Constant<16>
        0x7f006e016bc8: i32 = Constant<16>
    0x7f006e0167b8: i64,ch = CopyFromReg 0x7f006e017840, Register:i64 %200
      0x7f006e016f70: i64 = Register %200
    In function: _ZN6base646decode6decode17hddc925063b1fb17bE
    error: Could not compile `hyper`.
    
  • x86_64-unknown-netbsd

    libc issue? (click here to expand)
    note: /usr/local/lib/gcc/x86_64-unknown-netbsd/5.3.0/../../../../x86_64-unknown-netbsd/bin/ld: /src/target/x86_64-unknown-netbsd/release/deps/libopenssl_sys-5b5efcbe61655b57.rlib(randfile.o): undefined reference to symbol 'setvbuf'
            /usr/local/x86_64-unknown-netbsd/lib/libc.so.12: error adding symbols: DSO missing from command line
            collect2: error: ld returned 1 exit status
    

EDIT: I'll bump dependencies again to see how much it helps when I get some time.
EDIT2: Linked upstream issue for PowerPC64LE

@mati865
Copy link
Contributor Author

mati865 commented Jul 2, 2018

Rebased with newer versions.
Android 64bit should get as far as 32bit now, it still needs fixing in libc or xattr crates.
PowerPC64LE and NetBSD will most likely fail again.

@Stebalien
Copy link

xattr v0.2.2 should fix builds on unsupported platforms

@mati865
Copy link
Contributor Author

mati865 commented Jul 6, 2018

@Stebalien thank you for fix and notification.

PR rebased with updated xattr and other minor bumps, PowerPC64LE issue will be fixed in Rust 1.28.
Updating hyper to 0.12 could workaround the issue but it would be breaking change.

@Diggsey
Copy link
Contributor

Diggsey commented Jul 7, 2018

@bors r+

@bors
Copy link
Contributor

bors commented Jul 7, 2018

📌 Commit f75f71b has been approved by Diggsey

@bors
Copy link
Contributor

bors commented Jul 7, 2018

⌛ Testing commit f75f71b with merge 1e62163...

bors added a commit that referenced this pull request Jul 7, 2018
Update dependencies

This PR fixes all the warnings, tested on Linux.
@bors
Copy link
Contributor

bors commented Jul 7, 2018

💔 Test failed - status-travis

@Diggsey Diggsey merged commit eae7116 into rust-lang:master Jul 7, 2018
@bors bors mentioned this pull request Jul 7, 2018
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