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 build failure with CMake #116

Closed

Conversation

carlocab
Copy link
Contributor

@carlocab carlocab commented May 3, 2024

Without this, the build fails with

  Undefined symbols for architecture arm64:
    "fizz::extensions::generateDelegatedCredentialPEM(fizz::extensions::DelegatedCredential, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >)", referenced from:
        fizz::tool::fizzGenerateDelegatedCredentialCommand(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&) in FizzGenerateDelegatedCredentialCommand.cpp.o
  ld: symbol(s) not found for architecture arm64
  clang: error: linker command failed with exit code 1 (use -v to see invocation)

This was missed in 19dca4c.

See https://github.com/Homebrew/homebrew-core/actions/runs/8932339958/job/24536092094?pr=170668#step:3:326

Without this, the build fails with

```
  Undefined symbols for architecture arm64:
    "fizz::extensions::generateDelegatedCredentialPEM(fizz::extensions::DelegatedCredential, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >)", referenced from:
        fizz::tool::fizzGenerateDelegatedCredentialCommand(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&) in FizzGenerateDelegatedCredentialCommand.cpp.o
  ld: symbol(s) not found for architecture arm64
  clang: error: linker command failed with exit code 1 (use -v to see invocation)
```

This was missed in 19dca4c.

See https://github.com/Homebrew/homebrew-core/actions/runs/8932339958/job/24536092094?pr=170668#step:3:326
@facebook-github-bot
Copy link
Contributor

@mingtaoy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot pushed a commit to facebook/hhvm that referenced this pull request May 7, 2024
Summary:
Without this, the build fails with

```
  Undefined symbols for architecture arm64:
    "fizz::extensions::generateDelegatedCredentialPEM(fizz::extensions::DelegatedCredential, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >)", referenced from:
        fizz::tool::fizzGenerateDelegatedCredentialCommand(std::__1::vector<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > > const&) in FizzGenerateDelegatedCredentialCommand.cpp.o
  ld: symbol(s) not found for architecture arm64
  clang: error: linker command failed with exit code 1 (use -v to see invocation)
```

This was missed in 19dca4cbe227ad621be1b7bce916840a1b77b12c.

See https://github.com/Homebrew/homebrew-core/actions/runs/8932339958/job/24536092094?pr=170668#step:3:326

X-link: facebookincubator/fizz#116

Reviewed By: frqiu

Differential Revision: D57024059

Pulled By: mingtaoy

fbshipit-source-id: 0788db522ebd4ab516636415012e51316238ed96
@facebook-github-bot
Copy link
Contributor

@mingtaoy merged this pull request in b21c619.

@carlocab carlocab deleted the cmake-fix-pem-utils branch May 8, 2024 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants