-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
add notcurses/2.4.8 recipe #4036
add notcurses/2.4.8 recipe #4036
Conversation
This comment has been minimized.
This comment has been minimized.
Hrmm, clicking on "details" to fix up my build draws a 403 :/. |
This comment has been minimized.
This comment has been minimized.
@dankamongmen Basically:
|
thanks for the detailed feedback; i'll get on top of this and push anew |
I still (at least) need to generate that CMake wrapper and work out pkg-config integration. |
This comment has been minimized.
This comment has been minimized.
Here is a recipe relying on pkgconf (under condition in this case):
|
This comment has been minimized.
This comment has been minimized.
What about `notcurses++`?
Hrmmm. In Debian, libnotcurses.so and libnotcurses++.so are
delivered in two different packages. Most other distros only use
a single package. Everywhere, however, two .pc pkg-config files
are built. Many programs don't need libnotcurses++, and those
that do generally oughtn't need an explicit -lnotcurses. Ought I
make a second recipe for libnotcurses++?
|
This comment has been minimized.
This comment has been minimized.
Sounds to me like one recipe with two components (at least) should work well. Components will generate individual targets (e.g. CMake targets) people can link against. |
sounds perfect, thanks. i appreciate y'alls' patient assistance! i'm reading the conan-index guide, which seems wildly different from the original documentation i read, but i ought have something smashed together soon. |
1bee0f5
to
ea3a41c
Compare
This comment has been minimized.
This comment has been minimized.
ea3a41c
to
2c3fbfa
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
I've updated to 2.1.4 and smacked it into shape enough to build with
|
Down to a single warning:
|
bddcaea
to
d4073fb
Compare
This comment has been minimized.
This comment has been minimized.
e90c250
to
16ac7f5
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
16ac7f5
to
01fd16f
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
I've double-checked, package |
This comment has been minimized.
This comment has been minimized.
Failure in build 2 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This pull request has been automatically closed because it has not had recent activity. Thank you for your contributions. |
Specify library name and version: notcurses/2.1.2
conan-center hook activated.
This is my first recipe. I'm the upstream author. Please see dankamongmen/notcurses#1098 for background. Closes #4035.