-
Notifications
You must be signed in to change notification settings - Fork 200
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
Inclusion request of fruit into Conan Center #83
Comments
Hi! Did you have the time to check the points indicated above? Thanks! |
Not yet, sorry.
I've been a bit busy lately and didn't spend much time on Fruit.
I'll take a look when I have some time, sorry for the delay.
…On Mon, 25 Feb 2019, 17:14 Daniel, ***@***.***> wrote:
Hi!
Did you have the time to check the points indicated above?
We would be glad to have fruit packages available for Conan
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#83 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABeeF7-7siVzYu4bpEm5pn9B6MlVtpfyks5vRBoIgaJpZM4a0J53>
.
|
No problem. Ping me with any doubt you may have. Thanks! |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi guys!
My name is Daniel and I am one of the https://conan.io developers. I recently reviewed your inclusion request to include the Conan Packages for your library
fruit
into the central repository Conan Center.As you may know we are not asking for any requirement for library authors, only to keep in mind that you are responsible for maintaining the uploaded packages.
However, there are some things that might be interesting to do so I would like to give you some suggestions to improve its visibility and distribution:
fruit
package is installed -> Add aexports = "COPYING"
attribute to you conanfile.pypackage()
method:self.copy("COPYING", dst="licenses", ignore_case=True, keep_path=False)
.Most of this points are recommendations but please consider including the license in the packages as it is important for the distribution of the library itself.
If you have any question or need help with the changes do not hesitate to ask.
Thanks a lot!
The text was updated successfully, but these errors were encountered: