Skip to content

Rename library for vcpkg.#140

Merged
ArthurSonzogni merged 1 commit intomasterfrom
vcpkg-rename-lib
Jul 6, 2021
Merged

Rename library for vcpkg.#140
ArthurSonzogni merged 1 commit intomasterfrom
vcpkg-rename-lib

Conversation

@ArthurSonzogni
Copy link
Owner

@ArthurSonzogni ArthurSonzogni commented Jul 4, 2021

Instead of putting the archive as:

  • <lib_dir>/ftxui/libcomponent.a
  • <lib_dir>/ftxui/libdom.a
  • <lib_dir>/ftxui/libscreen.a

Use:

  • <lib_dir>/libftxui-component.a
  • <lib_dir>/libftxui-dom.a
  • <lib_dir>/libftxui-screen.a

This fixes an issue users have with VCPKG.

See:
microsoft/vcpkg#16327 (comment)

@ArthurSonzogni
Copy link
Owner Author

Hi @myd7349, could you please take a look?

Instead of putting the archive as:
- <lib_dir>/ftxui/libcomponent.a
- <lib_dir>/ftxui/libdom.a
- <lib_dir>/ftxui/libscreen.a

Use:
- <lib_dir>/libftxui-component.a
- <lib_dir>/libftxui-dom.a
- <lib_dir>/libftxui-screen.a

This fixes an issue users have with VCPKG.

See:
microsoft/vcpkg#16327 (comment)
@ArthurSonzogni ArthurSonzogni merged commit 82adc3b into master Jul 6, 2021
@ArthurSonzogni ArthurSonzogni deleted the vcpkg-rename-lib branch July 6, 2021 00:10
@myd7349
Copy link
Contributor

myd7349 commented Jul 9, 2021

Hi! @ArthurSonzogni Sorry for the long delay.
I tested on my PC this afternoon, and it works as expected.

microsoft/vcpkg#18360 is already merged, without this patch however.
I think we can update vcpkg's port file later.

ArthurSonzogni pushed a commit that referenced this pull request Feb 19, 2022
PREFIX is by default "lib", which most of the libraries use.
this changes makes the library files libftxui-foo.a
as #140 intended originally
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