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
libcerf: 1.5 -> 1.13 #69218
libcerf: 1.5 -> 1.13 #69218
Conversation
@GrahamcOfBorg build libcerf |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
generally the "v" prefix is left off in nixpkgs when specifying the version of the package, as it already is in a context that it's understood to be the version.
Also, I seem to get a build error now:
In file included from /build/cef_binary_3.3497.1833.g13f506f_linux64/libcef_dll/ctocpp/browser_ctocpp.h:28,
from /build/cef_binary_3.3497.1833.g13f506f_linux64/libcef_dll/cpptoc/display_handler_cpptoc.cc:16:
/build/cef_binary_3.3497.1833.g13f506f_linux64/libcef_dll/ctocpp/ctocpp_ref_counted.h:77:27: error: 'cfi-icall' attribute directive ignored [-Werror=attributes]
void UnderlyingAddRef() const {
^~~~~
/build/cef_binary_3.3497.1833.g13f506f_linux64/libcef_dll/ctocpp/ctocpp_ref_counted.h:85:28: error: 'cfi-icall' attribute directive ignored [-Werror=attributes]
bool UnderlyingRelease() const {
^~~~~
/build/cef_binary_3.3497.1833.g13f506f_linux64/libcef_dll/ctocpp/ctocpp_ref_counted.h:94:30: error: 'cfi-icall' attribute directive ignored [-Werror=attributes]
bool UnderlyingHasOneRef() const {
^~~~~
cc1plus: all warnings being treated as errors
make[3]: *** [libcef_dll_wrapper/CMakeFiles/libcef_dll_wrapper.dir/build.make:362: libcef_dll_wrapper/CMakeFiles/libcef_dll_wrapper.dir/cpptoc/display_handler_cpptoc.cc.o] Error 1
make[2]: *** [CMakeFiles/Makefile2:106: libcef_dll_wrapper/CMakeFiles/libcef_dll_wrapper.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:113: libcef_dll_wrapper/CMakeFiles/libcef_dll_wrapper.dir/rule] Error 2
make: *** [Makefile:118: libcef_dll_wrapper] Error 2
builder for '/nix/store/s6yhkwy7d5hp57iq38sbgyk4620wl3hy-cef-binary-3.3497.1833.g13f506f.drv' failed with exit code 2
error: build of '/nix/store/s6yhkwy7d5hp57iq38sbgyk4620wl3hy-cef-binary-3.3497.1833.g13f506f.drv' failed
Thanks for catching the superfluous I'm re-running |
Upstream has moved the src tarball download location and homepage for `libcerf`, so the current package is broken. While fixing it, let's upgrade to the latest version, which is built with cmake now. Co-Authored-By: Jon <jonringer@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Result of nix-review pr 69218
1
2 package were build:
- gnuplot (libsForQt512.khotkeys ,plasma5.kde-cli-tools ,plasma5.kdeplasma-addons ,libsForQt5.khotkeys ,plasma5.kmenuedit ,plasma5.plasma-desktop ,plasma5.plasma-workspace ,plasma5.powerdevil ,plasma5.systemsettings)
- libcerf (libsForQt512.khotkeys ,plasma5.kde-cli-tools ,plasma5.kdeplasma-addons ,libsForQt5.khotkeys ,plasma5.kmenuedit ,plasma5.plasma-desktop ,plasma5.plasma-workspace ,plasma5.powerdevil ,plasma5.systemsettings)
cef-binary build failure was fixed in #72190 |
Upstream has moved the src tarball download location and homepage for
libcerf
,so the current package is broken. While fixing it, let's upgrade to the latest
version, which is built with cmake now.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @orivej