Skip to content

[licensepp] update to 1.2.0#48065

Merged
BillyONeal merged 2 commits into
microsoft:masterfrom
toge:licensepp-1.2.0
Nov 3, 2025
Merged

[licensepp] update to 1.2.0#48065
BillyONeal merged 2 commits into
microsoft:masterfrom
toge:licensepp-1.2.0

Conversation

@toge
Copy link
Copy Markdown
Contributor

@toge toge commented Nov 1, 2025

  • Changes comply with the maintainer guide.
  • SHA512s are updated for each updated download.
  • The "supports" clause reflects platforms that may be fixed by this new version.
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

https://github.com/abumq/licensepp/releases/tag/v1.2.0

Copy link
Copy Markdown
Member

@BillyONeal BillyONeal left a comment

Choose a reason for hiding this comment

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

Thanks!

#ifndef LICENSEPP_License_h
#define LICENSEPP_License_h
-
+#include <stdint.h>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It makes me very sad that this isn't landed yet but it's submitted as abumq/licensepp#48

- set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -O3 -Wall -Werror")
+ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -O3 -Wall")
- set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++14 -O3 -Wall -Werror -Wno-return-stack-address")
+ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++14 -O3 -Wall -Wno-return-stack-address")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yikes! That's a scary warning to disable. But not actually in the patch :/

@BillyONeal BillyONeal merged commit acbbfad into microsoft:master Nov 3, 2025
18 checks passed
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.

2 participants