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

Fix Maven Sha256 Checksum Issue #22600

Merged
merged 1 commit into from
Oct 25, 2024
Merged

Fix Maven Sha256 Checksum Issue #22600

merged 1 commit into from
Oct 25, 2024

Conversation

idiskyle
Copy link
Contributor

Description

Changes applied to maven related signing:

  • Windows sha256 file encoded by utf8(no BOM)
  • powershell script task used latest version, previous 5.1 version only supports utf8 with BOM.
  • Windows sha256 file content in format 'sha256value *filename.extension'.
  • Linux sha256 file content in format 'sha256value *filename.extension'.

More information about powershell encoding:
Windows powershell encoding reference: about_Character_Encoding - PowerShell | Microsoft Learn

  • for version 5.1, it only has 'UTF8 Uses UTF-8 (with BOM).'
  • for version v7.1 and higher, it has:
    utf8: Encodes in UTF-8 format (no BOM).
    utf8BOM: Encodes in UTF-8 format with Byte Order Mark (BOM)
    utf8NoBOM: Encodes in UTF-8 format without Byte Order Mark (BOM)

Motivation and Context

@idiskyle idiskyle requested a review from a team as a code owner October 25, 2024 08:58
Copy link
Member

@snnn snnn left a comment

Choose a reason for hiding this comment

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

Thanks!

@snnn snnn merged commit 10bdf6e into main Oct 25, 2024
183 of 192 checks passed
@snnn snnn deleted the zhanxi/fix_maven_checksum branch October 25, 2024 15:13
@sophies927 sophies927 added release:1.20.1 triage:approved Approved for cherrypicks for release and removed triage:approved Approved for cherrypicks for release labels Oct 31, 2024
@sophies927 sophies927 added the triage:approved Approved for cherrypicks for release label Nov 11, 2024
yf711 pushed a commit that referenced this pull request Nov 11, 2024
### Description
<!-- Describe your changes. -->
**Changes applied to maven related signing:** 
* Windows sha256 file encoded by utf8(no BOM)
* powershell script task used latest version, previous 5.1 version only
supports utf8 with BOM.
* Windows sha256 file content in format 'sha256value
*filename.extension'.
* Linux sha256 file content in format 'sha256value *filename.extension'.

**More information about powershell encoding:**
Windows powershell encoding reference: [about_Character_Encoding -
PowerShell | Microsoft
Learn](https://learn.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_character_encoding?view=powershell-7.4)
- for version 5.1, it only has 'UTF8 Uses UTF-8 (with BOM).'
- for version v7.1 and higher, it has:
     utf8: Encodes in UTF-8 format (no BOM).
     utf8BOM: Encodes in UTF-8 format with Byte Order Mark (BOM)
     utf8NoBOM: Encodes in UTF-8 format without Byte Order Mark (BOM)
@sophies927 sophies927 added the cherry-picked Cherry-picked for a cherrypicks branch label Nov 18, 2024
ishwar-raut1 pushed a commit to ishwar-raut1/onnxruntime that referenced this pull request Nov 19, 2024
### Description
<!-- Describe your changes. -->
**Changes applied to maven related signing:** 
* Windows sha256 file encoded by utf8(no BOM)
* powershell script task used latest version, previous 5.1 version only
supports utf8 with BOM.
* Windows sha256 file content in format 'sha256value
*filename.extension'.
* Linux sha256 file content in format 'sha256value *filename.extension'.

**More information about powershell encoding:**
Windows powershell encoding reference: [about_Character_Encoding -
PowerShell | Microsoft
Learn](https://learn.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_character_encoding?view=powershell-7.4)
- for version 5.1, it only has 'UTF8 Uses UTF-8 (with BOM).'
- for version v7.1 and higher, it has:
     utf8: Encodes in UTF-8 format (no BOM).
     utf8BOM: Encodes in UTF-8 format with Byte Order Mark (BOM)
     utf8NoBOM: Encodes in UTF-8 format without Byte Order Mark (BOM)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-picked Cherry-picked for a cherrypicks branch release:1.20.1 triage:approved Approved for cherrypicks for release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants