all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Csepp <raingloom@riseup.net>
To: Liliana Marie Prikler <liliana.prikler@gmail.com>
Cc: 54691@debbugs.gnu.org, control@debbugs.gnu.org,
	Csepp <raingloom@riseup.net>
Subject: bug#54691: [PATCH] gnu: fortune-mod: Remove off database.
Date: Sun, 17 Jul 2022 15:05:09 +0200	[thread overview]
Message-ID: <87r12jsx0m.fsf@riseup.net> (raw)
In-Reply-To: <5dfc6d81f8b0600cde73254c42323a2c79c0453c.camel@gmail.com>


Liliana Marie Prikler <liliana.prikler@gmail.com> writes:

> Am Samstag, dem 16.07.2022 um 15:40 +0200 schrieb Csepp:
>> 
>> Liliana Marie Prikler <liliana.prikler@student.tugraz.at> writes:
>> 
>> > merge 56599 54691
>> > thanks
>> > 
>> > Am Samstag, dem 16.07.2022 um 14:21 +0200 schrieb raingloom:
>> > > * gnu/packages/games.scm (fortune-mod): Delete off database after
>> > > install phase.
>> > > ---
>> > >  gnu/packages/games.scm | 5 ++++-
>> > >  1 file changed, 4 insertions(+), 1 deletion(-)
>> > > 
>> > > diff --git a/gnu/packages/games.scm b/gnu/packages/games.scm
>> > > index 713f9339b2..cf10ebc9b6 100644
>> > > --- a/gnu/packages/games.scm
>> > > +++ b/gnu/packages/games.scm
>> > > @@ -7721,7 +7721,10 @@ (define-public fortune-mod
>> > >                   (lambda _
>> > >                     (with-directory-excursion #$output
>> > >                       (rename-file "games/fortune" "bin/fortune")
>> > > -                     (rmdir "games")))))))
>> > > +                     (rmdir "games"))))
>> > > +               (add-after 'install 'commit-censorship
>> > > +                 (lambda* (#:key outputs #:allow-other-keys)
>> > > +                   (delete-file-recursively (string-append
>> > Truly offensive files should be removed with a snippet rather than
>> > a funnily named phase.  I'm not sure if this applies to all of off,
>> > but let it be said, that there's plenty of fortunes that'd violate
>> > the CoC in files not labeled as such.  Some files could probably be
>> > salvaged if we only removed some quotes – for example limericks
>> > about numbers are probably fine, but other limericks aren't. 
>> > Furthermore, we should also revise the list of fortunes not in off,
>> > men-women is just an alternate spelling of sexism.
>> > 
>> > Cheers
>> 
>> That's probably better done as a fork of upstream. If no one wants to
>> do that, I wouldn't miss the package if it got yeeted. :shrug:
> I'm not necessarily asking you to go over joke for joke; that would
> need to be a collective decision of Guix folk anyway.  Right now I just
> want to (1) make that a snippet and (2) also take a critical look at
> the files not labeled "offensive". 
>
> Cheers

Doing it before the install phase requires also patching the build
scripts.  There are a bunch of other bugs calling me sadly, so I'm not
super keen on putting more energy into this one.




  reply	other threads:[~2022-07-17 13:14 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-16 12:21 [bug#56599] [PATCH] gnu: fortune-mod: Remove off database raingloom
2022-07-16 13:32 ` bug#54691: " Liliana Marie Prikler
2022-07-16 13:40   ` Csepp
2022-07-16 13:46     ` Liliana Marie Prikler
2022-07-17 13:05       ` Csepp [this message]
2022-08-03 15:02 ` [bug#56599] " kiasoc5 via Guix-patches via

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

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

  git send-email \
    --in-reply-to=87r12jsx0m.fsf@riseup.net \
    --to=raingloom@riseup.net \
    --cc=54691@debbugs.gnu.org \
    --cc=control@debbugs.gnu.org \
    --cc=liliana.prikler@gmail.com \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.