https://bugzilla.redhat.com/show_bug.cgi?id=1458441 --- Comment #1 from Alfredo Moralejo <amoralej@xxxxxxxxxx> --- Some comments: - About adding unit tests in %check, unit tests are broken in tarball from pipy as described in https://bitbucket.org/mhallin/py-scrypt/issues/21/test-breaks-importerror-no-module-named - This package is bundling scrypt release 1.2.0 from https://github.com/Tarsnap/scrypt. As scrypt does not exist in fedora, i understand this is ok and we can go on without using the "Provides: bundled(scrypt)" as shown in https://fedoraproject.org/wiki/Bundled_Libraries?rd=Packaging:Bundled_Libraries - Package doesn't have a LICENSE file upstream. It'd be nice if it would have one, but license seems correctly set in setup.py -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx