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
rclone-browser: init at 1.2 #44589
rclone-browser: init at 1.2 #44589
Conversation
pkgs/top-level/all-packages.nix
Outdated
@@ -18129,7 +18129,8 @@ with pkgs; | |||
}; | |||
|
|||
rclone = callPackage ../applications/networking/sync/rclone { }; | |||
|
|||
rclone-browser = callPackage ../applications/networking/sync/rclone/browser.nix { }; |
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.
one empty line inbetween please
description = "Graphical Frontend to Rclone written in Qt"; | ||
homepage = https://github.com/mmozeiko/RcloneBrowser; | ||
license = licenses.unlicense; | ||
platforms = platforms.all; |
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.
Did you test on macOS?
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.
Fails on Darwin:
CMake Error at CMakeLists.txt:14 (find_package):
By not providing "FindQt5MacExtras.cmake" in CMAKE_MODULE_PATH this project
has asked CMake to find a package configuration file provided by
"Qt5MacExtras", but CMake did not find one.
Could not find a package configuration file provided by "Qt5MacExtras" with
any of the following names:
Qt5MacExtrasConfig.cmake
qt5macextras-config.cmake
Add the installation prefix of "Qt5MacExtras" to CMAKE_PREFIX_PATH or set
"Qt5MacExtras_DIR" to a directory containing one of the above files. If
"Qt5MacExtras" provides a separate development package or SDK, be sure it
has been installed.
-- Configuring incomplete, errors occurred!
See also "/tmp/nix-build-rclone-browser-1.2.drv-0/source/build/CMakeFiles/CMakeOutput.log".
builder for '/nix/store/bbcpjpzqhnnalljyafvb9zfbdzzxlj7q-rclone-browser-1.2.drv' failed with exit code 1
error: build of '/nix/store/bbcpjpzqhnnalljyafvb9zfbdzzxlj7q-rclone-browser-1.2.drv' failed
sha256 = "1ldradd5c606mfkh46y4mhcvf9kbjhamw0gahksp9w43h5dh3ir7"; | ||
}; | ||
buildInputs = [ qt5.qtbase cmake ]; | ||
meta = with stdenv.lib; { |
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.
This could also use some whitespace.
src = fetchFromGitHub { | ||
owner = "mmozeiko"; | ||
repo = "RcloneBrowser"; | ||
rev = "${version}"; |
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.
rev = version
@@ -0,0 +1,20 @@ | |||
{ stdenv, fetchFromGitHub , rclone, qt5,cmake }: |
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.
rclone
seems not to be used
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.
Please incorporate dotlambda's feedback
I would rather close the PR, since it isn't going to work as expected. |
What isn't working? It's working fine for me. |
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)