Skip to content

Commit

Permalink
subversion: add license
Browse files Browse the repository at this point in the history
  • Loading branch information
@orivej
orivej committed Aug 9, 2018
1 parent e6a3e47 commit 42eca7c
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions pkgs/applications/version-management/subversion/default.nix
View file
Expand Up @@ -91,11 +91,12 @@ let

doCheck = false; # fails 10 out of ~2300 tests

meta = {
meta = with stdenv.lib; {
description = "A version control system intended to be a compelling replacement for CVS in the open source community";
license = licenses.asl20;
homepage = http://subversion.apache.org/;
maintainers = with stdenv.lib.maintainers; [ eelco lovek323 ];
platforms = stdenv.lib.platforms.linux ++ stdenv.lib.platforms.darwin;
maintainers = with maintainers; [ eelco lovek323 ];
platforms = platforms.linux ++ platforms.darwin;
};

} // stdenv.lib.optionalAttrs stdenv.isDarwin {
Expand Down

3 comments on commit 42eca7c

@pSub
Copy link
Member

@pSub pSub commented on 42eca7c Aug 9, 2018

Choose a reason for hiding this comment

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

FYI: I already added the license in commit eaf2786 which is currently in PR #44358. I try to tick all boxes in issue #43716, so that we do no duplicate work.

@orivej
Copy link
Contributor Author

@orivej orivej commented on 42eca7c Aug 9, 2018

Choose a reason for hiding this comment

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

OK. I noticed that subversion_1_10 and subversionClient were not ticked, so I have commited this and ticked them.

@pSub
Copy link
Member

@pSub pSub commented on 42eca7c Aug 9, 2018

Choose a reason for hiding this comment

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

Ok, then it seems I forgot to tick them in this case. Thank you anyways!

Please sign in to comment.