[GH-ISSUE #392] [vcpkg] license #312

Closed
opened 2026-05-05 12:08:03 -06:00 by gitea-mirror · 13 comments
Owner

Originally created by @Teles1 on GitHub (Feb 3, 2023).
Original GitHub issue: https://github.com/jmcnamara/libxlsxwriter/issues/392

I submitted a pull request and they recommended updating the license agreement in there in order to make it complete. I thought it was Free BSD but the header files mention something else. Perhaps could you submit a pull request to vcpkg updating it?
Here is the PR I sent-> https://github.com/microsoft/vcpkg/pull/29401

Originally created by @Teles1 on GitHub (Feb 3, 2023). Original GitHub issue: https://github.com/jmcnamara/libxlsxwriter/issues/392 I submitted a pull request and they recommended updating the license agreement in there in order to make it complete. I thought it was Free BSD but the header files mention something else. Perhaps could you submit a pull request to vcpkg updating it? Here is the PR I sent-> https://github.com/microsoft/vcpkg/pull/29401
Author
Owner

@FrankXie05 commented on GitHub (Feb 3, 2023):

@Teles1 I don't think the upstream licenses did anything wrong, they just mentioned all the licenses involved in the files they used. This is normal behavior. If you want to add Free BSD licenses, you can suggest that they separate all licenses.
Take it out and perfect it. 🧐

<!-- gh-comment-id:1414725054 --> @FrankXie05 commented on GitHub (Feb 3, 2023): @Teles1 I don't think the upstream licenses did anything wrong, they just mentioned all the licenses involved in the files they used. This is normal behavior. If you want to add `Free BSD` licenses, you can suggest that they separate all licenses. Take it out and perfect it. 🧐
Author
Owner

@Teles1 commented on GitHub (Feb 3, 2023):

@FrankXie05 Licenses are just a nightmare to deal with. That's way too much work for the creator. I'll leave the issue here none the less until the creator sees it.

<!-- gh-comment-id:1414733095 --> @Teles1 commented on GitHub (Feb 3, 2023): @FrankXie05 Licenses are just a nightmare to deal with. That's way too much work for the creator. I'll leave the issue here none the less until the creator sees it.
Author
Owner

@FrankXie05 commented on GitHub (Feb 3, 2023):

For open source and sharing, the license is a must. It constrains others to use the open source library under the conditions of the license. Although this sounds contradictory, it is the fundamental guarantee for using the open source library.
Even if it's a headache to hwo to add the license. 😣

<!-- gh-comment-id:1414761119 --> @FrankXie05 commented on GitHub (Feb 3, 2023): For open source and sharing, the license is a must. It constrains others to use the open source library under the conditions of the license. Although this sounds contradictory, it is the fundamental guarantee for using the open source library. Even if it's a headache to hwo to add the license. 😣
Author
Owner

@jmcnamara commented on GitHub (Feb 3, 2023):

Folks I would prefer the license field to say FreeBSD or link to the license file.

I put a lot of work into clarifying the main license in the docs and the licences of all of the optional sub-components. I've also detailed how any of the sub-licensed components can be compiled in/out. The issue there, from a vcpkg point of view, is that adding those additional compilation options to compile out the embedded libraries and replace them with system libraries will make the compilation more error prone (in particular openssl).

But either way a license file of null feels sub-optimal.

<!-- gh-comment-id:1415623316 --> @jmcnamara commented on GitHub (Feb 3, 2023): Folks I would prefer the license field to say FreeBSD or link to the license file. I put a lot of work into clarifying the main license in the [docs](https://libxlsxwriter.github.io/license.html) and the licences of all of the optional sub-components. I've also detailed how any of the sub-licensed components can be [compiled in/out](http://libxlsxwriter.github.io/getting_started.html#gsg_compilation_options). The issue there, from a `vcpkg` point of view, is that adding those additional compilation options to compile out the embedded libraries and replace them with system libraries will make the compilation more error prone (in particular openssl). But either way a license file of `null` feels sub-optimal.
Author
Owner

@Teles1 commented on GitHub (Feb 3, 2023):

Oh right. Fair enough. I tried putting in OpenBSD as the license in there in first place but can't merge it in hence the mixture of licenses. And I have to agree with @jmcnamara my nightmare when I was getting started with c++ was libraries with multiple dependencies and back then i didn't know how to put them together without having to compile every single one with MSVC.

<!-- gh-comment-id:1415744433 --> @Teles1 commented on GitHub (Feb 3, 2023): Oh right. Fair enough. I tried putting in OpenBSD as the license in there in first place but can't merge it in hence the mixture of licenses. And I have to agree with @jmcnamara my nightmare when I was getting started with c++ was libraries with multiple dependencies and back then i didn't know how to put them together without having to compile every single one with MSVC.
Author
Owner

@jmcnamara commented on GitHub (Feb 3, 2023):

@Teles1 do you know what are the allowed licences in that field? The specific license for libxlsxwriter is: BSD 2-Clause "Simplified" License. https://spdx.org/licenses/BSD-2-Clause.html

<!-- gh-comment-id:1415782419 --> @jmcnamara commented on GitHub (Feb 3, 2023): @Teles1 do you know what are the allowed licences in that field? The specific license for libxlsxwriter is: BSD 2-Clause "Simplified" License. https://spdx.org/licenses/BSD-2-Clause.html
Author
Owner

@jmcnamara commented on GitHub (Feb 3, 2023):

I'll answer my own question by reading the docs :-)

The license of the port. This is an SPDX license expression, or null for proprietary licenses and other licenses for which one should "just read the copyright file" (e.g., Qt).

So, the license field should be the SPDX identifier BSD-2-Clause or if you feel that isn't complete, I think it is, then set it to null.

<!-- gh-comment-id:1415791544 --> @jmcnamara commented on GitHub (Feb 3, 2023): I'll answer my own question by reading the [docs](https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/manifest-files.md#license) :-) > The license of the port. This is an [SPDX license expression](https://spdx.dev/ids/#how), or null for proprietary licenses and other licenses for which one should "just read the copyright file" (e.g., Qt). So, the license field should be the SPDX identifier `BSD-2-Clause` or if you feel that isn't complete, I think it is, then set it to `null`.
Author
Owner

@Teles1 commented on GitHub (Feb 3, 2023):

Let's wait for @FrankXie05 He's the specialist in this case lol. But I did try BSD and then Frank mentioned this. Thank you for spending some of your time on this.

<!-- gh-comment-id:1415823760 --> @Teles1 commented on GitHub (Feb 3, 2023): Let's wait for @FrankXie05 He's the specialist in this case lol. But I did try BSD and then Frank [mentioned this](https://github.com/microsoft/vcpkg/pull/29401#issuecomment-1414702679). Thank you for spending some of your time on this.
Author
Owner

@Teles1 commented on GitHub (Feb 3, 2023):

To be fair I think there is something else wrong with vcpkg + libxlswriter. I just tried using it and I can't import it on my cmake project even after the fact that I installed it. It can't find it. Works fine on Visual Studio tho.

I tried
find_package(xlsxwriter CONFIG REQUIRED)
and
find_package(libxlsxwriterCONFIG REQUIRED)

<!-- gh-comment-id:1415836798 --> @Teles1 commented on GitHub (Feb 3, 2023): To be fair I think there is something else wrong with vcpkg + libxlswriter. I just tried using it and I can't import it on my cmake project even after the fact that I installed it. It can't find it. Works fine on Visual Studio tho. I tried `find_package(xlsxwriter CONFIG REQUIRED)` and `find_package(libxlsxwriterCONFIG REQUIRED)`
Author
Owner

@Teles1 commented on GitHub (Feb 3, 2023):

Yah, it works fine standalone cmake + add_subdirectory.
`add_subdirectory(libxlsxwriter)

target_link_libraries(excel PRIVATE xlsxwriter)`

Something to do with vcpkg export methodology that I am not familiar with.

<!-- gh-comment-id:1415868529 --> @Teles1 commented on GitHub (Feb 3, 2023): Yah, it works fine standalone cmake + add_subdirectory. `add_subdirectory(libxlsxwriter) target_link_libraries(excel PRIVATE xlsxwriter)` Something to do with vcpkg export methodology that I am not familiar with.
Author
Owner

@FrankXie05 commented on GitHub (Feb 6, 2023):

@Teles1 do you know what are the allowed licences in that field? The specific license for libxlsxwriter is: BSD 2-Clause "Simplified" License. https://spdx.org/licenses/BSD-2-Clause.html

@Teles1 We respect the choice of the upstream, since the author stated that the specific license is BSD-2-Clause, we also allow the license to be added to port libxlsxwriter. I will complete this task today. :)

To be fair I think there is something else wrong with vcpkg + libxlswriter. I just tried using it and I can't import it on my cmake project even after the fact that I installed it. It can't find it. Works fine on Visual Studio tho.

I tried find_package(xlsxwriter CONFIG REQUIRED) and find_package(libxlsxwriterCONFIG REQUIRED)

Maybe you missed the CMAKE_TOOLCHAIN_FILE.

<!-- gh-comment-id:1418405398 --> @FrankXie05 commented on GitHub (Feb 6, 2023): > @Teles1 do you know what are the allowed licences in that field? The specific license for libxlsxwriter is: BSD 2-Clause "Simplified" License. https://spdx.org/licenses/BSD-2-Clause.html @Teles1 We respect the choice of the upstream, since the author stated that the specific license is `BSD-2-Clause`, we also allow the license to be added to port `libxlsxwriter`. I will complete this task today. :) > To be fair I think there is something else wrong with vcpkg + libxlswriter. I just tried using it and I can't import it on my cmake project even after the fact that I installed it. It can't find it. Works fine on Visual Studio tho. > > I tried `find_package(xlsxwriter CONFIG REQUIRED)` and `find_package(libxlsxwriterCONFIG REQUIRED)` Maybe you missed the `CMAKE_TOOLCHAIN_FILE`.
Author
Owner

@Teles1 commented on GitHub (Feb 6, 2023):

I'm using vcpkg for other libraries and couldn't get it to work. It may
just be me being dumb too. Thank you sir for your great work

On Sun, Feb 5, 2023, 9:24 p.m. Frank @.***> wrote:

@Teles1 https://github.com/Teles1 do you know what are the allowed
licences in that field? The specific license for libxlsxwriter is: BSD
2-Clause "Simplified" License. https://spdx.org/licenses/BSD-2-Clause.html

@Teles1 https://github.com/Teles1 We respect the choice of the
upstream, since the author stated that the specific license is
BSD-2-Clause, we also allow the license to be added to port libxlsxwriter.
I will complete this task today. :)

To be fair I think there is something else wrong with vcpkg +
libxlswriter. I just tried using it and I can't import it on my cmake
project even after the fact that I installed it. It can't find it. Works
fine on Visual Studio tho.

I tried find_package(xlsxwriter CONFIG REQUIRED) and find_package(libxlsxwriterCONFIG
REQUIRED)

Maybe you missed the CMAKE_TOOLCHAIN_FILE.


Reply to this email directly, view it on GitHub
https://github.com/jmcnamara/libxlsxwriter/issues/392#issuecomment-1418405398,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP5QR4DLWEMRMK23AZBAJGLWWBOETANCNFSM6AAAAAAUPX6QQQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

<!-- gh-comment-id:1418492652 --> @Teles1 commented on GitHub (Feb 6, 2023): I'm using vcpkg for other libraries and couldn't get it to work. It may just be me being dumb too. Thank you sir for your great work On Sun, Feb 5, 2023, 9:24 p.m. Frank ***@***.***> wrote: > @Teles1 <https://github.com/Teles1> do you know what are the allowed > licences in that field? The specific license for libxlsxwriter is: BSD > 2-Clause "Simplified" License. https://spdx.org/licenses/BSD-2-Clause.html > > @Teles1 <https://github.com/Teles1> We respect the choice of the > upstream, since the author stated that the specific license is > BSD-2-Clause, we also allow the license to be added to port libxlsxwriter. > I will complete this task today. :) > > To be fair I think there is something else wrong with vcpkg + > libxlswriter. I just tried using it and I can't import it on my cmake > project even after the fact that I installed it. It can't find it. Works > fine on Visual Studio tho. > > I tried find_package(xlsxwriter CONFIG REQUIRED) and find_package(libxlsxwriterCONFIG > REQUIRED) > > Maybe you missed the CMAKE_TOOLCHAIN_FILE. > > — > Reply to this email directly, view it on GitHub > <https://github.com/jmcnamara/libxlsxwriter/issues/392#issuecomment-1418405398>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AP5QR4DLWEMRMK23AZBAJGLWWBOETANCNFSM6AAAAAAUPX6QQQ> > . > You are receiving this because you were mentioned.Message ID: > ***@***.***> >
Author
Owner

@FrankXie05 commented on GitHub (Feb 6, 2023):

@Teles1 For the use of most of the ports provided by vcpkg. Just add two lines of code. :)
The first add find_package(), and second link your project: target_link_libraries(project_name PRIVATE xlsxwriter).

<!-- gh-comment-id:1418554157 --> @FrankXie05 commented on GitHub (Feb 6, 2023): @Teles1 For the use of most of the ports provided by vcpkg. Just add two lines of code. :) The first add `find_package()`, and second link your project: `target_link_libraries(project_name PRIVATE xlsxwriter)`.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/libxlsxwriter#312
No description provided.