unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "pelzflorian (Florian Pelz)" <pelzflorian@pelzflorian.de>
To: Efraim Flashner <efraim@flashner.co.il>
Cc: guix-devel@gnu.org, pjotr2019@thebird.nl
Subject: Re: Mailman packaging (was: Re: Python package naming: Dots vs hyphens)
Date: Mon, 18 Nov 2019 14:14:12 +0100	[thread overview]
Message-ID: <20191118131412.e6zjtw5tx6pnznl7@pelzflorian.localdomain> (raw)
In-Reply-To: <20191118101351.GH15074@E5400>

Thank you for your work and for looking at projects’ mailing lists
etc. :)  I still have not set up mailman on my server, but am still
going to work on it.  I agree everywhere except the non-free emails
and a typo in a description, see below.


On Mon, Nov 18, 2019 at 12:13:51PM +0200, Efraim Flashner wrote:
> On Wed, Nov 06, 2019 at 02:18:33PM +0100, pelzflorian (Florian Pelz) wrote:
> > You have:
> > 
> > (define-public mailman
> >   (package
> >     ...
> >     (description
> >      "GNU Mailman is software for managing email discussion and mailing
> > lists.  Both users and administrators generally perform their actions in a
> > web interface, although email and command-line interfaces are also provided.
> > The system features built-in archiving, automatic bounce processing, content
> > filtering, digest delivery, and more.")
> > 
> > I have:
> > 
> >     (description "This is the core engine for the GNU Mailing List manager. It
> > is part of the full GNU Mailman 3 suite.  Mailman is the GNU Mailing List
> > Manager, a program that manages electronic mail discussion groups.")
> > 
> > Maybe your mailman description is more appropriate later for a service
> > in doc/guix.texi instead of this package?  Feel free to disagree.
> > 
> 
> I kept mine for now
> 

OK.  It is good for finding the main mailman package, I agree.



> > You have:
> > 
> > (define-public python-flufl.bounce
> >   ...
> >   (description "The @code{flufl.bounce} library provides a set of heuristics
> > and an API for detecting the original bouncing email addresses from a bounce
> > message.  Many formats found in the wild are supported, as are VERP and
> > RFC 3464.")
> >     (license license:asl2.0)))
> > 
> > 
> > I have the same package inputs etc., except:
> > 
> >     (synopsis "Email bounce detectors")
> >     (description "The @dfn{flufl.bounce} library provides a set of heuristics
> > and an API for detecting the original bouncing email addresses from a bounce
> > message.  Many formats found in the wild are supported, as are VERP and RFC
> > 3464 (DSN).")
> >     (license (list license:asl2.0 ;except for an emails used as test data
> >                                   ;which may be non-free
> >                    license:lgpl3)))) ;only for setup_helpers.py
> > 
> > I prefer your description without DSN at the end, but I prefer my
> > license list.
> > 
> 
> According to Debian the test emails are OK
> https://sources.debian.org/src/flufl.bounce/3.0-1/debian/copyright/
> 

AFAIU non-free e-mails as test data are not a problem for the FSDG
anyway, but I am not convinced the authors listed in that Debian
copyright file have asked the e-mails’ authors for permission.  I
cannot decide if a comment like

> >     (license (list license:asl2.0 ;except for an emails used as test data
> >                                   ;which may be non-free

is necessary though.

Further below:

> > I have:
> > 
> > (define-public python-dkimpy
> >   […]
> >     (license
> >      license:zlib)));except for an email used as test data which may be non-free
> > 
> > Yours seems much better, except the license header.
> > 
> >

This is the same as the above test data issue.

I continue:



> > You have:
> > 
> > (define-public python-flufl.lock
> >   ...
> >     (synopsis "NFS-safe file locking with timeouts for POSIX systems")
> >     (description
> >      "This package provides NFS-safe file locking with timeouts for POSIX systems.")
> >     (license license:asl2.0)))
> > 
> > I have:
> > 
> >     (synopsis
> >      "NFS-safe file locking with timeouts for POSIX systems.")
> >     (description
> >      "The @dfn{flufl.lock} package provides NFS-safe file locking with
> > timeouts for POSIX systems.  It is similar to the @code{O_EXCL} option of the
> > @code{open} system call but uses a lockfile.  Lock objects support
> > lock-breaking so that you can’t wedge a process forever.  Locks have a
> > lifetime, which is the maximum length of time the process expects to retain
> > the lock.")
> >     (license (list license:asl2.0
> >                    license:lgpl3)))) ;only for setup_helpers.py
> > 
> > I think my description is too long and yours is too short.  I prefer
> > my license list.
> 
> I took the first two sentences of yours and then combined the last two.
> 

Good. :)  You are missing a plural s in the last sentence now though:

    (description
     "The @dfn{flufl.lock} package provides NFS-safe file locking with
timeouts for POSIX systems.  It is similar to the @code{O_EXCL} option of the
@code{open} system call but uses a lockfile.  Lock object support lock-breaking
and have a maximum lifetime built-in.")





> > 
> > 
> > You have:
> > 
> > (define-public python-importlib-resources
> >   ...
> > 
> > I have no such package because it is no longer needed, I think.  I
> > have not packaged all dependencies yet so I cannot try building
> > mailman.  You made a comment in your mailman package that
> > importlib-resources is no longer needed.
> > 
> 
> I tried to build mailman without it but it complained that it was
> missing.
> 

You are right.  Changing this would require changes to many files.
Your importlib_resources package looks fine.


> > I have:
> > 
> > (define-public python-py3dns
> >   ...
> >     […]
> >     ;; license variant is the CNRI License Agreement for Python:
> >     (license license:psfl)))
> > 
> > Yours is better except the description and possibly license.
> 
> The license is already installed to %out/share/doc/name-version/LICENSE

OK, I agree, such comments are superfluous.

> I think I intended to come back for the description later :)
> 

Regards,
Florian

  reply	other threads:[~2019-11-18 13:14 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-06  6:49 Python package naming: Dots vs hyphens pelzflorian (Florian Pelz)
2019-11-06  7:19 ` Efraim Flashner
2019-11-06 13:13   ` pelzflorian (Florian Pelz)
2019-11-07  7:34     ` Efraim Flashner
2019-11-06 13:18   ` Mailman packaging (was: Re: Python package naming: Dots vs hyphens) pelzflorian (Florian Pelz)
2019-11-13 22:16     ` pelzflorian (Florian Pelz)
2019-11-18 10:13     ` Efraim Flashner
2019-11-18 13:14       ` pelzflorian (Florian Pelz) [this message]
2019-11-20 16:36       ` Pjotr Prins
2021-09-29 16:34       ` Christine Lemmer-Webber
2021-09-30  7:54         ` pelzflorian (Florian Pelz)
2019-11-06 13:53   ` Python package naming: Dots vs hyphens Tobias Geerinckx-Rice
2019-11-10 10:57 ` Andreas Enge
2019-11-11  9:30   ` Efraim Flashner
2019-11-11 10:24     ` pelzflorian (Florian Pelz)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191118131412.e6zjtw5tx6pnznl7@pelzflorian.localdomain \
    --to=pelzflorian@pelzflorian.de \
    --cc=efraim@flashner.co.il \
    --cc=guix-devel@gnu.org \
    --cc=pjotr2019@thebird.nl \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).