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

starship: update to 1.17.0 #19580

Merged
merged 1 commit into from
Dec 31, 2023
Merged

starship: update to 1.17.0 #19580

merged 1 commit into from
Dec 31, 2023

Conversation

ognevny
Copy link
Collaborator

@ognevny ognevny commented Dec 31, 2023

patches are not needed now

patches are not needed now
@Biswa96 Biswa96 merged commit 9015227 into msys2:master Dec 31, 2023
8 checks passed
@ognevny ognevny deleted the update-starship branch December 31, 2023 12:52
@ognevny
Copy link
Collaborator Author

ognevny commented Dec 31, 2023

clangarm64 fails with

C:/Users/Administrator/.cargo/registry/src/index.crates.io-6f17d22bba15001f/libz-ng-sys-1.1.12/src/zlib-ng/arch/arm/neon_intrins.h:5:12: fatal error: 'arm64_neon.h' file not found
        5 | #  include <arm64_neon.h>
          |            ^~~~~~~~~~~~~~
    1 warning and 1 error generated.
    make[2]: *** [CMakeFiles/zlib.dir/build.make:524: CMakeFiles/zlib.dir/arch/arm/compare256_neon.c.obj] Error 1
    make[2]: *** Waiting for unfinished jobs....
    warning: unknown warning option '-Wno-pedantic-ms-format'; did you mean '-Wno-pedantic-macros'? [-Wunknown-warning-option]
    1 warning generated.
    make[1]: *** [CMakeFiles/Makefile2:83: CMakeFiles/zlib.dir/all] Error 2
    make: *** [Makefile:136: all] Error 2
    thread 'main' panicked at C:\Users\Administrator\.cargo\registry\src\index.crates.io-6f17d22bba15001f\cmake-0.1.50\src\lib.rs:1098:5:

@Biswa96
Copy link
Member

Biswa96 commented Dec 31, 2023

@ognevny
Copy link
Collaborator Author

ognevny commented Dec 31, 2023

This condition needs to be modified somehow

#if !defined(__MINGW32__)...?

edit: I think it's vendor by cargo vendor. I sadly has no test machine for it. I can try to fix it, but not very soon

@Biswa96
Copy link
Member

Biswa96 commented Dec 31, 2023

Maybe || defined(__aarch64__)

lazka added a commit that referenced this pull request Dec 31, 2023
This reverts commit 9015227.

Fails to build for clangarm64, see:
#19580 (comment)
@lazka
Copy link
Member

lazka commented Dec 31, 2023

(reverted for now)

@ognevny
Copy link
Collaborator Author

ognevny commented Dec 31, 2023

(reverted for now)

oh, maybe it would be better to just disable clangarm64... but ok

@lazka
Copy link
Member

lazka commented Dec 31, 2023

Also fine with me. Let's ask @naveen521kk who added the package in the first place.

@naveen521kk
Copy link
Member

Let's ask @naveen521kk who added the package in the first place.

Fine with me :)

@ognevny
Copy link
Collaborator Author

ognevny commented Jan 1, 2024

in libz-ng-sys crate's repository I found this (maybe someone can help with it) and this (zlib-ng source is always vendored)

@mati865
Copy link
Collaborator

mati865 commented Jan 7, 2024

It should be fixed upstream by zlib-ng/zlib-ng@f9228d8
We can wait for a new release with rust-lang/libz-sys#162 or patch the vendored sources.

@ognevny
Copy link
Collaborator Author

ognevny commented Jan 7, 2024

It should be fixed upstream by zlib-ng/zlib-ng@f9228d8
We can wait for a new release with rust-lang/libz-sys#162 or patch the vendored sources.

ok. it's better to wait IMO

@ognevny
Copy link
Collaborator Author

ognevny commented Jan 7, 2024

opened a suggestion to bump crate's version
rust-lang/libz-sys#166

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.

5 participants