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

[uthenticode] Add new port #11199

Merged
merged 7 commits into from
May 8, 2020
Merged

Conversation

woodruffw
Copy link
Contributor

Adds a port for uthenticode, a cross-platform library for verifying Authenticode signatures.

  • What does your PR fix? Fixes issue #

None.

  • Which triplets are supported/not supported? Have you updated the CI baseline?

All default triplets should work.

I believe so!

@BillyONeal
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@woodruffw
Copy link
Contributor Author

woodruffw commented May 6, 2020

Looking at the artifact logs, it looks like the problem might be some gcc/clang-style flags that are passed on Debug builds:

[1/3] "C:\PROGRA~2\Microsoft Visual Studio\2019\Enterprise\VC\Tools\MSVC\14.25.28610\bin\Hostx64\x64\cl.exe"   /TP -DUTHENTICODE_VERSION=\"1.0.0\" -Duthenticode_EXPORTS -IC:\agent\_work\1\s\buildtrees\uthenticode\src\v1.0.0-925e2c938f\src\include -IC:\agent\_work\1\s\installed\x64-windows\include /DWIN32 /D_WINDOWS /W3 /utf-8 /GR /EHsc /MP  /D_DEBUG /MDd /Z7 /Ob0 /Od /RTC1    -Wall -Wextra -pedantic -Werror -fsanitize=address -std:c++17 /showIncludes /Fosrc\CMakeFiles\uthenticode.dir\uthenticode.cpp.obj /Fdsrc\CMakeFiles\uthenticode.dir\ /FS -c C:\agent\_work\1\s\buildtrees\uthenticode\src\v1.0.0-925e2c938f\src\uthenticode.cpp
FAILED: src/CMakeFiles/uthenticode.dir/uthenticode.cpp.obj 
"C:\PROGRA~2\Microsoft Visual Studio\2019\Enterprise\VC\Tools\MSVC\14.25.28610\bin\Hostx64\x64\cl.exe"   /TP -DUTHENTICODE_VERSION=\"1.0.0\" -Duthenticode_EXPORTS -IC:\agent\_work\1\s\buildtrees\uthenticode\src\v1.0.0-925e2c938f\src\include -IC:\agent\_work\1\s\installed\x64-windows\include /DWIN32 /D_WINDOWS /W3 /utf-8 /GR /EHsc /MP  /D_DEBUG /MDd /Z7 /Ob0 /Od /RTC1    -Wall -Wextra -pedantic -Werror -fsanitize=address -std:c++17 /showIncludes /Fosrc\CMakeFiles\uthenticode.dir\uthenticode.cpp.obj /Fdsrc\CMakeFiles\uthenticode.dir\ /FS -c C:\agent\_work\1\s\buildtrees\uthenticode\src\v1.0.0-925e2c938f\src\uthenticode.cpp
Microsoft (R) C/C++ Optimizing Compiler Version 19.25.28614 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.

cl : Command line error D8021 : invalid numeric argument '/Wextra'
ninja: build stopped: subcommand failed.

Weirdly enough, though, these don't cause issues on our own CI (which also does Debug builds). I'm making a patch release to exclude them on Windows to see if that helps.

See: trailofbits/uthenticode#24

ports/uthenticode/portfile.cmake Show resolved Hide resolved
ports/uthenticode/portfile.cmake Show resolved Hide resolved
@NancyLi1013 NancyLi1013 added info:reviewed Pull Request changes follow basic guidelines and removed waiting for response labels May 8, 2020
@NancyLi1013
Copy link
Contributor

LGTM now, thanks for this PR.

@NancyLi1013 NancyLi1013 removed the info:reviewed Pull Request changes follow basic guidelines label May 8, 2020
@woodruffw
Copy link
Contributor Author

The current CI failure looks unrelated to this changeset.

@NancyLi1013
Copy link
Contributor

Yes, the regressions for libqcow on Linux is not related to this PR. You can ignore it.
We will re run this once solved.

@NancyLi1013 NancyLi1013 added the info:reviewed Pull Request changes follow basic guidelines label May 8, 2020
@strega-nil
Copy link
Contributor

Alright cool! Thanks so much @woodruffw :D

@strega-nil strega-nil merged commit a1c8879 into microsoft:master May 8, 2020
@woodruffw
Copy link
Contributor Author

woodruffw commented May 8, 2020 via email

@woodruffw woodruffw deleted the ww/uthenticode branch May 8, 2020 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants