Skip to content
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

dragon: init at 18.04.3 #45315

Merged
merged 2 commits into from Aug 19, 2018
Merged

dragon: init at 18.04.3 #45315

merged 2 commits into from Aug 19, 2018

Conversation

JonathanReeve
Copy link
Contributor

Add Dragon Player, the KDE video and media player, so that KDE
users will have a way to play videos and audio files.

Motivation for this change

There is currently no video/media player for KDE in the Nixpkgs repos. However, the mechanism for sourcing the application is already provided by the KDE infrastructure in nixpkgs. So this just makes a formal package out of it, so that KDE users can play videos out-of-the-box.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Fits CONTRIBUTING.md.

Add Dragon Player, the KDE video and media player, so that KDE
users will have a way to play videos and audio files.
meta = {
license = with lib.licenses; [ gpl2 fdl12 ];
description = "A simple media player for KDE";
maintainers = [ lib.maintainers.ttuegel ];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You probably don't want to add @ttuegel as a maintainer :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're probably right. That was just copypasta. Do I add myself as a maintainer, and add that to this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just added myself as maintainer, and changed this line.

@ttuegel ttuegel merged commit ab328de into NixOS:master Aug 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants