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
libqmlbind: init at 0.2.0 #23547
libqmlbind: init at 0.2.0 #23547
Conversation
Can you rename the commit title to match the pull request title? |
|
||
src = fetchgit { | ||
url = "https://github.com/seanchas116/libqmlbind"; | ||
rev = "7ca90d1993934cd7748da7ed89484802290b9489"; |
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.
The following should also work here:
rev = "refs/tags/v${version}";
|
||
meta = with stdenv.lib; { | ||
homepage = https://github.com/seanchas116/libqmlbind; | ||
description = "C bindings for QML, used for binding to other languages"; |
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.
license = with licenses; [ mit fdl13 ];
Do you want to maintain this package?
preBuild = '' | ||
pushd . | ||
doxygen Doxyfile | ||
popd |
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.
pushd .
is a no-op here and can be removed.
@@ -49,6 +48,6 @@ stdenv.mkDerivation rec { | |||
homepage = https://github.com/seanchas116/libqmlbind; | |||
description = "C bindings for QML, used for binding to other languages"; | |||
license = with licenses; [ mit fdl13 ]; | |||
maintainers = [ "0BobTheJanitor <Changing Email>" ] | |||
maintainers = [ "0BobTheJanitor <Changing Email>" ]; |
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.
Could please an entry to lib/maintainers.nix
instead and reference it here.
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.
I'm currently getting a new email and domain, but I can add it now and just update it later.
I am going to switch to a different library, don't see the need to have this merged anymore |
Motivation for this change
C bindings for QtQuick, since a lot of programming languages have issues dealing with C++, and so is useful for writing Qt bindings to other languages. Not the highest priority package, since the only language I know that uses these bindings is Ruby, Swift (which I believe was abandoned), and my bindings for Crystal, but I thought I'd submit a pull request anyways.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)