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

(#7323) Rename CMake files according to project name #7378

Merged
merged 2 commits into from Aug 6, 2022
Merged

(#7323) Rename CMake files according to project name #7378

merged 2 commits into from Aug 6, 2022

Conversation

clanghans
Copy link
Contributor

This PR addresses (#7323).

CMake installation output is attached. The files are renamed and the targets are working.

installation.txt

Rename FlatbuffersConfigVersion.cmake to match CMake project name
Rename FlatBuffersTargets to match CMake project name
@dbaileychess dbaileychess merged commit aa395e5 into google:master Aug 6, 2022
@dbaileychess
Copy link
Collaborator

Thanks!

sssooonnnggg pushed a commit to sssooonnnggg/flatbuffers that referenced this pull request Aug 8, 2022
* CMake find_package fixes (google#7323)

Rename FlatbuffersConfigVersion.cmake to match CMake project name

* CMake find_package fixes (google#7323)

Rename FlatBuffersTargets to match CMake project name
yan12125 pushed a commit to archlinuxcn/repo that referenced this pull request Aug 27, 2022
Upstream flatbuffers renamed the package from Flatbuffers to FlatBuffers
(capital B) [1].

Fixes the issue reported at
https://aur.archlinux.org/packages/python-onnxruntime#comment-879320

Also drops patches that are part of the latest version.

[1] google/flatbuffers#7378
archlinux-github pushed a commit to archlinux/aur that referenced this pull request Aug 27, 2022
Upstream flatbuffers renamed the package from Flatbuffers to FlatBuffers
(capital B) [1].

Fixes the issue reported at
https://aur.archlinux.org/packages/python-onnxruntime#comment-879320

Also drops patches that are part of the latest version.

[1] google/flatbuffers#7378
@SpaceIm
Copy link

SpaceIm commented Aug 31, 2022

It's a breaking change (as you can see in linked commit of archlinux), find_package(Flatbuffers CONFIG) won't work anymore. Would have been a better idea to rename this file to flatbuffers-config.cmake to be case insensitive.

@clanghans
Copy link
Contributor Author

@SpaceIm you are right. This slipped me. I will open an issue and propose a PR to fix it right away.

@clanghans
Copy link
Contributor Author

Created #7502 to address this issue.

Sorry for any inconvenience caused.

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.

None yet

3 participants