unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: ludo@gnu.org (Ludovic Courtès)
To: Hartmut Goebel <h.goebel@crazy-compilers.com>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] gnu: Add tipp10 touch typing tutor.
Date: Mon, 19 Dec 2016 22:39:24 +0100	[thread overview]
Message-ID: <87poknzjhv.fsf@gnu.org> (raw)
In-Reply-To: <1482074241-30845-1-git-send-email-h.goebel@crazy-compilers.com> (Hartmut Goebel's message of "Sun, 18 Dec 2016 16:17:21 +0100")

Hi!

Hartmut Goebel <h.goebel@crazy-compilers.com> skribis:

> * gnu/packages/education.scm (touch10): New variable.
> * gnu/packages/patches/tipp10-FixCompiling.patch,
>   gnu/packages/patches/tipp10-RemoveLicenseCode.patch: New files.
> * gnu/local.mk (dist_patch_DATA): Register them.

[...]

> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append "mirror://debian/pool/main/"
> +                                  "t/tipp10/tipp10_2.1.0.orig.tar.gz"))

Is there really no upstream for this program?  tipp10.com seems to be
live, no?

> +              (patches (search-patches "tipp10-FixCompiling.patch"
> +                                       "tipp10-RemoveLicenseCode.patch"))))

Please use lowercase and hyphens for file names.  :-)

> +    (home-page "https://www.tipp10.com/")
> +    (synopsis "Touch typing tutor")
> +    (description " TIPP10 is a free touch typing tutor for Windows, Mac OS and
                     ^
Extra space.

Please don’t mention supported operating systems; what matters is that
it works on GNU.

> +Linux.  The ingenious thing about the software is its intelligence feature.

“Intelligence feature” sounds vague; could it be rephrased in more
precise terms?

> +Note: To change the language settings choose Datei (File) →
> +Grundeinstellungen (Generell Settings) → Sprache (Language) and change from
> +Deutsch to English. The you have restart the program to have the change take
> +effect.")

This information doesn’t belong here.

Does Debian have patches to make this program honor the current locale?
That would be the best option.

> +    (license license:gpl2)))

Version 2 only?

> +--- a/tipp10.desktop
> ++++ b/tipp10.desktop
> +@@ -1,10 +1,10 @@
> + [Desktop Entry]
> +-Encoding=UTF-8
> + Name=TIPP10
> + Comment=Touch Typing Tutor
> + Comment[de]=10-Finger-Schreibtrainer
> +-Exec=tipp10
> ++Exec=/usr/bin/tipp10

This doesn’t look useful.

Could you send an updated version?

Thank you!

Ludo’.

  reply	other threads:[~2016-12-19 21:39 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-18 15:17 [PATCH] gnu: Add tipp10 touch typing tutor Hartmut Goebel
2016-12-19 21:39 ` Ludovic Courtès [this message]
2016-12-31 14:36   ` Hartmut Goebel
2017-01-05 10:55     ` Ludovic Courtès
2016-12-31 14:50 ` Hartmut Goebel
2017-01-05 10:56   ` Ludovic Courtès
2017-01-14 19:18     ` Hartmut Goebel

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=87poknzjhv.fsf@gnu.org \
    --to=ludo@gnu.org \
    --cc=guix-devel@gnu.org \
    --cc=h.goebel@crazy-compilers.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 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).