Path: csiph.com!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod From: Paul Wise Newsgroups: linux.debian.maint.python Subject: Re: Looking for a Sponsor - Papereshaper Date: Thu, 31 Dec 2020 02:10:03 +0100 Message-ID: References: X-Mailbox-Line: From debian-python-request@lists.debian.org Thu Dec 31 01:03:25 2020 Old-Return-Path: X-Amavis-Spam-Status: No, score=-6.398 tagged_above=-10000 required=5.3 tests=[BAYES_00=-2, FOURLA=0.1, FREEMAIL_FORGED_FROMDOMAIN=0.249, FREEMAIL_FROM=0.001, HEADER_FROM_DIFFERENT_DOMAINS=0.25, LDO_WHITELIST=-5, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001] autolearn=ham autolearn_force=no X-Policyd-Weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 CL_IP_EQ_HELO_IP=-2 (check from: .gmail. - helo: .mail-ot1-f48.google. - helo-domain: .google.) FROM/MX_MATCHES_HELO(DOMAIN)=-2; rate: -5.5 X-Google-Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=/GfGLy7A+PaqymBL8yRmoT8y0DJHwGuHbgKAQGyaGYQ=; b=GsWhM04Kk/yYIT9+xwpTtWpiNXf2Kn/p3MIQbVh63KSh8MqufWtJYbpEWF8pqavNs2 I/+ILZ3HRcY5hXjqWC1CfTNo6gZIY8RwSQhaHupPQqMIu5I7u2a40sO6fBeNHAASIkYZ zUSiiaSRIJcGI8jVIMcNM37m34mpXYExoTEuQ4Ek4R7rhS/864HlHRJjZOnMMQ0OcmRz WvjU+aAMaU1srb0GzQcAzrIgrAucPiPt4Fz1KOMx/szFQtlVSVxY8djoAarh0NrMOWjO 2qswrlYtaFWNEFKn3sYBQ+ZGXPb9IPvaSqFismpD4o3bknphrpbS+wFCVLK3+bYy6qfC PoOA== X-Gm-Message-State: AOAM530+1Hg04HduxNzLFkRwa8peU6FyNL38x6R8HeXUUMhPLCtknsXP BQj1Gwd1eLriyQSgl9OdeocBYmhfDLtufYXhgwk6rofocbU= X-Google-SMTP-Source: ABdhPJxjW6kGxl2Tdv7byjSdsOw0LeizBdS2NFyw6eLHecL9fH9SlFd0uFeLU8fA/3b4yR+WtnDBABS7qIVMnKP4Bss= X-Received: by 2002:a9d:6d08:: with SMTP id o8mr40708382otp.334.1609376588169; Wed, 30 Dec 2020 17:03:08 -0800 (PST) MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" X-Mailing-List: archive/latest/18866 List-ID: List-URL: List-Archive: https://lists.debian.org/msgid-search/CAKTje6FpTXAby0PpE8FPwibqkLc=u8Pg+k3ey1+qZToR_Dq2ug@mail.gmail.com Approved: robomod@news.nic.it Lines: 71 Organization: linux.* mail to news gateway Sender: robomod@news.nic.it X-Original-Cc: debian-python@lists.debian.org X-Original-Date: Thu, 31 Dec 2020 01:02:57 +0000 X-Original-Message-ID: X-Original-References: <1262d7a2-5a19-dca7-8842-8276de343b47@pkcarlisle.com> Xref: csiph.com linux.debian.maint.python:13092 On Wed, Dec 30, 2020 at 4:50 PM Devops PK Carlisle LLC wrote: > The git is here: https://github.com/pkcarlislellc/git-papershaper I don't intend to package nor sponsor this, but here is a review: Drop git- from the name of the GitHub repository. Add a git repository to the SourceForge project also and push to both repos. Remove the possibly non-free and or non-redistributable images in the git repository. If they are actually redistributable and freely licensed, please document the location where they were obtained from, their copyright owner and license. If they are non-redistributable then you will need to rewrite the history of the git repository to exclude them. Merge the two scripts into one supporting both versions of Python (or just drop the Python 2 one). IIRC you can use the print function in Python 2 by adding the line below. After that the remaining changes don't look necessary, or can be made conditional with sys.version_info. from future import print_function The first thing in the README.md should be a one-sentence description of what the program does. Drop the Google Code mention/link from the README.md. The README.md mentions that webcamgrab.txt was taken from another program, but doesn't mention the copyright/license information for the file anywhere. While the SourceForge project for the other program says GPLv3, the tarball containing the list of webcams doesn't mention any license. Change the GPL reference at the start of the README.md to mention which versions. The list of webcams is liable to get out of date, especially when shipped with the script, it might be a good idea to have continuous monitoring of the list, plus a way to auto-update it on end-user systems. Consider switching to the XDG standard for some of the paths: https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html https://wiki.archlinux.org/index.php/XDG_Base_Directory https://github.com/srstevenson/xdg When I run check-all-the-things in the source tree, a number of the checks produce output that is worth reviewing. https://github.com/collab-qa/check-all-the-things/ grep -ri gimp codespell doc8 grep -r http: proselint spellintian pycodestyle pyflakes pylint vulture -- bye, pabs https://wiki.debian.org/PaulWise