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
nextcloud-client: 2.5.2 -> 2.5.3 #67306
Conversation
@Ma27 Can the patch be upstreamed? |
It ought to be fixed with |
That I'm aware of @jtojnar I don't think cmake has a way to do this. Guess we'll have to override the pkgconfig variable. |
Unfortunately I didn't find a proper way to override the value with a cmake flag. I thought that the simplest approach would be to explicitly set a different install destination in the cmake flag as you'd have to override the pkgconfig vars from Is that fine for you or would you suggest something else? : ) |
In the absence of ability to use I doubt it is upstreamable though. Opened an issue against CMake: https://gitlab.kitware.com/cmake/cmake/issues/19632 |
I'm aware of that. I figured that it's the best solution for now to add a simple patch which explicitly copies dbus configs to a store dir as Nix(OS) doesn't allow to imperatively alter global directories :) Will fix the review comment of @worldofpeace, then this should be good to go. |
https://github.com/nextcloud/desktop/releases/tag/v2.5.3 Also move explicitly dbus configuration files to the store path rather than `/etc/dbus-1/services`.
bc47d0a
to
8d62a39
Compare
@GrahamcOfBorg build nextcloud-client |
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.
Works for me, verified it's wrapped.
Continuation of NixOS#67306 * Switched to non-deprecated dbus config path. * Removed CMake flags for finding dependencies, the project can find them on its own. * Properly fixed the RPATH issue from NixOS#15406 – ${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_LIBDIR}. * Dropped changing Icon name in the desktop file as it was fixed upstream. * Switched to fetchFromGitHub, as the submodules are only for Win, Mac & crash reporting. * Dropped enableParallelBuilding = true, as it is on by default with CMake.
Motivation for this change
https://github.com/nextcloud/desktop/releases/tag/v2.5.3
Also move explicitly dbus configuration files to the store path rather than
/etc/dbus-1/services
.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 @