Skip to content

Conversation

@snnn
Copy link
Contributor

@snnn snnn commented Jun 11, 2025

  1. Updated abseil to the latest version, which fixed all BinSkim warnings so that we no longer need to patch it.
  2. However, the new abseil itself has some extra warnings, so I have to patch it.
  3. Added "--compile_no_warning_as_error" to windows 32-bit build. I will fix the warnings later.

@snnn snnn force-pushed the snnn/update_abseil branch from 42a52cf to 255017f Compare June 11, 2025 17:46
@snnn snnn changed the title snnn/update abseil Update abseil Jun 11, 2025
Changming Sun added 2 commits June 11, 2025 11:04
@snnn snnn marked this pull request as draft June 11, 2025 22:27
@snnn snnn marked this pull request as ready for review June 20, 2025 00:20
@snnn snnn merged commit bc95e99 into main Jun 20, 2025
89 checks passed
@snnn snnn deleted the snnn/update_abseil branch June 20, 2025 01:42
mingyueliuh pushed a commit to mingyueliuh/onnxruntime that referenced this pull request Jul 22, 2025
  - The build error started after onnxruntime updated abseil (microsoft#25033)
  - EP needs to add abseil target link
jywu-msft pushed a commit that referenced this pull request Jul 22, 2025
### Description
The build error started after ONNXRuntime update abseil (#25033). 
The EP needs to explicitly add the abseil target to its link
dependencies.


### Motivation and Context
For fix the issue #25486

Co-authored-by: liumingyue <[email protected]>
sanketkaleoss pushed a commit to sanketkaleoss/onnxruntime that referenced this pull request Aug 11, 2025
### Description
The build error started after ONNXRuntime update abseil (microsoft#25033). 
The EP needs to explicitly add the abseil target to its link
dependencies.


### Motivation and Context
For fix the issue microsoft#25486

Co-authored-by: liumingyue <[email protected]>
quic-ankus pushed a commit to CodeLinaro/onnxruntime that referenced this pull request Nov 25, 2025
1. Updated abseil to the latest version, which fixed all BinSkim
warnings so that we no longer need to patch it.
2. However, the new abseil itself has some extra warnings, so I have to
patch it.
3. Added "--compile_no_warning_as_error" to windows 32-bit build. I will
fix the warnings later.
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.

3 participants